Fix reporting error status for multi-command actions#20
Conversation
When multiple commands are called, properly report result of the last one as the final result (when status_mode=last - the default), instead of choosing last command that itself reported the 'status' field. Furthermode, for the status_mode=all, really fail if there is any failure. Previously if there was a success that had 'result' field, it covered earlier failure (including one calculated for status_mode=last). Fixes QubesOS/qubes-issues#10752
OpenQA test summaryComplete test suite and dependencies: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2026031800-4.3&flavor=pull-requests Test run included the following:
New failures, excluding unstableCompared to: https://openqa.qubes-os.org/tests/overview?distri=qubesos&version=4.3&build=2026020304-devel&flavor=update
Failed tests24 failures
Fixed failuresCompared to: https://openqa.qubes-os.org/tests/166096#dependencies 25 fixed
Unstable testsDetails
Performance TestsPerformance degradation:4 performance degradations
Remaining performance tests:35 tests
|
When multiple commands are called, properly report result of the last
one as the final result (when status_mode=last - the default), instead
of choosing last command that itself reported the 'status' field.
Furthermode, for the status_mode=all, really fail if there is any
failure. Previously if there was a success that had 'result' field, it
covered earlier failure (including one calculated for status_mode=last).
Fixes QubesOS/qubes-issues#10752