-
Notifications
You must be signed in to change notification settings - Fork 18
Open
Labels
bugSomething isn't workingSomething isn't workingneeds triageThis issue needs to be investigated and confirmed as a validThis issue needs to be investigated and confirmed as a valid
Description
Is there an existing issue for this?
- I have searched the existing issues
Current Behavior
NeonBee can't handle failing HealthChecks.
Local
In non-clustered mode the method getLocalHealthCheckResults is called which simply omits failed HealthChecks.
...
asyncCheckResults.stream().filter(Future::succeeded)
...Cluster
In clustered mode the method collectHealthCheckResults is called which sends requests to all HealthCheckVerticles. But as soon as one HealthCheck fails the HealthCheckVerticle will respond with a failure, because a AsyncHelper.allComposite collector is used to collect the results [1].
[1]
| return AsyncHelper.allComposite(checkList).map(v -> new JsonArray( |
Expected Behavior
- If a HealthCheck fails, I should see this in the result.
- If a HealthCheck fails, I should see the results of other HealthChecks.
HealthCheckVerticle.retrieveData(..)should re-usegetLocalHealthCheckResultsto remove code redundancy.
Steps To Reproduce
No response
Environment
- OS:
- Java:
- NeonBee:Relevant log output
No response
Anything else?
No response
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workingneeds triageThis issue needs to be investigated and confirmed as a validThis issue needs to be investigated and confirmed as a valid