Improve MetricsTest reliability

[brunobat]

No description provided.

[quarkus-bot]

Thanks for your pull request!

Your pull request does not follow our editorial rules. Could you have a look?

• description should not be empty, describe your intent or provide links to the issues this PR is fixing (using Fixes #NNNNN) or changelogs

_{This message is automatically generated by a bot.}

[quarkus-bot]

/cc @ebullient (metrics), @jmartisk (metrics)

[gsmet]

Maybe you should display the content of spans instead? That might be more helpful if we have to debug it.

[brunobat]

Kind of... Most likely it was failing because it was also reading a span from a previous test, throwing the count to 3.
I just want make sure, if it happens again.

[quarkus-bot]

Status for workflow Quarkus CI

This is the status report for running Quarkus CI on commit c34a2de.

✅ The latest workflow run for the pull request has completed successfully.

It should be safe to merge provided you have a look at the other checks in the summary.

You can consult the Develocity build scans.

[holly-cummins]

I've just re-implemented this fix. I was about to do a PR to submit it to main. I rebased before my PR, and discovered you'd done the same fix at around the same time! Doh!

So yes, +1 for this fix from me. Especially with this test order, the lack of a wait in the reset caused quite consistent failures.

2025-01-27T13:47:46.7866097Z [INFO] Running io.quarkus.it.opentelemetry.EagerAuthEndUserEnabledTest
2025-01-27T13:47:56.7538432Z [INFO] Running io.quarkus.it.opentelemetry.LazyAuthEndUserEnabledTest
2025-01-27T13:48:04.7432007Z [INFO] Running io.quarkus.it.opentelemetry.StaticResourceTest
2025-01-27T13:48:06.4511240Z [INFO] Running io.quarkus.it.opentelemetry.OpenTelemetryInjectionsTest
2025-01-27T13:48:07.3800407Z [INFO] Running io.quarkus.it.opentelemetry.TracingTest
2025-01-27T13:48:20.0560604Z [INFO] Running io.quarkus.it.opentelemetry.LoggingResourceTest
2025-01-27T13:48:22.7209597Z [INFO] Running io.quarkus.it.opentelemetry.tck.SpanBeanTest
2025-01-27T13:48:22.7279496Z [INFO] Running io.quarkus.it.opentelemetry.MetricsTest

I have a minor variant of the fix that reduces the chattiness in the 'success' case, so I'll pop that in a PR.