Commit Graph

145 Commits

Author SHA1 Message Date
447bdfb2e1 refactor ♻️: Rename task from 'Install, auto-tune, and make it persistent' to 'Install powrtop, auto-tune, and make it persistent'
Refactored the task name to include the specific tool being installed (powrtop) for clarity.
2026-02-12 17:55:44 +01:00
f285e63dfc Merge pull request 'chore 📦: Add environment variable to powertop.service.j2' (#8) from dev into main
All checks were successful
ansible-lint / Ansible Lint (push) Successful in 12s
Gitleaks Scan / gitleaks (push) Successful in 5s
Markdown Lint / markdown-lint (push) Successful in 5s
Mark stale issues and pull requests / stale (push) Successful in 3s
Reviewed-on: #8
2026-02-10 20:56:04 +01:00
d2b127c70c chore 📦: Add environment variable to powertop.service.j2
All checks were successful
ansible-lint / Ansible Lint (push) Successful in 17s
Gitleaks Scan / gitleaks (push) Successful in 4s
Markdown Lint / markdown-lint (push) Successful in 7s
ai-reviews / Review PR (pull_request) Successful in 12s
ansible-lint / Ansible Lint (pull_request) Successful in 11s
Gitleaks Scan / gitleaks (pull_request) Successful in 4s
Markdown Lint / markdown-lint (pull_request) Successful in 5s
This commit adds an environment variable to the powertop service configuration file. This change is necessary to ensure that power management settings are applied correctly during system startup.
2026-02-10 20:54:35 +01:00
9f29808a6a Merge pull request 'fix 🐛: Fix typo and add file permission in logrotate.yml task' (#7) from dev into main
All checks were successful
ansible-lint / Ansible Lint (push) Successful in 11s
Gitleaks Scan / gitleaks (push) Successful in 4s
Markdown Lint / markdown-lint (push) Successful in 5s
Mark stale issues and pull requests / stale (push) Successful in 3s
Reviewed-on: #7
2026-02-09 18:25:09 +01:00
3e6e1cb893 fix 🐛: Fix typo and add file permission in logrotate.yml task
All checks were successful
ansible-lint / Ansible Lint (push) Successful in 13s
Gitleaks Scan / gitleaks (push) Successful in 5s
Markdown Lint / markdown-lint (push) Successful in 5s
ai-reviews / Review PR (pull_request) Successful in 37s
ansible-lint / Ansible Lint (pull_request) Successful in 12s
Gitleaks Scan / gitleaks (pull_request) Successful in 9s
Markdown Lint / markdown-lint (pull_request) Successful in 26s
This commit fixes a typo in the logrotate configuration file and adds necessary file permissions to ensure proper rotation of log files.
2026-02-09 18:23:01 +01:00
abdba53053 refactor ♻️: Refactor logrotate tasks for better readability and consistency - Rename 'logrotate reload' task to 'Logrotate reload' - Refactor task names in subscription.yml for better organization and clarity
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 11s
Gitleaks Scan / gitleaks (push) Successful in 5s
Markdown Lint / markdown-lint (push) Successful in 5s
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.
2026-02-09 18:20:19 +01:00
75cff9590d feat : Add new script pve-remove-nag.sh for removing Nag messages in PVE UIs
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 16s
Gitleaks Scan / gitleaks (push) Successful in 4s
Markdown Lint / markdown-lint (push) Successful in 5s
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.
2026-02-09 18:12:13 +01:00
b42e72a835 refactor ♻️: Refactor Proxmox repo management tasks, add comments, and ensure apt update is triggered on changes.
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 7s
Gitleaks Scan / gitleaks (push) Successful in 5s
Markdown Lint / markdown-lint (push) Successful in 5s
This commit refactors the Proxmox repository management tasks by adding necessary comments for better understanding. It also ensures that `apt update` is triggered whenever there are changes to the repository configuration.
2026-02-09 18:07:22 +01:00
7227f51790 Merge pull request 'style 💎: Remove redundant comment and update commit reference' (#6) from dev into main
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 6s
Gitleaks Scan / gitleaks (push) Successful in 4s
Markdown Lint / markdown-lint (push) Successful in 5s
Mark stale issues and pull requests / stale (push) Successful in 3s
Reviewed-on: #6
2026-02-08 20:13:52 +01:00
ceeca6e7e0 Merge pull request 'refactor ♻️: Refactor Proxmox repository management' (#5) from dev into main
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 6s
Gitleaks Scan / gitleaks (push) Successful in 4s
Markdown Lint / markdown-lint (push) Successful in 5s
Reviewed-on: #5
2026-02-08 20:09:56 +01:00
ae15b76575 style 💎: Remove redundant comment and update commit reference
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 6s
Gitleaks Scan / gitleaks (push) Successful in 4s
Markdown Lint / markdown-lint (push) Successful in 4s
ai-reviews / Review PR (pull_request) Successful in 10s
ansible-lint / Ansible Lint (pull_request) Failing after 6s
Gitleaks Scan / gitleaks (pull_request) Successful in 4s
Markdown Lint / markdown-lint (pull_request) Successful in 5s
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.
2026-02-08 20:07:33 +01:00
85e021c864 chore 📦: Add setup scripts for /usr/local/bin and APT post-invoke hook
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 6s
ai-reviews / Review PR (pull_request) Successful in 28s
ansible-lint / Ansible Lint (pull_request) Failing after 6s
Gitleaks Scan / gitleaks (push) Successful in 5s
Gitleaks Scan / gitleaks (pull_request) Successful in 4s
Markdown Lint / markdown-lint (pull_request) Successful in 5s
Markdown Lint / markdown-lint (push) Successful in 5s
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.
2026-02-08 20:03:53 +01:00
f96b3da66e refactor ♻️: Refactor variable and comment formatting in repos.yml
Some checks failed
ansible-lint / Ansible Lint (push) Has been cancelled
Gitleaks Scan / gitleaks (push) Has been cancelled
Markdown Lint / markdown-lint (push) Has been cancelled
ai-reviews / Review PR (pull_request) Successful in 9s
ansible-lint / Ansible Lint (pull_request) Failing after 11s
Gitleaks Scan / gitleaks (pull_request) Successful in 4s
Markdown Lint / markdown-lint (pull_request) Successful in 5s
Updated variable names and comments for better readability and consistency.
2026-02-08 20:02:25 +01:00
e680fdf162 refactor ♻️: Refactor Proxmox repository management
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 5s
Gitleaks Scan / gitleaks (push) Successful in 5s
Markdown Lint / markdown-lint (push) Successful in 5s
ai-reviews / Review PR (pull_request) Successful in 16s
ansible-lint / Ansible Lint (pull_request) Failing after 6s
Gitleaks Scan / gitleaks (pull_request) Successful in 4s
Markdown Lint / markdown-lint (pull_request) Successful in 5s
Introduce a variable for repo path and update regex patterns to improve code readability and maintainability.
2026-02-08 20:00:43 +01:00
c139461cef refactor ♻️: Refactor Proxmox repository management
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 6s
Gitleaks Scan / gitleaks (push) Successful in 4s
Markdown Lint / markdown-lint (push) Successful in 4s
Conditionally comment out and uncomment repositories, ensuring `apt update` is run only when necessary.
2026-02-08 19:49:12 +01:00
ba322dd97b Merge pull request 'refactor ♻️: Refactor logrotate configuration for dynamic handling of multiple files' (#4) from dev into main
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 6s
Gitleaks Scan / gitleaks (push) Successful in 5s
Markdown Lint / markdown-lint (push) Successful in 5s
Reviewed-on: #4
2026-02-08 19:47:25 +01:00
f33d1563b7 style 💎: Remove unnecessary blank line from logrotate.yml
Cleaned up the configuration file by removing an extra blank line that was not necessary.
2026-02-08 19:46:25 +01:00
4a02a41d43 refactor ♻️: Refactor logrotate configuration for dynamic handling of multiple files
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 8s
Gitleaks Scan / gitleaks (push) Successful in 6s
Markdown Lint / markdown-lint (push) Successful in 5s
ai-reviews / Review PR (pull_request) Successful in 43s
ansible-lint / Ansible Lint (pull_request) Failing after 6s
Gitleaks Scan / gitleaks (pull_request) Successful in 4s
Markdown Lint / markdown-lint (pull_request) Successful in 5s
Replaced legacy patches with a single logrotate reload command. Enhanced logrotate configuration to dynamically handle multiple files and include improved settings and notifications.
2026-02-08 19:42:44 +01:00
daf4d74d93 Merge pull request 'refactor ♻️: Refactor task name and comment' (#3) from dev into main
All checks were successful
ansible-lint / Ansible Lint (push) Successful in 11s
Gitleaks Scan / gitleaks (push) Successful in 4s
Markdown Lint / markdown-lint (push) Successful in 5s
Reviewed-on: #3
2026-02-08 08:09:07 +01:00
e033fccbdf refactor ♻️: Refactor task name and comment
All checks were successful
ansible-lint / Ansible Lint (push) Successful in 12s
Gitleaks Scan / gitleaks (push) Successful in 5s
Markdown Lint / markdown-lint (push) Successful in 5s
ai-reviews / Review PR (pull_request) Successful in 9s
ansible-lint / Ansible Lint (pull_request) Successful in 11s
Gitleaks Scan / gitleaks (pull_request) Successful in 4s
Markdown Lint / markdown-lint (pull_request) Successful in 4s
Renamed the task from 'task' to 'repos | Comment out Proxmox enterprise repo lines' and updated the accompanying comment for clarity.
2026-02-08 08:07:37 +01:00
c5cbedf314 Merge pull request 'refactor ♻️: Refactor legacy and minified proxmoxlib.js patching logic to use handlers' (#2) from dev into main
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 11s
Gitleaks Scan / gitleaks (push) Successful in 5s
Markdown Lint / markdown-lint (push) Successful in 5s
Reviewed-on: #2
2026-02-08 08:06:27 +01:00
410f87bb2c chore 📦: Comment out enterprise repo lines and enable Proxmox no-subscription repo
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 12s
Gitleaks Scan / gitleaks (push) Successful in 4s
ai-reviews / Review PR (pull_request) Successful in 27s
Markdown Lint / markdown-lint (push) Successful in 5s
ansible-lint / Ansible Lint (pull_request) Failing after 11s
Gitleaks Scan / gitleaks (pull_request) Successful in 4s
Markdown Lint / markdown-lint (pull_request) Successful in 5s
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.
2026-02-08 08:04:32 +01:00
f51de33275 fix 🐛: Fix typo in task name
All checks were successful
ansible-lint / Ansible Lint (push) Successful in 12s
Gitleaks Scan / gitleaks (push) Successful in 5s
ai-reviews / Review PR (pull_request) Successful in 27s
Markdown Lint / markdown-lint (push) Successful in 5s
ansible-lint / Ansible Lint (pull_request) Successful in 11s
Gitleaks Scan / gitleaks (pull_request) Successful in 4s
Markdown Lint / markdown-lint (pull_request) Successful in 5s
Corrected a typographical error in the task name to ensure accurate documentation and clarity.
2026-02-08 07:59:26 +01:00
1f2ff1d9ba refactor ♻️: Refactor task names and add namespace for clarity
All checks were successful
ansible-lint / Ansible Lint (push) Successful in 12s
Gitleaks Scan / gitleaks (push) Successful in 4s
Markdown Lint / markdown-lint (push) Successful in 5s
ai-reviews / Review PR (pull_request) Successful in 24s
ansible-lint / Ansible Lint (pull_request) Successful in 12s
Gitleaks Scan / gitleaks (pull_request) Successful in 5s
Markdown Lint / markdown-lint (pull_request) Successful in 5s
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.
2026-02-08 07:12:21 +01:00
12b1b87af1 refactor ♻️: Refactor legacy and minified proxmoxlib.js patching logic to use handlers
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 12s
Gitleaks Scan / gitleaks (push) Successful in 5s
Markdown Lint / markdown-lint (push) Successful in 5s
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.
2026-02-08 07:07:23 +01:00
afc80dc57d refactor ♻️: Rename 'task' to 'Read checksum after patch'
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 15s
Gitleaks Scan / gitleaks (push) Successful in 5s
Markdown Lint / markdown-lint (push) Successful in 5s
Refactored the task name to more accurately reflect its purpose, updating the description for clarity.
2026-02-08 06:44:18 +01:00
1ee2eced45 test 🧪: Add todo molecule tests
Updates several TODO items in the TODO to reflect completed tasks and areas that still need attention.
2026-02-08 06:44:01 +01:00
f16d6a62b7 fix 🐛: updates the checksum storage mechanism to ensure idempotency.
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 11s
Gitleaks Scan / gitleaks (push) Successful in 5s
Markdown Lint / markdown-lint (push) Successful in 6s
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.
2026-02-07 18:54:32 +01:00
85ce6e0044 Merge pull request 'feat : NAG checksum-based detection (auto-repatch after upgrades)' (#1) from dev into main
All checks were successful
ansible-lint / Ansible Lint (push) Successful in 11s
Gitleaks Scan / gitleaks (push) Successful in 5s
Markdown Lint / markdown-lint (push) Successful in 5s
Mark stale issues and pull requests / stale (push) Successful in 3s
Reviewed-on: #1
2026-02-07 18:42:42 +01:00
e0831c473e refactor ♻️: Refactor task to use ansible.builtin.slurp for compatibility
All checks were successful
ansible-lint / Ansible Lint (push) Successful in 11s
Gitleaks Scan / gitleaks (push) Successful in 4s
Markdown Lint / markdown-lint (push) Successful in 5s
ai-reviews / Review PR (pull_request) Successful in 19s
ansible-lint / Ansible Lint (pull_request) Successful in 11s
Gitleaks Scan / gitleaks (pull_request) Successful in 4s
Markdown Lint / markdown-lint (pull_request) Successful in 5s
Updated the task to utilize `ansible.builtin.slurp` instead of the deprecated `slurp` module for improved compatibility with newer Ansible versions.
2026-02-07 18:36:33 +01:00
4d73c5a844 style 💎: Fix indentation and logic in subscription.yml tasks
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 11s
Gitleaks Scan / gitleaks (push) Successful in 5s
Markdown Lint / markdown-lint (push) Successful in 5s
Refactored the indentation and corrected logical errors in the tasks section of subscription.yml to improve readability and functionality.
2026-02-07 18:34:05 +01:00
7907e71a5e feat : Add task to ensure nag patch state directory exists and manage checksums for legacy and minified proxmoxlib.js files
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 6s
Gitleaks Scan / gitleaks (push) Successful in 4s
Markdown Lint / markdown-lint (push) Successful in 6s
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
2026-02-07 18:32:40 +01:00
91a6b9eb2c docs 📝: Fix table formatting in README.md
All checks were successful
ansible-lint / Ansible Lint (push) Successful in 11s
Gitleaks Scan / gitleaks (push) Successful in 5s
Markdown Lint / markdown-lint (push) Successful in 5s
Updated the table formatting in the README.md file for better readability and consistency.
2026-02-07 09:43:10 +01:00
6b2645edac docs 📝: Add spaces between table cells in README.md
Some checks failed
ansible-lint / Ansible Lint (push) Successful in 11s
Gitleaks Scan / gitleaks (push) Successful in 5s
Markdown Lint / markdown-lint (push) Failing after 4s
Updated the README.md file by adding spaces between table cells to improve readability and formatting.
2026-02-07 09:37:02 +01:00
4d282f6b7d fix 🐛: Fix typo in handlers/main.yml
Some checks failed
ansible-lint / Ansible Lint (push) Successful in 11s
Gitleaks Scan / gitleaks (push) Successful in 4s
Markdown Lint / markdown-lint (push) Failing after 4s
Corrected a typographical error in the main handler configuration file.

(parser-error: conflicting action statements)
2026-02-07 09:30:14 +01:00
7a939ff91a refactor ♻️: Refactor stat modules to use ansible.builtin.stat
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.
2026-02-07 09:27:07 +01:00
bb7835c350 refactor ♻️: Refactor subscription task to check and remove nag messages
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 11s
Gitleaks Scan / gitleaks (push) Successful in 5s
Markdown Lint / markdown-lint (push) Failing after 5s
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
2026-02-07 09:24:46 +01:00
3c88ce811e refactor ♻️: Refactor service reload command to use systemd_service
Updated the service reload command to utilize the `systemd_service` module, enhancing compatibility and maintainability.
2026-02-07 09:20:20 +01:00
d72e207a29 chore 📦: Update Proxmox repo configuration with file permissions and ownership
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
2026-02-07 09:19:12 +01:00
78d0f31fbb chore 📦: Remove unused import tasks for backup and monitoring
This commit removes unnecessary import tasks related to backup and monitoring, streamlining the codebase and reducing potential conflicts.
2026-02-07 08:55:27 +01:00
fff929df77 docs 📝: Remove unnecessary blank line from README.md
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 11s
Gitleaks Scan / gitleaks (push) Successful in 5s
Markdown Lint / markdown-lint (push) Failing after 5s
Cleaned up the README by removing an extra blank line to improve readability and maintain consistency.
2026-02-07 08:49:46 +01:00
3d018fd178 refactor ♻️: Refactor task names and module references for clarity and consistency
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 11s
Gitleaks Scan / gitleaks (push) Successful in 4s
Markdown Lint / markdown-lint (push) Failing after 4s
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.
2026-02-07 08:45:57 +01:00
07686643bd docs 📝: Reorganize and add new tasks to README.md
Updated the README.md file by reorganizing existing content and adding new task sections to provide clearer instructions and better organization for users.
2026-02-07 08:45:39 +01:00
8341d6e295 refactor ♻️: Refactor README and update role for Proxmox VE provisioning
Some checks failed
ansible-lint / Ansible Lint (push) Failing after 11s
Gitleaks Scan / gitleaks (push) Successful in 5s
Markdown Lint / markdown-lint (push) Failing after 4s
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.
2026-02-07 08:29:45 +01:00
8c9ad60602 Initial commit 2026-02-07 07:22:24 +01:00