Register all implementation of ExtensionAdapter in k8s client extension

[geoand]

Fixes: #29096

[geoand]

cc @manusa

[quarkus-bot]

Failing Jobs - Building 98f02aa

Status	Name	Step	Failures	Logs	Raw logs
✔️	Gradle Tests - JDK 11
✖	Gradle Tests - JDK 11 Windows	Build	Failures	Logs	Raw logs

Full information is available in the Build summary check run.

Failures

⚙️ Gradle Tests - JDK 11 Windows #

- Failing: integration-tests/gradle 

📦 integration-tests/gradle

✖ io.quarkus.gradle.QuarkusPluginFunctionalTest.canDetectClasspathChangeWhenBuilding line 98 - More details - Source on GitHub

org.opentest4j.AssertionFailedError: 

Expecting value to be true but was false

✖ io.quarkus.gradle.QuarkusPluginFunctionalTest.canDetectUpToDateBuild line 58 - More details - Source on GitHub

org.opentest4j.AssertionFailedError: 

Expecting value to be true but was false

[geoand]

@iocanel can you take a look at this one?

It would be nice to get it into the next 2.14 release.

Thanks

[iocanel]

This is amazing!

[geoand]

Thanks for the review

[manusa]

cc @manusa

Hi @geoand sorry for the late reply 😓

This one looks OK.

However, maybe we want to do:

serviceProviderProducer.produce(ServiceProviderBuildItem.allProvidersFromClassPath(ExtensionAdapter.class.getName()));

for the HttpClient.Factory.class class too.

Regarding this option ServiceProviderBuildItem.allProvidersFromClassPath, I learned about this on this PR. I assume this will discover any SPI provider that's available on the application's compile classpath.

If so, this will be very handy once we merge and release fabric8io/kubernetes-client#4511

[geoand]

@manusa thanks for the info. Feel free to send a PR updating the k8s client support

[manusa]

👍 Will do. I'll probably tackle everything once we release Kubernetes Client 6.3.0 with the mentioned improvement (hopefully next week 🤞)

[geoand]

👌

[gsmet]

Actually, this can't be backported to 2.13 as io.fabric8.kubernetes.client.extension.ExtensionAdapter is a new thing in Kubernetes Client 6.x and 2.13 is still using 5.x.

[gsmet]

Ah no, actually, it's there but in a different package. I will adjust the package in the backport.