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
FISH-660 Upgrade MicroProfile Health Implementation to 3.0-RC5 #4986
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.
One big question mark where I need some help understanding the change.
...ofile/healthcheck/src/main/java/fish/payara/microprofile/healthcheck/HealthCheckService.java
Outdated
Show resolved
Hide resolved
@health annotation has been removed, so the backward compatibility flag has also been removed. Signed-off-by: Matthew Gill <matthew.gill@live.co.uk>
This is a breaking change, but the MP health configuration was incorrectly called "metrics-healthcheck-configuration" which was confusing. Signed-off-by: Matthew Gill <matthew.gill@live.co.uk>
jenkins test please |
Monitoring still collected Health data, which has been removed. The 'overall' metric has also been renamed to health as it now feels more appropriate. Signed-off-by: Matthew Gill <matthew.gill@live.co.uk>
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.
Have not tested it yet but looks good to me now.
I checked InSight docs and don't think it needs an update. It was general enough to not be affected by this change.
Jenkins test please |
jenkins test please |
1 similar comment
jenkins test please |
jenkins test please |
1 similar comment
jenkins test please |
Fails because the TCK is randomly downgraded, waiting on payara/MicroProfile-TCK-Runners#125 |
Conflicts arose caused by FISH-327 MP Sniffers. These are integrated into the Health upgrade. Signed-off-by: Matthew Gill <matthew.gill@live.co.uk>
jenkins test please |
1 similar comment
jenkins test please |
FISH-660 Upgrade MicroProfile Health Implementation to 3.0-RC5
FISH-660 Upgrade MicroProfile Health Implementation to 3.0-RC5
Description
This is a feature. Will need upgrading to the final release version when it's release, but for now the release candidate is good enough.
Testing
New tests
TCK updated: payara/MicroProfile-TCK-Runners#122
Testing Performed
Ran the new TCK against our current implementation
Documentation
TBW
Notes for Reviewers