Remove com.google.code.findbugs:jsr305 from gRPC

[mabartos]

Closes #44325

We will see how CI will behave.

Is there any problem with removing the dep from BOM? I don't think so, but not sure about your policy around this. The dep needs to be part of the BOM - at least mongodb-client extension uses it.

Unfortunately, grpc/grpc-java still provides the dependency, so we are not able to remove these exclusions.

[cescoffier]

The mongo compilation failure is unexpected - can you have a look?

[mabartos]

@cescoffier Yes, I'll look.

[gsmet]

@mabartos you can drop the @Nullable annotation from MongoTracingCommandListener. But please make sure this thing compiles before pushing as we might have other funkiness.

Running ./mvnw -T0.8C -Dquickly should be enough to verify things are not too bad.

[gsmet]

I think I handled the issue, let's see what CI says.

[quarkus-bot]

Status for workflow Quarkus Documentation CI

This is the status report for running Quarkus Documentation CI on commit 18eb86d.

✅ 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.

Warning

There are other workflow runs running, you probably need to wait for their status before merging.

[github-actions]

🎊 PR Preview bbe15e8 has been successfully built and deployed to https://quarkus-pr-main-44328-preview.surge.sh/version/main/guides/

• Images of blog posts older than 3 months are not available.
• Newsletters older than 3 months are not available.

[quarkus-bot]

Status for workflow Quarkus CI

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

✅ 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.

---

Flaky tests - Develocity

⚙️ Native Tests - HTTP

📦 integration-tests/rest-client-reactive

✖ io.quarkus.it.rest.client.BasicTestIT.shouldCreateClientSpans - History

• expected: <1> but was: <2> - org.opentest4j.AssertionFailedError

org.opentest4j.AssertionFailedError: expected: <1> but was: <2>
	at io.quarkus.it.rest.client.BasicTest.shouldCreateClientSpans(BasicTest.java:216)
	at java.base/java.lang.reflect.Method.invoke(Method.java:569)
	at io.quarkus.test.junit.QuarkusTestExtension.interceptTestMethod(QuarkusTestExtension.java:805)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)
	at java.base/java.util.ArrayList.forEach(ArrayList.java:1511)

[mabartos]

@gsmet Thanks for managing it! Didn't have time to look at it closer

[gsmet]

No worries, it looked like an easy win for 3.17.