diff options
author | Aija Jauntēva <aija.jaunteva@dell.com> | 2020-10-02 07:43:33 -0400 |
---|---|---|
committer | Julia Kreger <juliaashleykreger@gmail.com> | 2020-11-11 23:48:58 +0000 |
commit | e21361afb0db48025fb74b5ed7749a4a343a47a0 (patch) | |
tree | a15bfe43629c025d62071f7fc54ed76c735f1b05 /releasenotes | |
parent | e2b55a8cb835e0b63830de34a002b7652753cb3d (diff) | |
download | ironic-e21361afb0db48025fb74b5ed7749a4a343a47a0.tar.gz |
Fix redfish BIOS apply config error handling
Instead of using process_event('fail') use error_handlers,
otherwise in case of failure node gets stuck and fails
because of timeout, instead of failing earlier due to
step failure.
Besides adding new unit tests, also update related unit tests
to test for success correctly and have realistic data.
Story: 2008307
Task: 41196
Change-Id: If28ccb252a87610e3fd3dc78e1ed75bb8ca1cdcf
(cherry picked from commit 70b7ca345f39b5126b5e68aa507298b623932dd5)
Diffstat (limited to 'releasenotes')
-rw-r--r-- | releasenotes/notes/fix-redfish-bios-apply-configuration-error-handling-464695b09e4f81ac.yaml | 8 |
1 files changed, 8 insertions, 0 deletions
diff --git a/releasenotes/notes/fix-redfish-bios-apply-configuration-error-handling-464695b09e4f81ac.yaml b/releasenotes/notes/fix-redfish-bios-apply-configuration-error-handling-464695b09e4f81ac.yaml new file mode 100644 index 000000000..ecae3d933 --- /dev/null +++ b/releasenotes/notes/fix-redfish-bios-apply-configuration-error-handling-464695b09e4f81ac.yaml @@ -0,0 +1,8 @@ +--- +fixes: + - | + Fixes ``redfish`` BIOS ``apply_configuration`` clean and deploy step to + fail correctly in case of error when checking if BIOS updates are + successfully applied. Before the fix when BIOS updates were unsuccessful, + then node cleaning or deploying failed with timeout instead of actual error + in clean or deploy step. |