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
Cleanup Systemd drop-in file handling #1396
Cleanup Systemd drop-in file handling #1396
Conversation
postgresql::server::instance::config is a typethat may have no external impact to Forge modules. postgresql::server::instance::systemd is a typethat may have no external impact to Forge modules. This module is declared in 70 of 580 indexed public
|
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.
Small suggestion regarding puppet versions
6d6364d
to
548f5b2
Compare
@bastelfreak @chelnak @ekohl this is ready for a final review, all previous comments have been addressed |
e3a83ea
to
3261f59
Compare
3261f59
to
f15dd8b
Compare
* add puppet/systemd module * remove systemd daemon reload and raise minimal puppet version to 6.1 * remove old "drop-in file" removal, was in place 3 years now * move systemd drop-in file handling to seperate define * Implement recent feedback * define is now private * rename parameter in define call * fix unit tests * fix rubocop complains * fix path, set default fact * fix systemd drop in file, adding template parameters to systemd define * added reason for drop in file in a comment * added reviewers feedback * remove ensure parameter for systemd drop-in file handling, except for the define itself. * systemd_extra_args expects String, default is undef now
f15dd8b
to
5eb1e8e
Compare
for the define itself.