Skip to content
This repository has been archived by the owner on Feb 18, 2022. It is now read-only.

Added variable for confdir #2

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Added variable for confdir #2

wants to merge 2 commits into from

Conversation

jonnytdevops
Copy link

Hey Glenn,

Looks like this branch has been used by IT in production for nearly 1.5 years now! We should probably get this merged into production, thanks!

Cheers

JT

@genebean
Copy link
Contributor

genebean commented Apr 3, 2019

@jonnytdevops #1 was merged so lets NOT merge this one as it would undo some of that PR. I will contact you in Slack about how to migrate your control repo over to the newly merged changes so that we can close this branch out.

@jonnytdevops
Copy link
Author

@glennsarti I would be grateful if this could be merged. This PR removes the dependancy on puppet-agent for the sake of a folder location. I'm open to feedback to not hardcoding the location as a default value, but I'd prefer we used something that didn't need a whole other module. Thanks

@genebean
Copy link
Contributor

genebean commented Apr 3, 2019

FWIW @jonnytdevops #3 fixes things so you are also not dependent on the puppet_agent module while keeping it as an option.

@genebean
Copy link
Contributor

genebean commented Apr 5, 2019

The updates merged via #4 should make this unnecessary. Please let me know if you’d like any additional changes or corrections.

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

Successfully merging this pull request may close these issues.

2 participants