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/salt: restart on config changes #48499
Conversation
Success on x86_64-darwin (full log) Attempted: salt Partial log (click to expand)
|
Success on aarch64-linux (full log) Attempted: salt Partial log (click to expand)
|
Success on x86_64-linux (full log) Attempted: salt Partial log (click to expand)
|
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.
It does look reasonable. I am not entirely sure why I am the right person to review. It has been almost 2y since I used salt-stack regulary :/ (cc @Mic92)
@andir Then you have more experience then me. I only read other peoples configuration/documentation. |
@Mic92 it looks fine to me thus the +1, I can not test it more then reading it and it reads fine. I guess the only way we could know is if any of us would be using Salt instead of Nix... @aneeshusa Do you have some very basic configuration on how to test this? Could you write a nixos test so we can verify that it works in the future |
Should be fine: sudo systemctl status salt-minion.service
● salt-minion.service - Salt Minion
Loaded: loaded (/nix/store/gyibr8na6k3n44wa000c2d0ri6z3482i-unit-salt-minion.service/salt-minion.service; enabled; vendor preset: enabled)
Active: active (running) since Tue 2018-10-30 15:08:12 GMT; 30min ago
Main PID: 11807 (.salt-minion-wr)
Tasks: 4 (limit: 4915)
Memory: 80.3M
CGroup: /system.slice/salt-minion.service
├─11807 /nix/store/2brlr94ahy3a9mvcjy0qbqpv8zrb7b7s-python-2.7.15/bin/python2.7 /nix/store/v9jz3i2jlx72kqz55b5am7a7993bfznq-salt-2018.3.2/bin/.salt-minion-wrapped
├─11856 /nix/store/2brlr94ahy3a9mvcjy0qbqpv8zrb7b7s-python-2.7.15/bin/python2.7 /nix/store/v9jz3i2jlx72kqz55b5am7a7993bfznq-salt-2018.3.2/bin/.salt-minion-wrapped
└─11858 /nix/store/2brlr94ahy3a9mvcjy0qbqpv8zrb7b7s-python-2.7.15/bin/python2.7 /nix/store/v9jz3i2jlx72kqz55b5am7a7993bfznq-salt-2018.3.2/bin/.salt-minion-wrapped
Oct 30 15:36:47 turingmachine salt-minion[11807]: [ERROR ] DNS lookup or connection check of 'salt' failed.
Oct 30 15:36:47 turingmachine salt-minion[11807]: [ERROR ] Master hostname: 'salt' not found or not responsive. Retrying in 30 seconds
Oct 30 15:37:17 turingmachine salt-minion[11807]: [ERROR ] DNS lookup or connection check of 'salt' failed.
Oct 30 15:37:17 turingmachine salt-minion[11807]: [ERROR ] Master hostname: 'salt' not found or not responsive. Retrying in 30 seconds
Oct 30 15:37:47 turingmachine salt-minion[11807]: [ERROR ] DNS lookup or connection check of 'salt' failed.
Oct 30 15:37:47 turingmachine salt-minion[11807]: [ERROR ] Master hostname: 'salt' not found or not responsive. Retrying in 30 seconds
Oct 30 15:38:17 turingmachine salt-minion[11807]: [ERROR ] DNS lookup or connection check of 'salt' failed.
Oct 30 15:38:17 turingmachine salt-minion[11807]: [ERROR ] Master hostname: 'salt' not found or not responsive. Retrying in 30 seconds
Oct 30 15:38:47 turingmachine salt-minion[11807]: [ERROR ] DNS lookup or connection check of 'salt' failed.
Oct 30 15:38:47 turingmachine salt-minion[11807]: [ERROR ] Master hostname: 'salt' not found or not responsive. Retrying in 30 seconds |
Motivation for this change
Restart the Salt minion and master on config changes.
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)