Skip to content

overlay: fail the build when a post-overlay hook fails#338

Open
mRrvz wants to merge 1 commit into
omniosorg:masterfrom
mRrvz:feature/fail_if_postoverlay_hook_fail
Open

overlay: fail the build when a post-overlay hook fails#338
mRrvz wants to merge 1 commit into
omniosorg:masterfrom
mRrvz:feature/fail_if_postoverlay_hook_fail

Conversation

@mRrvz

@mRrvz mRrvz commented Jul 14, 2026

Copy link
Copy Markdown

apply_custom_overlay() runs every script in the overlay's .overlay-hooks directory, but a hook that exited non-zero was only reported as a "WARNING" and otherwise ignored: the loop kept going and the function still returned success. As a result a failing hook (for example, a package that could not be installed into the image) was invisible to the callers, and the image was built and shipped as if the overlay had applied cleanly. Make hook failures fatal.

apply_custom_overlay() runs every script in the overlay's .overlay-hooks
directory, but a hook that exited non-zero was only reported as a
"WARNING" and otherwise ignored: the loop kept going and the function
still returned success. As a result a failing hook (for example, a
package that could not be installed into the image) was invisible to the
callers, and the image was built and shipped as if the overlay had
applied cleanly. Make hook failures fatal.

Signed-off-by: Alexey Romanov <romanov.alexey2000@gmail.com>
@mRrvz
mRrvz force-pushed the feature/fail_if_postoverlay_hook_fail branch from fc4f96b to f9cbffe Compare July 16, 2026 08:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant