Commit Graph

17 Commits

Author SHA1 Message Date
Jose 4a02a41d43 refactor ♻️: Refactor logrotate configuration for dynamic handling of multiple files
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
Jose e033fccbdf refactor ♻️: Refactor task name and comment
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
Jose 410f87bb2c chore 📦: Comment out enterprise repo lines and enable Proxmox no-subscription repo
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
Jose 1f2ff1d9ba refactor ♻️: Refactor task names and add namespace for clarity
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
Jose 12b1b87af1 refactor ♻️: Refactor legacy and minified proxmoxlib.js patching logic to use handlers
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
Jose afc80dc57d refactor ♻️: Rename 'task' to 'Read checksum after patch'
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
Jose f16d6a62b7 fix 🐛: updates the checksum storage mechanism to ensure idempotency.
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
Jose e0831c473e refactor ♻️: Refactor task to use ansible.builtin.slurp for compatibility
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
Jose 4d73c5a844 style 💎: Fix indentation and logic in subscription.yml tasks
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
Jose 7907e71a5e feat : Add task to ensure nag patch state directory exists and manage checksums for legacy and minified proxmoxlib.js files
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
Jose 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
Jose bb7835c350 refactor ♻️: Refactor subscription task to check and remove nag messages
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
Jose 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
Jose 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
Jose 3d018fd178 refactor ♻️: Refactor task names and module references for clarity and consistency
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
Jose 8341d6e295 refactor ♻️: Refactor README and update role for Proxmox VE provisioning
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
Jose 8c9ad60602 Initial commit 2026-02-07 07:22:24 +01:00