Stream: containers

Topic: ✔ Payara 6.2024.4

view this post on Zulip Oliver Bertuch (Jan 31 2024 at 07:47):

OMG I tried 6.2024.1 just now... Payara's quality is dwindling IMO... First I have to maneuver around them not releasing payara-api packages for month. And now this:

dev_dataverse> Caused by: java.lang.NullPointerException: Cannot invoke "fish.payara.nucleus.microprofile.config.spi.ConfigValueImpl.getSourceOrdinal()" because "resultWithoutProfile" is null
dev_dataverse>  at fish.payara.nucleus.microprofile.config.spi.ConfigExpressionResolver.resolve(ConfigExpressionResolver.java:112)
dev_dataverse>  at fish.payara.nucleus.microprofile.config.spi.ConfigExpressionResolver.resolveExpression(ConfigExpressionResolver.java:227)
dev_dataverse>  at fish.payara.nucleus.microprofile.config.spi.ConfigExpressionResolver.resolveExpression(ConfigExpressionResolver.java:153)
dev_dataverse>  at fish.payara.nucleus.microprofile.config.spi.ConfigExpressionResolver.getValue(ConfigExpressionResolver.java:135)
dev_dataverse>  at fish.payara.nucleus.microprofile.config.spi.ConfigExpressionResolver.resolve(ConfigExpressionResolver.java:108)
dev_dataverse>  at fish.payara.nucleus.microprofile.config.spi.ConfigExpressionResolver.resolve(ConfigExpressionResolver.java:91)
dev_dataverse>  at fish.payara.nucleus.microprofile.config.spi.PayaraConfig.searchConfigSources(PayaraConfig.java:352)
dev_dataverse>  at fish.payara.nucleus.microprofile.config.spi.PayaraConfig.getConfigValue(PayaraConfig.java:251)
dev_dataverse>  at fish.payara.nucleus.microprofile.config.spi.PayaraConfig.getConfigValue(PayaraConfig.java:229)
dev_dataverse>  at fish.payara.nucleus.microprofile.config.spi.PayaraConfig.getValue(PayaraConfig.java:224)
dev_dataverse>  at fish.payara.nucleus.microprofile.config.spi.PayaraConfig.getValueInternal(PayaraConfig.java:200)
dev_dataverse>  at fish.payara.nucleus.microprofile.config.spi.PayaraConfig.getOptionalValue(PayaraConfig.java:173)
dev_dataverse>  at edu.harvard.iq.dataverse.settings.JvmSettings.lookupOptional(JvmSettings.java:382)
dev_dataverse>  at edu.harvard.iq.dataverse.settings.JvmSettings.lookupOptional(JvmSettings.java:341)
dev_dataverse>  at edu.harvard.iq.dataverse.util.SystemConfig.getDataverseSiteUrlStatic(SystemConfig.java:265)
dev_dataverse>  at edu.harvard.iq.dataverse.DatasetVersion.<init>(DatasetVersion.java:197)

It's causing the whole deployment to fail!!! (Without even giving a non-profiled value)

view this post on Zulip Philip Durbin 🚀 (Jan 31 2024 at 12:24):

@Oliver Bertuch I hope you don't mind... I moved this to its own topic.

view this post on Zulip Oliver Bertuch (Jan 31 2024 at 13:30):

Great. I was able to reproduce the problem with a simple unit test...

view this post on Zulip Philip Durbin 🚀 (Jan 31 2024 at 13:42):

woof

view this post on Zulip Oliver Bertuch (Jan 31 2024 at 13:43):

Writing a bug report just now and probably will create a PR to Payara... :see_no_evil:

view this post on Zulip Oliver Bertuch (Jan 31 2024 at 14:56):

https://github.com/payara/Payara/pull/6550

view this post on Zulip Oliver Bertuch (Jan 31 2024 at 14:56):

In the end, an easy fix. And once more showing the importance of doing proper unit testing

view this post on Zulip Philip Durbin 🚀 (Jan 31 2024 at 14:56):

Nice.

view this post on Zulip Philip Durbin 🚀 (Jan 31 2024 at 14:57):

By the way if you change Fixes #6549 to - Fixes #6549 you get a nice preview of the title of the issue.

view this post on Zulip Philip Durbin 🚀 (Jan 31 2024 at 14:58):

Badger? Mushroom? These tests are great. :grinning:

view this post on Zulip Oliver Bertuch (Jan 31 2024 at 14:59):

I didn't write those. I suppose these are references to some comic / cartoon I'm not familiar with

view this post on Zulip Oliver Bertuch (Jan 31 2024 at 16:36):

Philip Durbin said:

Badger? Mushroom? These tests are great. :grinning:

It might be a reference to https://en.wikipedia.org/wiki/Badgers_(animation)

view this post on Zulip Philip Durbin 🚀 (Jan 31 2024 at 16:52):

British. Yes, probably. :grinning:

view this post on Zulip Balázs Pataki (Feb 01 2024 at 16:04):

