-
Notifications
You must be signed in to change notification settings - Fork 4.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
[Elastic Agent] Increase checkin grace period. #23568
Conversation
Pinging @elastic/agent (Team:Agent) |
Pinging @elastic/ingest-management (Team:Ingest Management) |
💚 Build Succeeded
Expand to view the summary
Build stats
Test stats 🧪
💚 Flaky test reportTests succeeded. Expand to view the summary
Test stats 🧪
|
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
* Include checkin grace period. * Add changelog entry. (cherry picked from commit 2f50f9e)
* Include checkin grace period. * Add changelog entry. (cherry picked from commit 2f50f9e)
What does this PR do?
Increase the checkin grace period from 2 seconds to 30 seconds.
Why is it important?
Because the grace period is 2 seconds, it is possible that on a system that has enough load that between Agent and Endpoint checking in, it's possible that the grace period is not enough. Increasing the grace period doesn't really cause a negative effect, so its better to increase it so on systems with load degraded does not occur just because the process missed check-in by 2 seconds.
Checklist
[ ] I have commented my code, particularly in hard-to-understand areas[ ] I have made corresponding changes to the documentation[ ] I have made corresponding change to the default configuration files[ ] I have added tests that prove my fix is effective or that my feature worksCHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.Related issues