Skip to content

[DOC] Update FAQ about Salt self-restarting#46709

Merged
rallytime merged 1 commit intosaltstack:2017.7from
vutny:doc-faq-minion-master-restart
Mar 27, 2018
Merged

[DOC] Update FAQ about Salt self-restarting#46709
rallytime merged 1 commit intosaltstack:2017.7from
vutny:doc-faq-minion-master-restart

Conversation

@vutny
Copy link
Copy Markdown
Contributor

@vutny vutny commented Mar 26, 2018

What does this PR do?

Corrects the FAQ section on restarting a Salt Minion:

  • The "masterless mode" (--local) is no longer necessary
  • Salt Master could be restarted just like a Minion

What issues does this PR fix or reference?

Should finally resolve #5721

Commits signed with GPG?

Yes

Copy link
Copy Markdown
Contributor

@rallytime rallytime left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @vutny! This will be much more clear. I have a couple of suggestions and some grammatical changes. Otherwise, LGTM!

doc/faq.rst Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think in background by issuing --> in the background by issuing a is more readable. (Note the the and the a.)

I know the above line wasn't yours, but since we're here, can you also update restarting of the --> restarting the?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh, those English articles! 😃

doc/faq.rst Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

restarting of the --> restarting the

doc/faq.rst Outdated
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

version --> versions

And I think this part:

This allows the minion to keep the Minion

could be clearer. Maybe something like This allows the Minion to keep the Minion-Master connection alive ...

What do you think?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yep, that does not sound well. I will fix all these stuff. Thanks for reviewing!

@vutny
Copy link
Copy Markdown
Contributor Author

vutny commented Mar 26, 2018

@rallytime Thanks for grammar check again! 😄 I think I've sorted all of this out.

@rallytime rallytime merged commit 554400e into saltstack:2017.7 Mar 27, 2018
@vutny vutny deleted the doc-faq-minion-master-restart branch March 27, 2018 14:37
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.

2 participants