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.

Sign up for GitHub

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

Jump to bottom

(MODULES-8215) Fix spurious trigger with ensure => absent while other properties are set #10

Merged
merged 1 commit into from
Feb 21, 2020
Merged

(MODULES-8215) Fix spurious trigger with ensure => absent while other properties are set #10

merged 1 commit into from
Feb 21, 2020

Conversation

philomory
Copy link
Contributor

reidmv
reidmv approved these changes Feb 21, 2020
View reviewed changes
Copy link
Contributor

@reidmv reidmv left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@binford2k binford2k merged commit 5fb171e into puppetlabs:master Feb 21, 2020
binford2k added a commit to binford2k/puppetlabs-transition that referenced this pull request Feb 25, 2020
@binford2k
Correct an error with puppetlabs#10
506b9fe 
If transition were used with a resource with no `ensure` parameter (like
a notify) then it would stacktrace. This corrects that and adds an
example to validate that the type is working properly.
binford2k added a commit that referenced this pull request Feb 25, 2020
@binford2k
Merge pull request #15 from binford2k/fix_no_ensure
cb328fb 
Correct an error with #10
@DavidS DavidS changed the title Fix spurious trigger with ensure => absent while other properties are set (MODULES-8215) Fix spurious trigger with ensure => absent while other properties are set Mar 8, 2021
cegeka-jenkins pushed a commit to cegeka/puppet-transition that referenced this pull request Sep 27, 2022
@binford2k
Merge pull request puppetlabs#10 from philomory/fix_ensure_absent_wit…
ae5738e 
…h_other_properties_trigger

Fix spurious trigger with ensure => absent while other properties are set
cegeka-jenkins pushed a commit to cegeka/puppet-transition that referenced this pull request Sep 27, 2022
@binford2k
Correct an error with puppetlabs#10
31e8192 
If transition were used with a resource with no `ensure` parameter (like
a notify) then it would stacktrace. This corrects that and adds an
example to validate that the type is working properly.
cegeka-jenkins pushed a commit to cegeka/puppet-transition that referenced this pull request Sep 27, 2022
@binford2k
Merge pull request puppetlabs#15 from binford2k/fix_no_ensure
274afa0 
Correct an error with puppetlabs#10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@reidmv reidmv reidmv approved these changes

Assignees
No one assigned
Labels
bug
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@philomory @reidmv @binford2k @MartyEwings