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

Added parameter (startall = true | false ) to allow start all configs #124

Closed
wants to merge 1 commit into from
Closed

Conversation

marianschmotzer
Copy link

... in openvpn directory, not only puppet managed.Works only on debian

…igs in openvpn directory, not only puppet managed.Works only on debian
@luxflux
Copy link
Contributor

luxflux commented Feb 2, 2015

Thanks for your contribution. Unfortunately this won't work out as you implemented it. This way, it will create a default file with the following content if you specify multiple services:

AUTOSTART="all"
AUTOSTART="all"
AUTOSTART="all"
AUTOSTART="$AUTOSTART server123"

The last one would be a service which doesn't specify start_all => true...

This should be done on a central configuration class. Maybe we should add some parameters to openvpn directly in init.pp? What do you think?

@marianschmotzer
Copy link
Author

Hi
Maybe i can create facter to get line AUTOSTART="all" from config. If it
is there we don't need to add any lines, because all configuration files
will be executed anyway.
And switch startall can stay in place (or if you would like to change name,
lease write me new one).
Or maybe another option will be to add parameter 'managestartconfig =>
true|false' with obvious function - puppet will or will not manage config.
Please tell me what do you think is better - my personal opinion is that
facter option is better.

Tnx

2015-02-02 18:53 GMT+01:00 Raffael Schmid notifications@github.com:

Thanks for your contribution. Unfortunately this won't work out as you
implemented it. This way, it will create a default file with the
following content if you specify multiple services:

AUTOSTART="all"
AUTOSTART="all"
AUTOSTART="all"
AUTOSTART="$AUTOSTART server123"

The last one would be a service which doesn't specify start_all => true...

This should be done on a central configuration class. Maybe we should add
some parameters to openvpn directly in init.pp? What do you think?


Reply to this email directly or view it on GitHub
#124 (comment)
.

@luxflux
Copy link
Contributor

luxflux commented Mar 3, 2015

Thanks for your reply, @marianschmotzer. I have to dig into this first to find a good solution, stay tuned.

@luxflux
Copy link
Contributor

luxflux commented Mar 6, 2015

I went for the variable on openvpn and made a new pull request #144. Thanks for your work!

@luxflux luxflux closed this in 8fe2e3e Mar 8, 2015
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.

2 participants