fix: Idempotence failure with maintenance mode

geoffreyvanwyk · Nov 22, 2023 · b38b621 · b38b621
1 parent b55e913
commit b38b621
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 12 deletions.
diff --git a/tasks/install_moodle.yml b/tasks/install_moodle.yml
@@ -49,10 +49,13 @@
       else '7.4' }}"
 
 - name: Enable Moodle's maintenance mode {{ sandbox_item.version }}
-  ansible.builtin.command:
+  ansible.builtin.shell:
     chdir: /var/www/html/moodle-{{ sandbox_item.version }}
-    cmd: php{{ sandbox_php_version }} admin/cli/maintenance.php --enable
-  changed_when: yes
+    cmd: >
+      php{{ sandbox_php_version }} admin/cli/maintenance.php --enable
+      &&
+      touch maintenance.enabled
+    creates: /var/www/html/moodle-{{ sandbox_item.version }}/maintenance.enabled
 
 - name: Does archive of moodledata exists for Moodle {{ sandbox_item.version }}?
   ansible.builtin.stat:
@@ -122,7 +125,10 @@
         template: moodle-{{ sandbox_item.version | replace(".", "") }}
 
 - name: Disable Moodle's maintenance mode {{ sandbox_item.version }}
-  ansible.builtin.command:
+  ansible.builtin.shell:
     chdir: /var/www/html/moodle-{{ sandbox_item.version }}
-    cmd: php{{ sandbox_php_version }} admin/cli/maintenance.php --disable
-  changed_when: yes
+    cmd: >
+      php{{ sandbox_php_version }} admin/cli/maintenance.php --disable
+      &&
+      rm maintenance.enabled
+    removes: /var/www/html/moodle-{{ sandbox_item.version }}/maintenance.enabled
diff --git a/tasks/reset_moodle.yml b/tasks/reset_moodle.yml
@@ -35,10 +35,13 @@
       else '7.4' }}"
 
 - name: Enable Moodle's maintenance mode {{ sandbox_item.version }}
-  ansible.builtin.command:
+  ansible.builtin.shell:
     chdir: /var/www/html/moodle-{{ sandbox_item.version }}
-    cmd: php{{ sandbox_php_version }} admin/cli/maintenance.php --enable
-  changed_when: yes
+    cmd: >
+      php{{ sandbox_php_version }} admin/cli/maintenance.php --enable
+      &&
+      touch maintenance.enabled
+    creates: /var/www/html/moodle-{{ sandbox_item.version }}/maintenance.enabled
 
 - name: Rename current moodledata to old {{ sandbox_item.version }}
   become: yes
@@ -110,7 +113,10 @@
     template: moodle-{{ sandbox_item.version | replace(".", "") }}-fresh
 
 - name: Disable Moodle's maintenance mode {{ sandbox_item.version }}
-  ansible.builtin.command:
+  ansible.builtin.shell:
     chdir: /var/www/html/moodle-{{ sandbox_item.version }}
-    cmd: php{{ sandbox_php_version }} admin/cli/maintenance.php --disable
-  changed_when: yes
+    cmd: >
+      php{{ sandbox_php_version }} admin/cli/maintenance.php --disable
+      &&
+      rm maintenance.enabled
+    removes: /var/www/html/moodle-{{ sandbox_item.version }}/maintenance.enabled