Skip to content

(maint) Fail if puppetserver can't be started#816

Merged
mhashizume merged 1 commit into
mainfrom
maint/main/puppetserver-fail
Dec 22, 2025
Merged

(maint) Fail if puppetserver can't be started#816
mhashizume merged 1 commit into
mainfrom
maint/main/puppetserver-fail

Conversation

@mhashizume
Copy link
Copy Markdown
Contributor

By default, Beaker tests fail if they give a non-zero exit code. However, when starting puppetserver in the master setup presuite, the resource can fail and still return an exit code of 0.

This commit updates the start puppetserver step to use the --fail option, which will return an exit code of 1 if the resource could not be modified.

@mhashizume mhashizume requested review from a team and bastelfreak as code owners December 22, 2025 18:45
@mhashizume mhashizume added the maintenance Maintenance chores are typically excluded from changelogs label Dec 22, 2025
By default, Beaker tests fail if they give a non-zero exit code.
However, when starting puppetserver in the master setup presuite, the
resource can fail and still return an exit code of 0.

This commit updates the start puppetserver step to use the --fail
option, which will return an exit code of 1 if the resource could not be
modified.
@mhashizume mhashizume force-pushed the maint/main/puppetserver-fail branch from 4f426b7 to f7e6ce1 Compare December 22, 2025 21:54
@mhashizume mhashizume merged commit 34fa12b into main Dec 22, 2025
9 checks passed
@mhashizume mhashizume deleted the maint/main/puppetserver-fail branch December 22, 2025 22:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Maintenance chores are typically excluded from changelogs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants