Reinitialize com.sun.management.internal.PlatformMBeanProviderImpl

[zakkak]

Work around #25501 until
oracle/graal#4543 gets resolved

Fixes #25501

[zakkak]

@jiekang can you please have a look at this PR and verify that this reinitialization won't cause any JFR breakage? Thanks

[loicmathieu]

Thanks.
Are you sure the issue is only for gRPC based application ? Shouldn't this be on core-deployment instead ?
I can try this branch tomorrow afternoon to confirm it on GCP services if you didn't already did it.

[jiekang]

@jiekang can you please have a look at this PR and verify that this reinitialization won't cause any JFR breakage? Thanks

@zhengyu123 ^ any chance you have cycles for this?

[zakkak]

Are you sure the issue is only for gRPC based application ?

It's certainly not, but I don't really understand why it only pops up with gRPC.

Shouldn't this be on core-deployment instead ?

Not sure, I 'll leave that to the Quarkus team :)

I can try this branch tomorrow afternoon to confirm it on GCP services if you didn't already did it.

👍 I only tried it with the reproducer from #25501

[geoand]

Shouldn't this be on core-deployment instead ?

Not sure, I 'll leave that to the Quarkus team :)

I think it should be part of the core

[zakkak]

I think it should be part of the core

Done

[geoand]

Thanks!

[loicmathieu]

They fix it at Google library side, see GoogleCloudPlatform/native-image-support-java#395
If the issue occurs on other gRPC base extension it will not fix it for them as the Google native-image-support library is not integrated inside our gRPC support but inside the Google Services extension pack.

I hope they will do a release soon.

[zhengyu123]

@jiekang can you please have a look at this PR and verify that this reinitialization won't cause any JFR breakage? Thanks

@zhengyu123 ^ any chance you have cycles for this?

I don't see any problems.

-Zhengyu

[zakkak]

If the issue occurs on other gRPC base extension it will not fix it for them as the Google native-image-support library is not integrated inside our gRPC support but inside the Google Services extension pack.

To my understanding the issue itself is caused by the Google native-image-support library. If you look at the patch what it does is to remove some registrations for reflection, as a result if other extensions don't do similar registrations things should be fine.

[loicmathieu]

So maybe this fix is not needed anymore as the issue has been fixed on the upstream library ?
Or maybe we can keep it as a safety net in case other have done the same.

[quarkus-bot]

Failing Jobs - Building cbc41d4

Status	Name	Step	Failures	Logs	Raw logs
✖	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.devmode.QuarkusDevDependencyDevModeTest.main line 14 - More details - Source on GitHub

org.awaitility.core.ConditionTimeoutException: Condition with lambda expression in io.quarkus.test.devmode.util.DevModeTestUtils that uses java.util.function.Supplier, java.util.function.Supplierjava.util.concurrent.atomic.AtomicReference, java.util.concurrent.atomic.AtomicReferencejava.lang.String, java.lang.Stringboolean was not fulfilled within 1 minutes.
	at org.awaitility.core.ConditionAwaiter.await(ConditionAwaiter.java:167)
	at org.awaitility.core.CallableCondition.await(CallableCondition.java:78)

[zakkak]

So maybe this fix is not needed anymore as the issue has been fixed on the upstream library ?
Or maybe we can keep it as a safety net in case other have done the same.

I would prefer not getting this merged. If it turns out to be an actual issue for others as well we can reconsider. WDYT @geoand ?

[geoand]

Yeah, that makes sense to me