-
Notifications
You must be signed in to change notification settings - Fork 277
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
Enable health metrics by default #1871
Conversation
I suggest updating the pr name/description with more useful verbiage for the release notes. |
This change breaks some CustomLogManager tests. Investigating further |
@randomanderson I suspect this is because of DogStatsD's usage of |
a4587de
to
8fa3988
Compare
I suspect we may need to plug the executors used by JMXFetch/DogStatsD, since some of these are still failing with the same cause. |
This will likely be in a better state after #1958 is merged. |
8fa3988
to
ea9c5fb
Compare
4cf8466
to
39cb0c7
Compare
def tracer = CoreTracer.builder().build() | ||
// FIXME this line should be unnecessary but a system property is leaking from somewhere | ||
// Remove when config overrides is fixed | ||
System.setProperty(PREFIX + HEALTH_METRICS_ENABLED, "true") |
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.
perhaps this should clear the property instead of setting to true?
No description provided.