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

Made server config filepaths relative to directory parameter #79

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

Made server config filepaths relative to directory parameter #79

wants to merge 1 commit into from

Conversation

cliff-wakefield
Copy link

No description provided.

@cliff-wakefield
Copy link
Author

This partially addresses issue #67

@mcblum
Copy link

mcblum commented Dec 11, 2017

@thias Any chance this could be merged? Pretty sure this is broken on Ubuntu until the files can be relative to the directory.

@mcblum
Copy link

mcblum commented Dec 11, 2017

@cliff-wakefield probably a dumb question but is there a way I can use your version? somewhat new to Puppet so I've just been installing using Forge

@cliff-wakefield
Copy link
Author

@mcblum not a dumb question.

If you are doing manual module install then I don't think the puppet command can install from a git repo.

However, if you are using librarian-puppet then you can specify a git branch.

Seems I have deleted my repo, however, I will need to find the new home of the repo.

@mcblum
Copy link

mcblum commented Dec 11, 2017

@cliff-wakefield thanks man. This will work great - given the current repo you can specify the location in named.conf but that leads to a permission error anyway. Could probably be fixed but the best thing would be to just write the file in the correct place. Maybe we can house a version somewhere for people to use?

@thias
Copy link
Owner

thias commented Apr 23, 2018

This didn't used to work, and I haven't found any information about when it might have started to work. Using parameters to set other parameters from the same class or definition doesn't always work as expected. See https://tickets.puppetlabs.com/browse/PUP-1080.

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.

4 participants