Skip to content

Commit 0b22460

Browse files
committed
Fix deprovisioning multiple overcloud hosts
Previously, the user prompt ran on one host, but the assertion ran on all hosts. This meant the check could only pass on one host, all others would fail every time. This change ensures the check and assertion always run once on localhost. Closes-Bug: #2115037 Change-Id: If2a26b42e6fe0152922e10df924c0797259e2e53 (cherry picked from commit 9a7aaff)
1 parent 08d10d1 commit 0b22460

File tree

2 files changed

+11
-0
lines changed

2 files changed

+11
-0
lines changed

ansible/overcloud-deprovision.yml

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -43,6 +43,8 @@
4343
{{ play_hosts | join(', ') }}
4444
If you want to proceed type: yes
4545
register: pause_prompt
46+
delegate_to: localhost
47+
run_once: true
4648
when: not confirm_deprovision
4749

4850
- name: Fail if deprovision is not confirmed
@@ -51,6 +53,8 @@
5153
msg: >
5254
Deprovision has not been confirmed. You must either type 'yes' when
5355
prompted, or set ``confirm_deprovision=yes``.
56+
delegate_to: localhost
57+
run_once: true
5458

5559
- name: Get PXE MAC address
5660
command: >
Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,7 @@
1+
---
2+
fixes:
3+
- |
4+
Fixes the user prompt for deprovisioning multiple overcloud hosts at once.
5+
Previously, the prompt would only confirm deprovisioning for one host. It
6+
will now correctly apply to all hosts that have been targeted.
7+
`LP#2115037 <https://bugs.launchpad.net/kayobe/+bug/2115037>`__

0 commit comments

Comments
 (0)