[fix][test] Disable OpenTelemetrySanityTest

[Technoboy-]

Motivation

OpenTelemetrySanityTest fails for every patch and blocks merging some important fixes.
It takes some time to fix it, so disable it .

We will track this issue by #22995.

Documentation

• doc
• doc-required
• doc-not-needed
• doc-complete

[lhotari]

Is the flaky group implemented for integration tests? I don't think so.

[lhotari]

flaky group isn't implemented for integration tests. You'd have to remove the test from tests/integration/src/test/resources/pulsar-metrics.xml to disable it.

[shibd]

flaky group isn't implemented for integration tests. You'd have to remove the test from tests/integration/src/test/resources/pulsar-metrics.xml to disable it.

It seems that this can also be avoided, although the CI is running, but in fact the test is not executed.

https://github.com/Technoboy-/pulsar/actions/runs/9777801504/job/26994205308?pr=52

[lhotari]

flaky group isn't implemented for integration tests. You'd have to remove the test from tests/integration/src/test/resources/pulsar-metrics.xml to disable it.

It seems that this can also be avoided, although the CI is running, but in fact the test is not executed.

https://github.com/Technoboy-/pulsar/actions/runs/9777801504/job/26994205308?pr=52

do you mean that the flaky group can be used for ignoring an integration test?

[shibd]

do you mean that the flaky group can be used for ignoring an integration test?

Yes, But this is not a good idea, and #22996 (review) is better

[Technoboy-]

do you mean that the flaky group can be used for ignoring an integration test?

Yes, But this is not a good idea, and #22996 (review) is better

Updated

[dragosvictor]

Test failure is addressed by 5a2e304. Upstream docker image had a breaking change in v0.104.0, where the listener binds to localhost unless otherwise instructed (ref).

[Technoboy-]

Test failure is addressed by 5a2e304. Upstream docker image had a breaking change in v0.104.0, where the listener binds to localhost unless otherwise instructed (ref).

Test failure is addressed by 5a2e304. Upstream docker image had a breaking change in v0.104.0, where the listener binds to localhost unless otherwise instructed (ref).

Ah, thanks for helping this, I try adding this in my local, and then run the test, but failed. it's a different error.

[shibd]

@Technoboy- Let wait for @dragosvictor to push a separate specifically PR from #22987 to fix the failing tests

[shibd]

Fix with by #22998