Oliver Bertuch said:

Philip Durbin said:

Badger? Mushroom? These tests are great. :grinning:

It might be a reference to https://en.wikipedia.org/wiki/Badgers_(animation)

Who wrote those tests then? :smile: GPT?

view this post on Zulip Oliver Bertuch (Mar 22 2024 at 18:27):

Woo-hoo my PR was just merged!

view this post on Zulip Oliver Bertuch (Mar 22 2024 at 18:28):

I suppose it will be in 6.2024.4 then

view this post on Zulip Philip Durbin 🚀 (Mar 22 2024 at 18:29):

Congrats! :tada:

view this post on Zulip Oliver Bertuch (Apr 10 2024 at 12:48):

6.2024.4 has been released today

view this post on Zulip Notification Bot (Apr 10 2024 at 12:48):

Oliver Bertuch has marked this topic as resolved.

view this post on Zulip Juan Pablo Tosca Villanueva (Apr 10 2024 at 18:31):

To test if this release solves the issues with OpenAPI how can we get an image for testing :thinking:

view this post on Zulip Oliver Bertuch (Apr 10 2024 at 18:32):

Let me hack on stuff some more and it will be served :stuck_out_tongue:

view this post on Zulip Notification Bot (Apr 10 2024 at 18:36):

Philip Durbin has marked this topic as unresolved.

view this post on Zulip Oliver Bertuch (Apr 10 2024 at 19:33):

Kids embedded, will hack on stuff now

view this post on Zulip Oliver Bertuch (Apr 10 2024 at 19:34):

Will probably attend to the base image versioning first though

view this post on Zulip Philip Durbin 🚀 (Apr 10 2024 at 19:56):

I wish I could embed my kids. They stay up all night. Teenagers. :rolling_eyes:

view this post on Zulip Oliver Bertuch (Apr 10 2024 at 19:59):

https://www.youtube.com/watch?v=HBmIJFtn_qQ

view this post on Zulip Oliver Bertuch (Apr 10 2024 at 20:41):

The lyrics are below... :wink: You might need to put it through DeepL though :smile_cat:

view this post on Zulip Oliver Bertuch (Apr 15 2024 at 13:47):

Created the upgrade PR: https://github.com/IQSS/dataverse/pull/10495

view this post on Zulip Oliver Bertuch (Apr 15 2024 at 13:48):

I can confirm that my patch does indeed fix the issue during deployment!

view this post on Zulip Oliver Bertuch (Apr 15 2024 at 13:48):

So we can make our move :tada:

view this post on Zulip Philip Durbin 🚀 (Apr 18 2024 at 14:33):

In the description for this... Upgrade to Payara 6.2024.4+ #10494

I just added this: In containers, can't set DATAVERSE_SITEURL using documented methods #10285

view this post on Zulip Oliver Bertuch (Apr 18 2024 at 14:34):

You beat me to it! I was typing the same thing!

view this post on Zulip Philip Durbin 🚀 (Apr 18 2024 at 14:34):

:racecar:

view this post on Zulip Oliver Bertuch (Apr 18 2024 at 14:34):

I'm going to add a "closes" to the PR

view this post on Zulip Philip Durbin 🚀 (Apr 18 2024 at 14:35):

Perfect!

view this post on Zulip Oliver Bertuch (Apr 18 2024 at 14:36):

As @Juan Pablo Tosca Villanueva doesn't need 2024.4 for OpenAPI anymore, and as long as @Don Sizemore doesn't tell me the sec issues are really bad ass - should we make the PR to apply the upgrade to the containers only? Just encourage people to upgrade?

view this post on Zulip Oliver Bertuch (Apr 18 2024 at 14:37):

(On the other hand: it's been a year without a Payara update for people, so maybe it's good to make them?)

view this post on Zulip Oliver Bertuch (Apr 18 2024 at 14:38):

I'm just saying: this doesn't look good.
image.png

view this post on Zulip Oliver Bertuch (Apr 18 2024 at 14:38):

(rev2 has 6.2024.4)

view this post on Zulip Oliver Bertuch (Apr 18 2024 at 14:39):

Most of the yellow and light yellow ones are caused by Ubuntu 22.04 having an old imagemagick...

view this post on Zulip Oliver Bertuch (Apr 18 2024 at 14:39):

image.png

view this post on Zulip Oliver Bertuch (Apr 18 2024 at 14:40):

This will be resolved once 24.04 is out and we can finally upgrade the Temurin images

view this post on Zulip Philip Durbin 🚀 (Apr 18 2024 at 14:52):

Hmm, I think I'd like to upgrade Payara everywhere at once: containers, production, etc.

Mostly I'm just thinking that developers should stay on the same version as production.

view this post on Zulip Oliver Bertuch (Jul 03 2024 at 21:15):

This is done now with 6.3.

view this post on Zulip Notification Bot (Jul 03 2024 at 21:15):

Oliver Bertuch has marked this topic as resolved.

Last updated: Oct 30 2025 at 05:14 UTC