-
Bug
-
Resolution: Done
-
Undefined
-
None
-
rhos-18.0.z
-
None
-
0
-
False
-
-
False
-
?
-
rhos-ops-day1day2-edpm
-
None
-
-
-
-
Moderate
To Reproduce Steps to reproduce the behavior:
- Provide `edpm_bootstrap_command` with multiple commands, where one of the commands fails but the last one succeeds.
- The bootstrap as a whole is considered successful and the `edpm.bootstrap.hash` fact is written to prevent re-executing the same command.
Expected behavior
- If one command in the `edpm_bootstrap_command` fails, i'd expect the bootstrap as a whole to fail. If there are commands that can fail expectedly, they should be suffixed with `|| true`.
- I believe we should unconditionally prepend `set -e` to edpm_bootstrap_command when we execute it, to ensure any failed command somewhere in the middle of the bootstrap snippet fails the whole bootstrap.
Bug impact
- This came to bite us at a customer. Will link the relevant conversation in a private comment.
Known workaround
- Include `set -e` at the beginning when setting the value for `edpm_bootstrap_command` variable.
- is cloned by
-
OSPRH-25580 Adoption: edpm_boostrap_command ignores failures except for the last command
-
- Closed
-