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

nixos/monit: reload monit if config changes #38019

Closed
wants to merge 1 commit into from

Conversation

ryantm
Copy link
Member

@ryantm ryantm commented Mar 28, 2018

Previously, if monit.config changes, the monit service would not reload it. This commit adds a systemd path and unit to check for changes and reload if the config file changes.

I'm not sure if this is the right way to do this. It seems kind of hacky that it isn't an activation script. I think most services accomplish this kind of reloading/restarting by having the configuration file passed as a parameter to the service in ExecStart, but we cannot do this because Monit expects the configuration file to have mode 400, which is not possible for a file directly in the nix store.

Previously, if monit.config changes, the monit service would not
reload it. This commit adds a systemd path and unit to check for
changes and reload if the config file changes.
@ryantm
Copy link
Member Author

ryantm commented Mar 28, 2018

Closing in favor of using restartTriggers

@ryantm ryantm closed this Mar 28, 2018
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.

None yet

1 participant