internet/ansible-collections-openstack
3
0
Fork 0
mirror of https://opendev.org/openstack/ansible-collections-openstack.git synced 2026-05-08 14:23:03 +00:00
Code Issues Packages Projects Releases Wiki Activity

Merge "Don't compare current state for reboot_* actions"

Browse Source
This commit is contained in:
Zuul
2025-05-13 18:15:17 +00:00
committed by Gerrit Code Review
parent 762fee2bad 5a6f5084dd
commit 3d7948a4e4
2 changed files with 5 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
ci/roles/server_action/tasks/main.yml
View File

@@ -553,7 +553,7 @@
assert: assert:
that: that:
- servers.servers.0.status == 'ACTIVE' - servers.servers.0.status == 'ACTIVE'
- server is not changed - server is changed
- name: Reboot server (HARD) - name: Reboot server (HARD)
openstack.cloud.server_action: openstack.cloud.server_action:
@@ -573,7 +573,7 @@
assert: assert:
that: that:
- servers.servers.0.status == 'ACTIVE' - servers.servers.0.status == 'ACTIVE'
- server is not changed - server is changed
- name: Delete server - name: Delete server
openstack.cloud.server: openstack.cloud.server:

3
plugins/modules/server_action.py
View File

@@ -136,6 +136,9 @@ class ServerActionModule(OpenStackModule):
# rebuild does not depend on state # rebuild does not depend on state
will_change = ( will_change = (
(action == 'rebuild') (action == 'rebuild')
# `reboot_*` actions do not change state, servers remain `ACTIVE`
or (action == 'reboot_hard')
or (action == 'reboot_soft')
or (action == 'lock' and not server['is_locked']) or (action == 'lock' and not server['is_locked'])
or (action == 'unlock' and server['is_locked']) or (action == 'unlock' and server['is_locked'])
or server.status.lower() not in [a.lower() or server.status.lower() not in [a.lower()