Jose/ansible_role_proxmox_provision
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

chore 📦: Update log2ram version to 1.7.2 and add dynamic versioning in ram.yml #40

Merged
Jose merged 1 commits from dev into main 2026-02-16 18:50:08 +01:00
Conversation 1 Commits 1 Files Changed 2 +3 -2
Jose commented 2026-02-16 18:48:57 +01:00
Owner
Copy Link

This commit updates the log2ram version to 1.7.2 and introduces dynamic versioning in the ram.yml file. This ensures that the log2ram version is always up-to-date with the latest release.

This commit updates the log2ram version to 1.7.2 and introduces dynamic versioning in the `ram.yml` file. This ensures that the log2ram version is always up-to-date with the latest release.
Jose added 1 commit 2026-02-16 18:48:57 +01:00
chore 📦: Update log2ram version to 1.7.2 and add dynamic versioning in ram.yml
All checks were successful
ansible-lint / Ansible Lint (push) Successful in 12s
Details
Gitleaks Scan / gitleaks (push) Successful in 5s
Details
Markdown Lint / markdown-lint (push) Successful in 5s
Details
ai-reviews / Review PR (pull_request) Successful in 13s
Details
PR check / Gitleaks (pull_request) Successful in 4s
Details
PR check / lint tests (pull_request) Successful in 14s
Details
PR check / labeler (pull_request) Successful in 2s
Details
PR check / handle_failures (pull_request) Has been skipped
Details
PR check / handle_success (pull_request) Successful in 2s
Details
6e44de23e9 
This commit updates the log2ram version to 1.7.2 and introduces dynamic versioning in the `ram.yml` file. This ensures that the log2ram version is always up-to-date with the latest release.
gitea-actions bot reviewed 2026-02-16 18:49:28 +01:00
gitea-actions bot left a comment
Copy Link

Review Summary

  • defaults/main.yml: Addition of log2ram version Other than the above observation, the changes look good and there are no critical or major issues found.
  • tasks/ram.yml: Summary: The provided git diff updates the version number of a package being downloaded and installed, improving its adaptability to new versions.
# Review Summary * **defaults/main.yml**: Addition of log2ram version Other than the above observation, the changes look good and there are no critical or major issues found. * **tasks/ram.yml**: **Summary:** The provided git diff updates the version number of a package being downloaded and installed, improving its adaptability to new versions.
defaults/main.yml
@@ -38,6 +38,7 @@ log2ram_ram_percent: 10
log2ram_min_size_mb: 128
log2ram_max_size_mb: 1024
log2ram_host_memtotal_mb: "{{ ansible_memtotal_mb }}"
log2ram_version: "1.7.2"
gitea-actions bot commented 2026-02-16 18:49:28 +01:00
Copy Link

[Score: 2] The version number is hardcoded. It's recommended to use variables or a configuration file for version control and easier updates.

[Score: 2] The version number is hardcoded. It's recommended to use variables or a configuration file for version control and easier updates.
tasks/ram.yml
@@ -40,3 +40,3 @@
- name: ram | Download log2ram
ansible.builtin.get_url:
url: https://github.com/azlux/log2ram/archive/refs/tags/1.7.2.tar.gz
url: https://github.com/azlux/log2ram/archive/refs/tags/{{ log2ram_version }}.tar.gz
gitea-actions bot commented 2026-02-16 18:49:28 +01:00
Copy Link

[Score: 2] The URL for downloading the package is dynamically generated based on the log2ram_version variable, ensuring the installation uses the correct version of the package.

[Score: 2] The URL for downloading the package is dynamically generated based on the `log2ram_version` variable, ensuring the installation uses the correct version of the package.
gitea-actions bot added the sourceci-pass labels 2026-02-16 18:49:54 +01:00
Jose merged commit 6458a4e0cf into main 2026-02-16 18:50:08 +01:00
Sign in to join this conversation.
Reviewers
No Reviewers
Labels
Clear labels
Documentation
bug
Something is not working
 ci-failed
action test workflow failed
 ci-pass
action test workflow pass
 duplicate
This issue or pull request already exists
 enhancement
New feature
 help wanted
Need some help
 invalid
Something is wrong
 question
More information is needed
 source
wontfix
This won't be fixed
No Label ci-pass source
Milestone
No items
No Milestone
Projects
Clear projects
No project
Assignees
Clear assignees
Jose
No Assignees
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: Jose/ansible_role_proxmox_provision#40
Delete Branch "dev"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?