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 unmanaged Puppet Agent runmode #659

Merged
merged 2 commits into from
Jan 11, 2019

Conversation

Rathios
Copy link
Contributor

@Rathios Rathios commented Dec 25, 2018

Fixes GH-383

@ekohl
Copy link
Member

ekohl commented Dec 25, 2018

How does this differ from runmode none?

@Rathios
Copy link
Contributor Author

Rathios commented Dec 25, 2018

Left a comment here too: #383

Bottom line is I would like to be able to leave the service stopped while testing changes, undisturbed by the service. In that context I would also like to be able to enable the service without starting it, which I can do with a profile in the control-repository, but it's not currently possible to manually manage the service yourself as long as this module is used to install the agent.

Copy link
Member

@ekohl ekohl left a comment

Choose a reason for hiding this comment

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

I guess this makes sense but I do have one inline comment that applies 3 times. After that I'm OK with merging this. @mmoll?

manifests/agent/service/cron.pp Outdated Show resolved Hide resolved
@mmoll
Copy link
Contributor

mmoll commented Jan 10, 2019

I'm OK with it after fixing 👍

@Rathios
Copy link
Contributor Author

Rathios commented Jan 10, 2019

Ha, you're right. The consistency OCD missed the fact that $::puppet::unavailable_runmodes is NOT a string. Things make sense now. Fixed.

@ekohl ekohl merged commit ac2fdb0 into theforeman:master Jan 11, 2019
@ekohl
Copy link
Member

ekohl commented Jan 11, 2019

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants