-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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 max client disconnect docs #12467
Add max client disconnect docs #12467
Conversation
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.
LGTM with some minor touch-ups.
Does this close #12448 ? |
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.
Awesome job!
ff9ed55
to
6791147
Compare
Committed all the suggestions. Thanks. I think it closes #12448 unless yall had something else in mind. We have the Edge Learn guide coming soon for in depth stuff too fwiw. |
Co-authored-by: Derek Strickland <1111455+DerekStrickland@users.noreply.github.com>
306b40c
to
27035cc
Compare
Rebased on the feature branch. |
Final preview link: https://nomad-3ug4o9ram-hashicorp.vercel.app/docs/job-specification/group |
I'm going to lock this pull request because it has been closed for 120 days ⏳. This helps our maintainers find and focus on the active contributions. |
No description provided.