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

Add restart delay when minion fails to start #58523

Merged
merged 1 commit into from Sep 30, 2020

Conversation

twangboy
Copy link
Contributor

@twangboy twangboy commented Sep 22, 2020

What does this PR do?

Adds a restart delay of 60 seconds if the minion fails to start

What issues does this PR fix or reference?

Fixes: #58669

Previous Behavior

The minion would repeatedly try to restart on failure, eating up 100% cpu

New Behavior

Wait for 60 seconds before trying to restart again

Merge requirements satisfied?

[NOTICE] Bug fixes or features added to Salt require tests.

Commits signed with GPG?

Yes

@twangboy twangboy requested a review from a team as a code owner September 22, 2020 22:00
@twangboy twangboy requested review from dwoz and removed request for a team September 22, 2020 22:00
@ghost ghost requested a review from DmitryKuzmenko September 22, 2020 22:00
@twangboy twangboy added has master-port port to master has been created Magnesium Mg release after Na prior to Al and removed has master-port port to master has been created labels Sep 23, 2020
@dwoz dwoz merged commit 983d20e into saltstack:master Sep 30, 2020
26 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Magnesium Mg release after Na prior to Al
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[BUG] Salt-Minion restart loop on Windows
3 participants