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

multi instances of config options kills 2018.3.0 minions #367

Closed
colin-stubbs opened this issue Apr 27, 2018 · 0 comments · Fixed by #368
Closed

multi instances of config options kills 2018.3.0 minions #367

colin-stubbs opened this issue Apr 27, 2018 · 0 comments · Fixed by #368

Comments

@colin-stubbs
Copy link
Contributor

in particular the 'master_alive_interval' option is written twice in f_defaults.conf at line 98 and and 471

Result is,

2018-04-27 16:32:10,468 [salt.minion :1854][INFO ][49997] Returning information for job: 20180427163158790191
2018-04-27 16:32:11,060 [salt.config :2112][ERROR ][50106] Error parsing configuration file: /etc/salt/minion.d/f_defaults.conf - while constructing a mapping
in "/etc/salt/minion.d/f_defaults.conf", line 19, column 1
found conflicting ID 'master_alive_interval'
in "/etc/salt/minion.d/f_defaults.conf", line 402, column 1
2018-04-27 16:32:11,061 [salt.config :2253][ERROR ][50106] Error parsing configuration file: /etc/salt/minion.d/f_defaults.conf - while constructing a mapping
in "/etc/salt/minion.d/f_defaults.conf", line 19, column 1
found conflicting ID 'master_alive_interval'
in "/etc/salt/minion.d/f_defaults.conf", line 402, column 1
2018-04-27 16:32:12,882 [salt.utils.parsers:1045][WARNING ][50395] Minion received a SIGTERM. Exiting.
2018-04-27 16:32:12,882 [salt.cli.daemons :85 ][INFO ][50395] Shutting down the Salt Minion

Duplicate line writes need to be removed.

jtriley added a commit to jtriley/salt-formula that referenced this issue Jun 1, 2018
aboe76 pushed a commit that referenced this issue Jun 8, 2018
ixs pushed a commit to bawuenet/salt-formula that referenced this issue Jul 22, 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 a pull request may close this issue.

1 participant