Closed denis-yuen closed 1 month ago
Attention: Patch coverage is 85.36585%
with 6 lines
in your changes are missing coverage. Please review.
Project coverage is 74.53%. Comparing base (
1550b54
) to head (e494629
).
Files | Patch % | Lines |
---|---|---|
...ebservice/resources/ConnectionPoolHealthCheck.java | 85.00% | 6 Missing :warning: |
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
Issues
0 New issues
0 Accepted issues
Measures
0 Security Hotspots
85.4% Coverage on New Code
0.0% Duplication on New Code
Continued and replaced by https://github.com/dockstore/dockstore/pull/5873
Description This adds the reporting of database metrics to the healthcheck connection pool endpoint. Active, idle, size of the pool are reported along with calculated load (active connections over total size). The idea is to report this information to CloudWatch metrics so that we can develop auto-scaling policies (described in the linked ticket) using that data.
Note that the healthcheck endpoint is configured to be called by ECS every 60 seconds up from 30 seconds.
Follow-up ticket should investigate creating a custom reporter for dropwizatd metrics instead https://metrics.dropwizard.io/4.2.0/getting-started.html#other-reporting instead of hitchhiking on the healthcheck endpoint.
https://ucsc-cgl.atlassian.net/browse/SEAB-6401
Review Instructions TBD, will probably need benchmarking Use https://ucsc-cgl.atlassian.net/browse/SEAB-6402 for tracking
Issue https://ucsc-cgl.atlassian.net/browse/SEAB-5858
Security and Privacy
Metrics PUT temporarily triggerable via healthcheck endpoint
Please make sure that you've checked the following before submitting your pull request. Thanks!
mvn clean install
@RolesAllowed
annotation