open-telemetry / opentelemetry-java-instrumentation

OpenTelemetry auto-instrumentation and instrumentation libraries for Java
https://opentelemetry.io
Apache License 2.0
1.79k stars 782 forks source link

remove conditional indy test in CI #11631

Closed SylvainJuge closed 1 week ago

SylvainJuge commented 1 week ago

As discussed during June 13th Java SIG meeting, we should start always testing for "indy compatibility" in CI.

This helps for sub-tasks of #11457 that do not require to have the test indy label added to PRs for the test to be effective.

This also helps to prevent adding new instrumentations that are not "indy compatible" as the ones that were fixed in https://github.com/open-telemetry/opentelemetry-java-instrumentation/pull/11582

Once this PR is merged then the test indy label could be removed.

trask commented 1 week ago

merging, if there are any unexpected side-effects we can revisit