-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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
Micrometer: allow veto of default MeterRegistry #17536
Conversation
/cc @phillip-kruger |
This workflow status is outdated as a new workflow run has been triggered. |
This workflow status is outdated as a new workflow run has been triggered. |
This workflow status is outdated as a new workflow run has been triggered. Failing Jobs - Building 27a9cf6
Full information is available in the Build summary check run. Test Failures⚙️ Native Tests - gRPC #📦 integration-tests/grpc-streaming✖ |
@stuartwdouglas .. This approach works (and hides complexity from the other registry extensions) but @luneo7 asked if |
This workflow status is outdated as a new workflow run has been triggered. 🚫 This workflow run has been cancelled. Failing Jobs - Building d21a3b1
Full information is available in the Build summary check run. Test Failures⚙️ JVM Tests - JDK 11 #📦 extensions/micrometer/deployment✖ ⚙️ JVM Tests - JDK 11 Windows #📦 extensions/micrometer/deployment✖ ⚙️ JVM Tests - JDK 16 #📦 extensions/micrometer/deployment✖ |
This workflow status is outdated as a new workflow run has been triggered. Failing Jobs - Building 418b9d0
Full information is available in the Build summary check run. Test Failures⚙️ JVM Tests - JDK 11 #📦 extensions/micrometer/deployment✖ ⚙️ JVM Tests - JDK 11 Windows #📦 extensions/micrometer/deployment✖ ⚙️ JVM Tests - JDK 16 #📦 extensions/micrometer/deployment✖ |
Superseded by #17570. |
Resolves #17481
Prefer #17570 to this one if the build is good.