-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
[WFLY-19351] Add Micrometer test to verify that applications with shared metrics names are not merged when exported #17898
base: main
Are you sure you want to change the base?
Conversation
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.
@jasondlee I made a bunch of code style comments but the substantive change LGTM.
...ile/src/test/java/org/wildfly/test/integration/observability/container/PrometheusMetric.java
Show resolved
Hide resolved
...org/wildfly/test/integration/observability/micrometer/MicrometerOtelIntegrationTestCase.java
Show resolved
Hide resolved
...ava/org/wildfly/test/integration/observability/micrometer/multiple/BaseMultipleTestCase.java
Show resolved
Hide resolved
...ava/org/wildfly/test/integration/observability/micrometer/multiple/BaseMultipleTestCase.java
Show resolved
Hide resolved
...ava/org/wildfly/test/integration/observability/micrometer/multiple/BaseMultipleTestCase.java
Outdated
Show resolved
Hide resolved
...java/org/wildfly/test/integration/observability/micrometer/multiple/MultipleWarTestCase.java
Show resolved
Hide resolved
...java/org/wildfly/test/integration/observability/micrometer/multiple/MultipleWarTestCase.java
Show resolved
Hide resolved
...test/integration/observability/micrometer/multiple/application/DuplicateMetricResource1.java
Show resolved
Hide resolved
...test/integration/observability/micrometer/multiple/application/DuplicateMetricResource2.java
Show resolved
Hide resolved
.../wildfly/test/integration/observability/micrometer/multiple/application/TestApplication.java
Show resolved
Hide resolved
Thanks. I'll get these updated as soon as I can. |
@bstansberry All changes should be address 'cept for the import order. Those have been updated, but I'll wait to resolve those until I can confirm I have the right order. :) |
bbec34b
to
c939c38
Compare
@jasondlee I'm withdrawing my approval because the tests are failing.
@bstansberry You're so picky. :) I've been reworking the tests so they should be more stable. Will be pushing an update "soon" |
3b7ea25
to
496bc27
Compare
…red metrics names are not merged when exported Add test with multiple war deployments Add test with ear with multiple war subdeployments Rework how Prometheus metrics are retrieved and handled Add test to verify that metrics with identical names are exported separately Add a TestEnricher to manage lifecycle and injection of the Otel container
https://issues.redhat.com/browse/WFLY-19351
Add test with multiple war deployments
Add test with ear with multiple war subdeployments
Rework how Prometheus metrics are retrieved and handled
Add test to verify that metrics with identical names are exported separately
More information about the wildfly-bot[bot]