Skip to content

Commit 7a9bf73

Browse files
committed
use legacy for all
1 parent bd26277 commit 7a9bf73

File tree

3 files changed

+7
-7
lines changed

3 files changed

+7
-7
lines changed

jmx-scraper/src/integrationTest/java/io/opentelemetry/contrib/jmxscraper/target_systems/JvmIntegrationTest.java

Lines changed: 0 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -32,12 +32,6 @@ protected JmxScraperContainer customizeScraperContainer(
3232
JmxScraperContainer scraper, GenericContainer<?> target, Path tempDir) {
3333
return scraper
3434
.withTargetSystem("jvm")
35-
// Since JVM metrics were be added to instrumentation, the default "auto" source
36-
// means that the definitions in instrumentation will be used, and thus this test will fail
37-
// due to metrics differences, adding an explicit "legacy" source is required to continue
38-
// testing the JVM metrics defined in this project.
39-
// https://github.com/open-telemetry/opentelemetry-java-instrumentation/pull/13392
40-
.withTargetSystemSource("legacy")
4135
// also testing custom yaml
4236
.withCustomYaml("custom-metrics.yaml");
4337
}

jmx-scraper/src/integrationTest/java/io/opentelemetry/contrib/jmxscraper/target_systems/TargetSystemIntegrationTest.java

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -143,6 +143,12 @@ protected void startContainers(Path tmpDir) {
143143
// Create and initialize scraper container
144144
scraper =
145145
new JmxScraperContainer(otlpEndpoint, scraperBaseImage())
146+
// Since JVM metrics were be added to instrumentation, the default "auto" source
147+
// means that the definitions in instrumentation will be used, and thus tests will fail
148+
// due to metrics differences, adding an explicit "legacy" source is required to
149+
// continue
150+
// testing metrics defined in this project.
151+
.withTargetSystemSource("legacy")
146152
.withLogConsumer(new Slf4jLogConsumer(jmxScraperLogger))
147153
.withNetwork(network)
148154
.withRmiServiceUrl(TARGET_SYSTEM_NETWORK_ALIAS, JMX_PORT);

jmx-scraper/src/integrationTest/java/io/opentelemetry/contrib/jmxscraper/target_systems/TomcatIntegrationTest.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -42,7 +42,7 @@ protected GenericContainer<?> createTargetContainer(int jmxPort) {
4242
@Override
4343
protected JmxScraperContainer customizeScraperContainer(
4444
JmxScraperContainer scraper, GenericContainer<?> target, Path tempDir) {
45-
return scraper.withTargetSystemSource("legacy").withTargetSystem("tomcat");
45+
return scraper.withTargetSystem("tomcat");
4646
}
4747

4848
@Override

0 commit comments

Comments
 (0)