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

Add a receipt so auth_cmd exec only runs when parameters change. #726

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

johnzimm
Copy link

Possible solution/workaround for #678

@rendhalver
Copy link

Can we get this merged please?

@johnzimm
Copy link
Author

It looks like the "if receipt" logic has been implemented over in the puppetlabs "supported" repo: https://github.com/puppetlabs/puppetlabs-docker/blob/master/manifests/registry.pp#L69

I would assume that the puppetlabs repo now supersedes this one?

@rendhalver
Copy link

Weird. The puppetlabs version is one behind this one.

@rendhalver
Copy link

Oops silly me. The puppetlabs repo is 1.1.0 and this one is 5.3.0
This one has not been updated since 2016 so it seems likely the puppetlabs repo has taken over.

@johnzimm
Copy link
Author

Yeah. I've made note to migrate my puppet-control. It seems the next issue opened here would be to just ask that a notice be posted in the README and Puppet Forge directing people to the puppetlabs module as it looks like the continuation of this code base and puppetlabs is tagged it as "supported" in the forge.

@rendhalver
Copy link

Sounds like a good plan.

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.

2 participants