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

Settle on how to treat openvpn options #115

Open
nkakouros opened this issue Feb 17, 2019 · 0 comments
Open

Settle on how to treat openvpn options #115

nkakouros opened this issue Feb 17, 2019 · 0 comments
Labels
help wanted Mainteners needs help in moving an issue/PR forward
Milestone

Comments

@nkakouros
Copy link
Collaborator

The way openvpn config options are supported is a bit messy. It is a long list of possible values with no real cohesion. Many PRs have to do with adding this or that option. There are two ways to move forward:

  1. Support directly options that are 100% essential (by essential I mean options without which the config file will be invalid) and options that need special handling. The rest will be supported only via openvpn_server_options.

  2. Support all options via some logic. Instead of including every option in defaults/main.yml, the approach could be like in this or this role, ie have some kind of logic that checks/generates options either via some role tasks or via a shell script.

Though option 2 would feel nice to have in terms of completeness, openvpn is quite complex and option 1 seems the more sane approach.

Waiting for feedback by anyone interested.

@nkakouros nkakouros added the help wanted Mainteners needs help in moving an issue/PR forward label Feb 17, 2019
@nkakouros nkakouros added this to the Version 3.0 milestone Feb 17, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Mainteners needs help in moving an issue/PR forward
Projects
None yet
Development

No branches or pull requests

1 participant