fix(ansible): update cleanup clients

Author: raisedadead

.github/workflows/ansible--cleanup-clients.yml

@@ -17,6 +17,11 @@ on:
       serial:
         description: 'The value for the variable_serial argument'
         type: number
+      dry_run:
+        description: 'Run in dry-run mode to see what would be deleted'
+        type: boolean
+        required: true
+        default: false
 
 jobs:
   run_playbook:
@@ -55,5 +60,5 @@ jobs:
           ansible-playbook -i inventory/linode.yml \
           play-any--cleanup-clients.yml \
           --private-key=~/.ssh/id_for_ansible.pem \
-          -e '{"variable_host": "${{ matrix.host_groups }}", "variable_serial": '$SERIAL_VALUE'}' \
+          -e '{"variable_host": "${{ matrix.host_groups }}", "variable_serial": '$SERIAL_VALUE', "variable_dry_run": '${{ inputs.dry_run }}'}' \
           ${{ inputs.verbosity }}

ansible/play-any--cleanup-clients.yml

@@ -3,17 +3,34 @@
   hosts: "{{ variable_host | default('null') }}"
   serial: '{{ variable_serial | default(1) }}'
   gather_facts: false
+  vars:
+    dry_run: '{{ variable_dry_run | default(false) }}'
   tasks:
-    - name: Capture directories older than 15 days
+    - name: Get all release directories
       ansible.builtin.find:
         paths: /home/freecodecamp/client/releases
         recurse: false
         file_type: directory
-        age: "{{ variable_age | default('15d') }}"
-      register: files_to_delete
+      register: all_releases
 
-    - name: Remove directories older than 15 days
+    - name: Sort releases by modification time
+      ansible.builtin.set_fact:
+        sorted_releases:
+          "{{ all_releases.files | sort(attribute='mtime', reverse=true) }}"
+
+    - name: Get releases to delete (excluding last 3)
+      ansible.builtin.set_fact:
+        releases_to_delete: '{{ sorted_releases[3:] }}'
+
+    - name: Log releases to be deleted
+      ansible.builtin.debug:
+        msg: 'Would delete release: {{ item.path }}'
+      with_items: '{{ releases_to_delete }}'
+      when: dry_run
+
+    - name: Remove old releases
       ansible.builtin.file:
         path: '{{ item.path }}'
         state: absent
-      with_items: '{{ files_to_delete.files }}'
+      with_items: '{{ releases_to_delete }}'
+      when: not dry_run