This commit updates the repository configuration by commenting out enterprise-specific lines and enabling the Proxmox no-subscription repository to simplify setup for users without a subscription.
This commit refactors the task names to improve readability and adds a namespace to avoid conflicts with other modules. It also fixes typos in task names and updates module references accordingly.
no-handler::Tasks that run when changed should likely be handlers
This commit refactors the existing logic for patching legacy and minified proxmoxlib.js files, improving readability and maintainability. It also adds new handlers to store checksums and restart pveproxy.
This commit addresses the issue by adding necessary checks to `proxmoxlib.js` and `proxmoxlib.min.js` after applying patches. Additionally, it updates the checksum storage mechanism to ensure integrity and consistency.
This commit introduces a new feature that ensures the existence of a nag patch state directory. It also reads and stores checksums for both legacy and minified proxmoxlib.js files, applying patches only when necessary.
No blind replaces
No silent failures
Upgrade-safe
Self-healing
Predictable failure mode
This refactoring updates the usage of `stat` modules in our Ansible playbooks to utilize the `ansible.builtin.stat` module. This change ensures consistency and potentially improves compatibility with newer versions of Ansible.
This refactoring ensures that the subscription task properly checks for and removes nag messages in both legacy and minified JavaScript files, enhancing robustness by handling cases where files may not exist.
✅ No silent failures
✅ Safe if file doesn’t exist
❌ Hard fail if Proxmox changes the JS logic and the patch no longer applies
✅ Clear signal that the role needs updating after a major upgrade
This commit updates the Proxmox repository configuration to include specific file permissions and ownership settings. This ensures that the repository is properly secured and accessible by authorized users.
✔ Explicit ownership
✔ Explicit permissions
✔ Idempotent
This commit refactors task names across multiple files to improve clarity and maintain consistency, ensuring that each task name accurately reflects its purpose and using the latest `ansible.builtin` modules where applicable.
This refactoring includes updating the README, changing the role name to focus on Proxmox VE, enhancing compatibility matrix, and refactoring default values for role variables. Handlers are updated to manage apt cache, restart pveproxy, and reload systemd. Role metadata is also updated for better Proxmox VE provisioning.
