Skip to content

also pass value of fail_on_error and hidden options of run_shell_cmd call down to pre/post run_shell_cmd hook#4911

Merged
ocaisa merged 1 commit intoeasybuilders:developfrom
boegel:fail_on_error_hidden_post_run_shell_cmd_hook
May 31, 2025
Merged

also pass value of fail_on_error and hidden options of run_shell_cmd call down to pre/post run_shell_cmd hook#4911
ocaisa merged 1 commit intoeasybuilders:developfrom
boegel:fail_on_error_hidden_post_run_shell_cmd_hook

Conversation

@boegel
Copy link
Copy Markdown
Member

@boegel boegel commented May 31, 2025

EasyBuild runs several "hidden" commands, and also sometimes runs commands that are allowed to fail.

Discriminating between these commands and "normal" commands like the ones that are part of an installation procedure could be interesting in run_shell_cmd hooks.

@boegel boegel added this to the next release (5.1.1?) milestone May 31, 2025
@boegel boegel force-pushed the fail_on_error_hidden_post_run_shell_cmd_hook branch from a9081f3 to 0ba881e Compare May 31, 2025 16:35
Copy link
Copy Markdown
Member

@ocaisa ocaisa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ocaisa ocaisa enabled auto-merge May 31, 2025 16:40
@ocaisa ocaisa merged commit b2b9c54 into easybuilders:develop May 31, 2025
37 checks passed
@boegel boegel deleted the fail_on_error_hidden_post_run_shell_cmd_hook branch May 31, 2025 19:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants