Skip to content

Add unmanaged Puppet Agent runmode#659

Merged
ekohl merged 2 commits intotheforeman:masterfrom
Rathios:service_unmanaged
Jan 11, 2019
Merged

Add unmanaged Puppet Agent runmode#659
ekohl merged 2 commits intotheforeman:masterfrom
Rathios:service_unmanaged

Conversation

@Rathios
Copy link
Copy Markdown
Contributor

@Rathios Rathios commented Dec 25, 2018

Fixes GH-383

@ekohl
Copy link
Copy Markdown
Member

ekohl commented Dec 25, 2018

How does this differ from runmode none?

@Rathios
Copy link
Copy Markdown
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
Copy Markdown
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?

@mmoll
Copy link
Copy Markdown
Contributor

mmoll commented Jan 10, 2019

I'm OK with it after fixing 👍

@Rathios
Copy link
Copy Markdown
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
Copy Markdown
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