fix(docker): update uninstall option (#55)

DeadNews · Sep 12, 2024 · 2b55775 · 2b55775
1 parent a7b2b34
commit 2b55775
Show file tree

Hide file tree

Showing 3 changed files with 10 additions and 7 deletions.
diff --git a/roles/docker/meta/argument_specs.yml b/roles/docker/meta/argument_specs.yml
@@ -83,7 +83,10 @@ argument_specs:
         required: false
         description: The home directory for the rootless user.
 
-      docker_uninstall:
-        type: bool
-        required: false
-        description: Uninstall Docker packages.
+      docker_state:
+        type: str
+        default: present
+        description: The state of the Docker installation.
+        choices:
+          - present
+          - absent
diff --git a/roles/docker/tasks/main.yml b/roles/docker/tasks/main.yml
@@ -8,13 +8,13 @@
       - ansible_os_family == 'Debian'
 
 - name: Uninstall Docker packages
-  when: docker_uninstall
+  when: docker_state == 'absent'
   ansible.builtin.import_role:
     name: deadnews.util.docker
     tasks_from: uninstall
 
 - name: Install Docker
-  when: not docker_uninstall
+  when: docker_state == 'present'
   block:
     - name: Install Docker
       ansible.builtin.import_role:

diff --git a/roles/docker/vars/main.yml b/roles/docker/vars/main.yml
@@ -9,4 +9,4 @@ docker_rootless_uid: 800
 docker_rootless_user_home: /var/lib/{{ docker_rootless_user }}
 docker_rootless_user: rootless
 docker_rootless: false
-docker_uninstall: false
+docker_state: present