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

Support running puppet generate types with r10k mcollective plugin? #293

Open
alexjfisher opened this issue Nov 7, 2016 · 7 comments
Open

Comments

@alexjfisher
Copy link
Member

I currently use this module's mcollective plugin to run R10K on multiple puppet servers. Is this plugin the right place to add support for running puppet generate types at the same time? https://docs.puppet.com/puppet/4.7/reference/environment_isolation.html#suggested-workflow

@ghoneycutt
Copy link
Member

+1 though I think it should be optional and default to false since it is both new and still experimental.

@rnelson0
Copy link
Sponsor Member

rnelson0 commented Nov 7, 2016

Yes. Currently you could run it as the optional 'after' command. We can look at better ways to integrate it based on version, though. I just peeked at the documents, would this be for all 4.x or just starting with a specific version?

@alexjfisher
Copy link
Member Author

alexjfisher commented Nov 8, 2016

@rnelson0 The 'after' command? Or perhaps you were referring to this option? https://github.com/puppetlabs/r10k/blob/master/doc/dynamic-environments/configuration.mkd#postrun

The generate types feature is only available since puppet 4.6

@rnelson0
Copy link
Sponsor Member

rnelson0 commented Nov 8, 2016

Yes, postrun (r10k::postrun in hiera) is the parameter whose name escaped me.

I assume we would look for puppet_version and use versioncmp on it, if it's above 4.6.0 generate types could be called. Sound correct?

@alexjfisher
Copy link
Member Author

The issue I see with postrun is that it's not per environment. It's only run if you're refreshing all environments. Maybe it's actually a patch to R10k that's needed before the R10K mcollective agent can be modified to support the additional functionality.

@rnelson0
Copy link
Sponsor Member

rnelson0 commented Jan 8, 2017

@alexjfisher IIRC from IRC, you were investigating the patch to r10k option. Any update?

@vchepkov
Copy link
Contributor

vchepkov commented Aug 7, 2017

+1, every time some custom type changes parameters, one has to restart puppetserver to fix an environment

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

No branches or pull requests

4 participants