newrelic / newrelic-java-agent

The New Relic Java agent
Apache License 2.0
202 stars 143 forks source link

Super agent health reporting #2000

Closed jtduffy closed 2 weeks ago

jtduffy commented 2 months ago

Implementation of file based health check reporting to the super agent/NR control.

In-flight agent spec: https://source.datanerd.us/agents/agent-specs/pull/690/files

codecov-commenter commented 4 weeks ago

Codecov Report

Attention: Patch coverage is 84.83412% with 32 lines in your changes missing coverage. Please review.

Project coverage is 70.65%. Comparing base (7cc5b0c) to head (e930b84). Report is 4 commits behind head on main.

Files with missing lines Patch % Lines
...nt/SuperAgentIntegrationHealthFileBasedClient.java 76.66% 4 Missing and 3 partials :warning:
...c/agent/superagent/SuperAgentIntegrationUtils.java 64.70% 5 Missing and 1 partial :warning:
...gent/config/SuperAgentIntegrationHealthConfig.java 70.58% 4 Missing and 1 partial :warning:
.../agent/config/SuperAgentIntegrationConfigImpl.java 82.35% 0 Missing and 3 partials :warning:
...a/com/newrelic/agent/transport/DataSenderImpl.java 78.57% 3 Missing :warning:
...om/newrelic/agent/config/ConfigServiceFactory.java 0.00% 2 Missing :warning:
...agent/superagent/SuperAgentIntegrationService.java 93.54% 0 Missing and 2 partials :warning:
...nt/introspec/internal/IntrospectorCoreService.java 0.00% 1 Missing :warning:
...ent/introspec/internal/IntrospectorRPMService.java 0.00% 1 Missing :warning:
...ic/agent/circuitbreaker/CircuitBreakerService.java 83.33% 1 Missing :warning:
... and 1 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2000 +/- ## ============================================ + Coverage 70.56% 70.65% +0.09% - Complexity 9879 9938 +59 ============================================ Files 829 836 +7 Lines 39961 40176 +215 Branches 6081 6096 +15 ============================================ + Hits 28197 28385 +188 - Misses 9032 9047 +15 - Partials 2732 2744 +12 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.