refactor ♻️: Refactor task names and module references for clarity and consistency

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.

handlers/main.yml

@@ -2,14 +2,14 @@
 # ansible_role_proxmox_provision/handlers/main.yml
 # Handlers for role
 # ==================
-- name: apt update
+- name: Run apt update
-  apt:
+  ansible.builtin.apt:
     update_cache: yes
 
-- name: restart pveproxy
+- name: Restart pveproxy
-  systemd:
+  ansible.builtin.systemd:
     name: pveproxy
     state: restarted
 
-- name: reload systemd
+- name: Reload systemd
-  command: systemctl daemon-reexec
+  ansible.builtin.command: systemctl daemon-reexec

tasks/logrotate.yml

@@ -1,6 +1,6 @@
 ---
-- name: Configure Proxmox logrotate limits
+- name: logrotate | Configure Proxmox logrotate limits
-  template:
+  ansible.builtin.template:
     src: logrotate-pve.j2
     dest: /etc/logrotate.d/proxmox
     owner: root

tasks/powertop.yml

@@ -1,21 +1,21 @@
 ---
-- name: Install powertop
+- name: powertop | Install powertop
-  apt:
+  ansible.builtin.apt:
     name: powertop
     state: present
     update_cache: yes
   when: proxmox_enable_powertop
 
-- name: Create powertop systemd service
+- name: powertop | Create powertop systemd service
-  template:
+  ansible.builtin.template:
     src: powertop.service.j2
     dest: /etc/systemd/system/powertop.service
     mode: "0644"
   when: proxmox_enable_powertop
   notify: reload systemd
 
-- name: Enable and start powertop service
+- name: powertop | Enable and start powertop service
-  systemd:
+  ansible.builtin.systemd:
     name: powertop
     enabled: true
     state: started

tasks/repos.yml

@@ -1,14 +1,14 @@
 ---
-- name: Remove enterprise repo files (all known locations)
+- name: repos | Remove enterprise repo files (all known locations)
-  file:
+  ansible.builtin.file:
     path: "{{ item }}"
     state: absent
   loop:
     - /etc/apt/sources.list.d/pve-enterprise.list
     - /etc/apt/sources.list.d/ceph.list
 
-- name: Enable Proxmox no-subscription repo
+- name: repos | Enable Proxmox no-subscription repo
-  copy:
+  ansible.builtin.copy:
     dest: /etc/apt/sources.list.d/pve-no-subscription.list
     content: |
       deb http://download.proxmox.com/debian/pve {{ ansible_distribution_release }} pve-no-subscription

tasks/subscription.yml

@@ -1,14 +1,14 @@
 ---
-- name: Remove subscription nag (legacy proxmoxlib.js)
+- name: subscription | Remove subscription nag (legacy proxmoxlib.js)
-  replace:
+  ansible.builtin.replace:
     path: /usr/share/javascript/proxmox-widget-toolkit/proxmoxlib.js
     regexp: "if \\(data.status !== 'Active'\\)"
     replace: "if (false)"
   ignore_errors: true
   notify: restart pveproxy
 
-- name: Remove subscription nag (minified bundle for VE 8/9)
+- name: subscription | Remove subscription nag (minified bundle for VE 8/9)
-  replace:
+  ansible.builtin.replace:
     path: /usr/share/javascript/proxmox-widget-toolkit/proxmoxlib.min.js
     regexp: "data.status!=='Active'"
     replace: "false"

tasks/swap.yml

@@ -1,20 +1,20 @@
 ---
-- name: Set vm.swappiness
+- name: swap | Set vm.swappiness
-  sysctl:
+  ansible.posix.sysctl:
     name: vm.swappiness
     value: "{{ proxmox_swapiness }}"
     state: present
     reload: yes
 
-- name: Disable swap if host has enough RAM
+- name: swap | Disable swap if host has enough RAM
-  command: swapoff -a
+  ansible.builtin.command: swapoff -a
   when:
     - proxmox_disable_swap
     - ansible_memtotal_mb >= proxmox_min_ram_mb_for_no_swap
   changed_when: false
 
-- name: Remove swap from fstab
+- name: swap | Remove swap from fstab
-  replace:
+  ansible.builtin.replace:
     path: /etc/fstab
     regexp: '^\S+\s+\S+\s+swap\s+.*$'
     replace: ''