Skip to content

Commit

Permalink
fix: add bootstrap to delete_nodes role (#99)
Browse files Browse the repository at this point in the history
Add deletion of the bootstrap node to the delete_nodes role, in case of
restarting 6_create_nodes before bootstrap is already deleted.

Add changed_when parameter so that "ok" is given when nodes are not
destroyed or undefined, and "changed" when they are.

Signed-off-by: Jacob Emery <jacob.emery@ibm.com>
  • Loading branch information
jacobemery committed Feb 17, 2023
1 parent ac87ae2 commit d7ace53
Showing 1 changed file with 11 additions and 0 deletions.
11 changes: 11 additions & 0 deletions roles/delete_nodes/tasks/main.yaml
Original file line number Diff line number Diff line change
@@ -1,5 +1,14 @@
---

- name: Delete bootstrap node, if exists
tags: delete_nodes
ansible.builtin.shell: |
set -o pipefail
virsh destroy "{{ env.cluster.nodes.bootstrap.vm_name }}" || true
virsh undefine "{{ env.cluster.nodes.bootstrap.vm_name }}" --remove-all-storage || true
register: delete_bootstrap
changed_when: "('destroyed' in delete_bootstrap.stdout) or ('undefined' in delete_bootstrap.stdout)"

- name: Delete control, compute and infra nodes, if exists
tags: delete_nodes
ansible.builtin.shell: |
Expand All @@ -9,6 +18,8 @@
loop: "{{ env.cluster.nodes.control.vm_name + env.cluster.nodes.compute.vm_name \
if env.cluster.nodes.infra.vm_name is not defined \
else env.cluster.nodes.control.vm_name + env.cluster.nodes.compute.vm_name + env.cluster.nodes.infra.vm_name }}"
register: delete_nodes
changed_when: "('destroyed' in delete_nodes.stdout) or ('undefined' in delete_nodes.stdout)"

- name: Get and print virsh list
tags: delete_nodes
Expand Down

0 comments on commit d7ace53

Please sign in to comment.