-
-
Notifications
You must be signed in to change notification settings - Fork 84
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
add class to generate configs #116
Conversation
openssl::configs is a classthat may have no external impact to Forge modules. This module is declared in 2 of 575 indexed public
|
f64727f
to
82e85aa
Compare
manifests/configs.pp
Outdated
# dpendencies: ensure config file is generated before potential usage | ||
File<| tag=='openssl-configs' |> -> Ssl_pkey<| |> | ||
File<| tag=='openssl-configs' |> -> X509_cert<| |> | ||
File<| tag=='openssl-configs' |> -> X509_request<| |> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe #117 makes this moot, as each type now autorequires its config file.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ack
The new class openssl::configs create openssl.conf files. The generation bases on a new .epp template since epp can use parameters which was easier to integrate. closes: voxpupuli#41
82e85aa
to
9cc6b9b
Compare
Co-authored-by: Raphaël Pinson <github+aem1eeshi1@raphink.net>
Co-authored-by: Raphaël Pinson <github+aem1eeshi1@raphink.net>
Co-authored-by: Raphaël Pinson <github+aem1eeshi1@raphink.net>
That was a bit fast to remove... this reverts ae510aa from pull request voxpupuli#116 (remove dependencies since we have autorequire) and adds the dependencies between openssl config file and cert generations. the autorequire mentioned in the the original commit, meant the newly available autorequire in the defined types. But that one is an autorequire for the directory where we put the file to generate, not the configuration file !
That was a bit fast to remove... this reverts ae510aa from pull request #116 (remove dependencies since we have autorequire) and adds the dependencies between openssl config file and cert generations. the autorequire mentioned in the the original commit, meant the newly available autorequire in the defined types. But that one is an autorequire for the directory where we put the file to generate, not the configuration file !
The new class openssl::configs create openssl.conf files.
The generation bases on a new .epp template since epp can
use parameters which was easier to integrate.
closes: #41