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
mysql: add restartTrigger for my.cnf #54475
Conversation
@@ -249,6 +249,7 @@ in | |||
|
|||
after = [ "network.target" ]; | |||
wantedBy = [ "multi-user.target" ]; | |||
restartTriggers = [ config.environment.etc."my.cnf".source ]; |
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.
Is the variable working as you would expect? config.environment.etc."my.cnf".source
points into /nix/store to a static file. It may work when you set restartIfChanged
for te service.
I'd guess that /etc/my.cnf
is the correct thing to put here, however i have not yet debugged restartTriggers.
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.
Yes, I checked
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.
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.
Ah okay, restartTriggers
(and therefore X-Restart-Triggers
) is actually NixOS specific and a random commit in nixpkgs actually contains the documentation:
The triggers are just arbitrary strings that are included in the unit
under X-Restart-Triggers. The idea is that if they change between
reconfigurations, switch-to-configuration will restart the unit
because its store path changed. This is mostly useful for services
that implicitly depend on generated files in /etc. Thus you can say
restartTriggers = [ confFile ];
where ‘confFile’ is the derivation that generated the /etc file in
question.
Thanks! |
Motivation for this change
Allow restart mysql.service on change config my.cnf
Things done
sandbox
innix.conf
on non-NixOS)nix-shell -p nox --run "nox-review wip"
./result/bin/
)nix path-info -S
before and after)