Merge pull request #286 from richm/cleanup-sshd2

test: ensure that sshd2 is completely stopped and removed
willshersystems · Apr 9, 2024 · 72c22bc · 72c22bc
2 parents 2bcb523 + fd06cf7
commit 72c22bc
Show file tree

Hide file tree

Showing 2 changed files with 47 additions and 6 deletions.
diff --git a/tests/tests_second_service.yml b/tests/tests_second_service.yml
@@ -5,13 +5,14 @@
     __sshd_test_backup_files:
       - /etc/ssh/sshd_config
       - /etc/ssh/sshd_config.d/00-ansible_system_role.conf
-      - /etc/ssh2/sshd_config
       - /etc/systemd/system/sshd.service
       - /etc/systemd/system/[email protected]
       - /etc/systemd/system/sshd.socket
       - /etc/systemd/system/ssh.service
       - /etc/systemd/system/[email protected]
       - /etc/systemd/system/ssh.socket
+    __sshd_test_remove_files:
+      - /etc/ssh2
       - /etc/systemd/system/sshd2.service
       - /etc/systemd/system/[email protected]
       - /etc/systemd/system/sshd2.socket
@@ -105,5 +106,25 @@
               - "' -f/etc/ssh/sshd_config' not in service_inst.content | b64decode"
               - "' -f/etc/ssh2/sshd_config' in service_inst.content | b64decode"
 
+    - name: Stop second service
+      ansible.builtin.service:
+        name: sshd2
+        state: stopped
+        enabled: false
+      ignore_errors: true  # noqa ignore-errors
+
+    - name: Remove second service
+      ansible.builtin.file:
+        path: "{{ item }}"
+        state: absent
+      loop: "{{ __sshd_test_remove_files }}"
+
+    - name: Refresh systemd
+      # noqa command-instead-of-module
+      ansible.builtin.shell: systemctl reset-failed; systemctl daemon-reload
+      when: ansible_facts["service_mgr"] == "systemd"
+      changed_when: true
+      ignore_errors: true  # noqa ignore-errors
+
     - name: "Restore configuration files"
       ansible.builtin.include_tasks: tasks/restore.yml
diff --git a/tests/tests_second_service_drop_in.yml b/tests/tests_second_service_drop_in.yml
@@ -5,8 +5,8 @@
     __sshd_test_backup_files:
       - /etc/ssh/sshd_config
       - /etc/ssh/sshd_config.d/00-ansible_system_role.conf
-      - /etc/ssh2/sshd_config
-      - /etc/sshd/sshd_config.d/04-ansible.conf
+    __sshd_test_remove_files:
+      - /etc/ssh2
       - /etc/systemd/system/sshd2.service
       - /etc/systemd/system/[email protected]
       - /etc/systemd/system/sshd2.socket
@@ -117,6 +117,26 @@
                 that:
                   - "' -f/etc/ssh/sshd_config' not in service_inst.content | b64decode"
                   - "' -f/etc/ssh2/sshd_config' in service_inst.content | b64decode"
-
-    - name: "Restore configuration files"
-      ansible.builtin.include_tasks: tasks/restore.yml
+      always:
+        - name: Stop second service
+          ansible.builtin.service:
+            name: sshd2
+            state: stopped
+            enabled: false
+          ignore_errors: true  # noqa ignore-errors
+
+        - name: Remove second service
+          ansible.builtin.file:
+            path: "{{ item }}"
+            state: absent
+          loop: "{{ __sshd_test_remove_files }}"
+
+        - name: Refresh systemd
+          # noqa command-instead-of-module
+          ansible.builtin.shell: systemctl reset-failed; systemctl daemon-reload
+          when: ansible_facts["service_mgr"] == "systemd"
+          changed_when: true
+          ignore_errors: true  # noqa ignore-errors
+
+        - name: "Restore configuration files"
+          ansible.builtin.include_tasks: tasks/restore.yml