mirror of
https://github.com/ansible-collections/community.general.git
synced 2026-06-11 02:25:36 +00:00
213581bef8
* fix: treat chdev execution failures as module errors in aix_devices ##### SUMMARY Fix the aix_devices module so that a failed chdev command is reported as a module failure instead of a successful result. The previous implementation called fail_json incorrectly by returning success through the normal completion path when chdev returned a non-zero exit status. This could allow a playbook to continue even though the requested device attribute change was not applied. This change replaces the success-style error handling with proper failure handling in the chdev execution path, making task results consistent with the actual command outcome. ##### ISSUE TYPE - Bugfix Pull Request ##### COMPONENT NAME aix_devices ##### ADDITIONAL INFORMATION The affected code path is in change_device_attr() when the module executes chdev to apply attribute updates. Before this change: - chdev could fail - the module could still report success - later tasks could run against an unexpected system state After this change: - chdev failures are returned through fail_json - the task stops with an error - playbook behavior matches the real execution result ```paste below Before: module.exit_json(msg="Failed to run chdev.", rc=rc, err=err) After: module.fail_json(msg="Failed to run chdev.", rc=rc, err=err) ``` * Add changelog fragment for aix_devices chdev failure fix (#12185) --------- Co-authored-by: Hirofumi Arimoto <harimoto@jp.ibm.com> |
||
|---|---|---|
| .. | ||
| action | ||
| become | ||
| cache | ||
| callback | ||
| connection | ||
| doc_fragments | ||
| filter | ||
| inventory | ||
| lookup | ||
| module_utils | ||
| modules | ||
| plugin_utils | ||
| test | ||