-
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
Set the unit unhealthy if output fails to start #35719
Conversation
This pull request does not have a backport label.
To fixup this pull request, you need to add the backport labels for the needed
|
0b8722c
to
93d7ea0
Compare
93d7ea0
to
809d9c7
Compare
If the output fails to start some inputs will go into a deadlock state, so we set the output unit as unhealthy and return early from the reload process.
6928a1a
to
02ee547
Compare
Pinging @elastic/elastic-agent (Team:Elastic-Agent) |
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
What does this PR do?
It fixes the status reporting when the output fails to start under Elastic-Agent and also prevents some deadlock conditions on inputs that will block its initialisation when the output is not ready.
Why is it important?
Checklist
- [ ] I have made corresponding changes to the documentation- [ ] I have made corresponding change to the default configuration filesCHANGELOG.next.asciidoc
orCHANGELOG-developer.next.asciidoc
.## Author's ChecklistHow to test this PR locally
Related issues
## Use cases## Screenshots## Logs