-
Notifications
You must be signed in to change notification settings - Fork 229
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
Systemd dependency missing #704
Comments
This is documented at https://github.com/theforeman/puppet-puppet#soft-dependencies |
I just wanted to question this dependency as I feel it potentially reaches too far outside of module scope. Should an application-specific module for puppet require a specific module for managing something as fundamental/system-wide as systemd? I certainly see that it makes things easier but it also means if this module is used, you can't use any other systemd module. |
Just curious why puppet::agent::service::systemd is creating systemd unit & timer files directly (not using systemd module) then? |
@jay7x that's a good point. The explanation is mostly historical. We added those when the systemd module wasn't as complete as today. It is probably a good idea to rewrite that to use the module as well. That might also be a good reason to make it a hard dependency since then there's functionality in both the agent and the server that uses it. |
#710 should fix that. |
I'm trying to install puppetsever with your module, but I get:
I have to manually install module
camptocamp/systemd
to fix it, so I guess it could be listed in requirements.The text was updated successfully, but these errors were encountered: