This commit refactors the logrotate tasks by renaming them for better readability and consistency, and refactoring task names in `subscription.yml` for improved organization and clarity.
This commit introduces a new script `pve-remove-nag.sh` designed to remove nag messages from both the Proxmox VE web and mobile user interfaces. The script is refactored to improve installation process, and the UI patching logic has been temporarily commented out for further refinement.
Removed unnecessary comments from the codebase and updated references to the latest commit. This refactoring improves readability and ensures consistency with the current version control.
This commit includes the installation of `pve-remove-nag.sh` script, checks for legacy `proxmoxlib.js`, and ensures that `/usr/local/bin` exists. Additionally, it sets up an APT post-invoke hook to run necessary tasks after package installations.
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 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.
