Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

(#9292) New command line flag for puppet resource, fail #9297

Conversation

cthorn42
Copy link
Collaborator

See discussion in issue #9292.
What this PR adds is the ability for on the command line if --fail is passed then the puppet resource command will exit with a non zero return code.

@cthorn42 cthorn42 requested a review from a team as a code owner March 15, 2024 04:55
@bastelfreak
Copy link
Contributor

Hi!
Should this default to true if strict is set to error?

@joshcooper
Copy link
Contributor

Should this default to true if strict is set to error?

Since strict defaults to true in puppet8 I think that would be a breaking change.

See discussion in issue puppetlabs#9292.
What this PR adds is the ability for on the command line if --fail is passed then
the `puppet resource` command will exit with a non zero return code.
@cthorn42 cthorn42 force-pushed the maint/main/puppet_resource_service_returns_correctly branch from 0b3d25a to cf6cb77 Compare March 15, 2024 18:24
@joshcooper joshcooper merged commit 00055eb into puppetlabs:main Mar 19, 2024
9 checks passed
@joshcooper joshcooper added the backport 7.x Generate a backport PR to 7.x label Mar 20, 2024
Copy link

Backport failed for 7.x, because it was unable to cherry-pick the commit(s).

Please cherry-pick the changes locally.

git fetch origin 7.x
git worktree add -d .worktree/backport-9297-to-7.x origin/7.x
cd .worktree/backport-9297-to-7.x
git checkout -b backport-9297-to-7.x
ancref=$(git merge-base 1a53bf7d66275ad4b4928484bb1314dc32e84677 cf6cb773c327098ee850fbffb17ce3564895dfa4)
git cherry-pick -x $ancref..cf6cb773c327098ee850fbffb17ce3564895dfa4

@joshcooper
Copy link
Contributor

Backport PR #9300

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 7.x Generate a backport PR to 7.x
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants