-
Notifications
You must be signed in to change notification settings - Fork 304
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-5827 Stuck Thread count as MicroProfile Metric Gauge #5952
Conversation
...rofile/metrics/src/main/java/fish/payara/microprofile/metrics/jmx/MetricsMetadataHelper.java
Outdated
Show resolved
Hide resolved
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.
Is there a reason it seems to always say there is a stuck thread?
Changing the base branch to Payara6-tck as JavaEE8 Samples requires CDI 4.0, see payara/patched-src-javaee8-samples@5956e84#diff-d26e10d72530f9acdf8b358d1aa078c602ed4aa38dea26ca966117cb1857e3a2 |
jenkins test please |
Hi @Pandrex247, At the start of the request to
|
I don't understand. |
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.
Yeah I think this is a bug.
Setting the stuck thread threshold to 8 minutes and poking the endpoint still has it show 1 stuck thread.
We're closing the Payara6-tck branch, changing base. |
jenkins test please |
jenkins test please |
Description
This PR adds HealthCheck stats (Stuck thread count) to MicroProfile Metrics.
Important Info
StuckThreadsHealthCheck
required byMetricsServiceImpl
.MetricsServiceImpl
required byFaultToleranceServiceImpl
andMicroProfileMetricsChecker
.All of these services started at
@RunLevel(StartupRunLevel.VAL)
.The enable status of HealthCheck can not be checked during registration of Metrics metadata which is by default registered and during response writer operation, health check status is checked.
Testing
Testing Performed
Open
http://localhost:8080/metrics/vendor
URL which so=houl contains the following result:Now disable health check stuck-thread:
Open
http://localhost:8080/metrics/vendor
URL which so=houl contains the following result:Testing Environment
Windows 11, JDK 11.0.16