cloudcaptainsh / cloudcaptain

Issue Tracker for CloudCaptain
13 stars 3 forks source link

Localization-Providers issues with `openjdk=11.0.13` caused by Flight Recorder #286

Closed igorrohal closed 1 year ago

igorrohal commented 1 year ago

Hi @axelfontaine.

We recently migrated to JRE11 and using the OpenJDK option of components.openjdk=11.0.13 which surfaces the following OpenJDK bug in our runtime - https://bugs.openjdk.org/browse/JDK-8261913 - and makes certain parts of our localization logic broken.

The Flight Recorded became the default part of OpenJDK as per here.

We can simulate a fix locally by not running the Java Flight Recorded which became the default part of OpenJDK 11.

How can we configure the final image to remove the Flight Recorder from the runtime and thus have the Localization Providers correctly loaded?

Thanks a lot

axelfontaine commented 1 year ago

For OpenJDK 11 and 17 we use the x64 Eclipse Temurin (AdoptOpenJDK's successor) distribution. If you wish to customize this, you can use our custom JRE "escape hatch" and strip down the OpenJDK distribution yourself, or build your own using jlink.

Please confirm things are now working for you again.

igorrohal commented 1 year ago

Hi Alex. Thanks a lot for getting back to me.

For OpenJDK 11 and 17 we use the x64 Eclipse Temurin (AdoptOpenJDK's successor) distribution

I'm currently looking into the option to provide our custom Eclipse Temurin JRE within the WAR archive, as it seems that the jdk.localedata (https://docs.oracle.com/en/java/javase/17/docs/api/jdk.localedata/module-summary.html) is by default included in the JRE that is available in the releases here.

Can it be that you're using a different and more stripped version of the JRE where you don't include the jdk.localedata module?

axelfontaine commented 1 year ago

@igorrohal I've pushed the newest JRE 11 (11.0.19) and 17 (17.0.7) versions to the Boxfuse Inventory. Their contents are unchanged from the source Temurin distributions, so hopefully this should now work for you.

igorrohal commented 1 year ago

@axelfontaine thanks a lot, it works all good now 🙏