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

refactor ♻️: Refactor task naming to include role context #36

Merged
Jose merged 2 commits from dev into main 2026-02-15 18:23:54 +01:00
Conversation 1 Commits 2 Files Changed 1 +5 -2
1 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
tasks/ram.yml
View File

@@ -4,10 +4,13 @@
that: ansible_os_family == "Debian"
fail_msg: "This role only supports Debian/Proxmox systems."
- name: ram | Gather only memory fact
ansible.builtin.setup:
filter: ansible_memtotal_mb
- name: ram | Calculate log2ram size
ansible.builtin.set_fact:
_total_ram_mb: "{{ ansible_memtotal_mb }}"
_calculated_size_mb: "{{ (_total_ram_mb * log2ram_ram_percent / 100) | int }}"
_calculated_size_mb: "{{ (ansible_memtotal_mb * log2ram_ram_percent / 100) | int }}"
log2ram_size_mb: >-
{{
[log2ram_min_size_mb,
gitea-actions commented 2026-02-15 18:23:04 +01:00
Review
Copy Link

[Lines 7-16] [Score: 2] Removed intermediate variables for a more efficient calculation of _calculated_size_mb.

[Lines 7-16] [Score: 2] Removed intermediate variables for a more efficient calculation of _calculated_size_mb.