Skip to content

Upgrade to Java 11 and to Metafacture 7.0.0 #455#459

Merged
Phu2 merged 12 commits intomasterfrom
455-updateJavaVersionTo11
Feb 23, 2026
Merged

Upgrade to Java 11 and to Metafacture 7.0.0 #455#459
Phu2 merged 12 commits intomasterfrom
455-updateJavaVersionTo11

Conversation

@TobiasNx
Copy link
Contributor

@TobiasNx TobiasNx commented Feb 18, 2026

@fsteeg not sure if this is a quick adjustment? but the tests seem to work fine I reproduced the way we use java 11 in context of lobid-organisations and build upon @Phu2 PR #456

This is not urgent, but maybe helpful.

Resolve #455

@TobiasNx TobiasNx changed the base branch from align-start-script to master February 18, 2026 09:26
@TobiasNx TobiasNx force-pushed the 455-updateJavaVersionTo11 branch from 2387a85 to bea8e65 Compare February 18, 2026 09:43
@TobiasNx TobiasNx force-pushed the 455-updateJavaVersionTo11 branch from bea8e65 to 55ba569 Compare February 18, 2026 09:44
@fsteeg
Copy link
Member

fsteeg commented Feb 18, 2026

@fsteeg not sure if this is a quick adjustment? but the tests seem to work fine

I don't think so, the app doesn't run for me. I think we need to at least update scalaVersion, javacOptions, and sbt.version (but probably more), see hbz/rpb@0b59e22#diff-5634c415cd8c8504fdb973a3ed092300b43c4b8fc1e184f7249eb29a55511f91

@TobiasNx
Copy link
Contributor Author

TobiasNx commented Feb 18, 2026

sbt run works if you use the weywots instead of the localhost index. FYI @fsteeg

@TobiasNx TobiasNx requested a review from fsteeg February 18, 2026 16:29
@TobiasNx TobiasNx assigned fsteeg and unassigned fsteeg Feb 18, 2026
@TobiasNx

This comment was marked as outdated.

@fsteeg
Copy link
Member

fsteeg commented Feb 19, 2026

Same as in hbz/nwbib#679, I think we should stick with our process and do a separate functional review (e.g. by @acka47) before code review.

@fsteeg fsteeg removed their request for review February 19, 2026 09:33
@acka47
Copy link
Contributor

acka47 commented Feb 19, 2026

I am happy to make a functional review, however, I am refraining from this for now as this is still a draft PR and the issue was not assigned to me.

@TobiasNx TobiasNx force-pushed the 455-updateJavaVersionTo11 branch from 86b3403 to dd62950 Compare February 19, 2026 14:33
@TobiasNx TobiasNx changed the title Quick test to upgrade to Java 11 #455 Upgrade to Java 11 and Metafacture to 7.0.0 #455 Feb 19, 2026
@TobiasNx TobiasNx changed the title Upgrade to Java 11 and Metafacture to 7.0.0 #455 Upgrade to Java 11 and to Metafacture 7.0.0 #455 Feb 19, 2026
@TobiasNx TobiasNx marked this pull request as ready for review February 19, 2026 14:43
@TobiasNx TobiasNx removed their assignment Feb 19, 2026
@fsteeg fsteeg assigned TobiasNx and unassigned fsteeg Feb 20, 2026
kill $(cat target/universal/stage/RUNNING_PID)
fi
JAVA_OPTS="$JAVA_OPTS -XX:+ExitOnOutOfMemoryError" sbt --java-home $JAVA_HOME "start $PORT"
sbt clean
Copy link
Contributor Author

@TobiasNx TobiasNx Feb 20, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The reintroduces sbt clean and sbt stage that @Phu2 dropped before: d7d693f

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Like I said, both is fine here, lobid-gnd doesn't log to target.

@TobiasNx TobiasNx requested a review from fsteeg February 20, 2026 11:45
@TobiasNx TobiasNx assigned fsteeg and unassigned TobiasNx Feb 20, 2026
@fsteeg fsteeg assigned TobiasNx and unassigned fsteeg Feb 20, 2026
@TobiasNx
Copy link
Contributor Author

@Phu2 you can deploy and merge

@TobiasNx TobiasNx assigned Phu2 and unassigned TobiasNx Feb 20, 2026
index {
boot: {
hosts: ["localhost"] # ["weywot3.hbz-nrw.de", "weywot4.hbz-nrw.de", "weywot5.hbz-nrw.de"]
hosts: ["localhost"] # ["weywot13.hbz-nrw.de", "weywot14.hbz-nrw.de", "weywot15.hbz-nrw.de"]
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why do we add internal server adresses here? I don't think we should publish them.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@fsteeg could you help here?

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Since these are commented out anyway, you could remove them here. The idea would be to have an internal Wiki page for the details, like we have for RPB.

@fsteeg fsteeg assigned TobiasNx and unassigned Phu2 Feb 23, 2026
@Phu2
Copy link
Contributor

Phu2 commented Feb 23, 2026

Trying to deploy this branch on test, monit (re)starts the app several times without success. From monit.log:

[2026-02-23T10:29:14+0100] error    : 'lobid-gnd' process is not running
[2026-02-23T10:29:14+0100] info     : 'lobid-gnd' trying to restart
[2026-02-23T10:29:14+0100] info     : 'lobid-gnd' start: '/home/sol/git/lobid-gnd/monit_restart.sh lobid-gnd start 7100'
[2026-02-23T10:29:45+0100] error    : 'lobid-gnd' failed to start (exit status 0) -- '/home/sol/git/lobid-gnd/monit_restart.sh lobid-gnd start 7100': WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by sbt.ivyint.ErrorMessageAuthenticator$ (file:/home/sol/.sbt/boot/scala-2.10.7/org.scala-sbt/sbt/0.13.18/ivy.jar) to field java.net.Authenticator.theAuthentic
[2026-02-23T10:31:05+0100] error    : 'lobid-gnd' process is not running
[2026-02-23T10:31:05+0100] info     : 'lobid-gnd' trying to restart
[2026-02-23T10:31:05+0100] info     : 'lobid-gnd' start: '/home/sol/git/lobid-gnd/monit_restart.sh lobid-gnd start 7100'
[2026-02-23T10:31:35+0100] error    : 'lobid-gnd' failed to start (exit status 0) -- '/home/sol/git/lobid-gnd/monit_restart.sh lobid-gnd start 7100': WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by sbt.ivyint.ErrorMessageAuthenticator$ (file:/home/sol/.sbt/boot/scala-2.10.7/org.scala-sbt/sbt/0.13.18/ivy.jar) to field java.net.Authenticator.theAuthentic
[2026-02-23T10:32:55+0100] error    : 'lobid-gnd' process is not running
[2026-02-23T10:32:55+0100] info     : 'lobid-gnd' trying to restart
[2026-02-23T10:32:55+0100] info     : 'lobid-gnd' start: '/home/sol/git/lobid-gnd/monit_restart.sh lobid-gnd start 7100'
[2026-02-23T10:33:26+0100] error    : 'lobid-gnd' failed to start (exit status 0) -- '/home/sol/git/lobid-gnd/monit_restart.sh lobid-gnd start 7100': WARNING: An illegal reflective access operation has occurred
WARNING: Illegal reflective access by sbt.ivyint.ErrorMessageAuthenticator$ (file:/home/sol/.sbt/boot/scala-2.10.7/org.scala-sbt/sbt/0.13.18/ivy.jar) to field java.net.Authenticator.theAuthentic

@TobiasNx @fsteeg I have no clue why it won't start.

@fsteeg
Copy link
Member

fsteeg commented Feb 23, 2026

@TobiasNx @fsteeg I have no clue why it won't start.

On q13? From logs/application.log (that's where you'd see errors from the app during startup too) it seems it started successfully at 10:45, but with Java 8. Did you roll it back there? Or are you testing elsewhere?

@Phu2
Copy link
Contributor

Phu2 commented Feb 23, 2026

On q13? From logs/application.log (that's where you'd see errors from the app during startup too) it seems it started successfully at 10:45, but with Java 8. Did you roll it back there? Or are you testing elsewhere?

Yes, on q13. I did a rollback in between.

@fsteeg
Copy link
Member

fsteeg commented Feb 23, 2026

The problem was the missing -no-version-check when starting the app in monit_restart.sh (see inline comment above). I probably forgot that I added it locally when testing this during initial review. Now running on Java 11 on https://test.lobid.org/gnd (In our regular process, only now would have been the point to start functional review, not sure how to handle that in this case, probably not needed to do another round with @acka47?).

From logs/application.log (that's where you'd see errors from the app during startup too)

That doesn't seem to be true (any longer?), since I saw nothing there during attempted startup without -no-version-check. Not sure if this is changed behavior due to our recent changes in the startup scripts or if this would have been the same before.

When manually running /target/universal/stage/bin/lobid-gnd -Dhttp.port=7100 (what monit_restart.sh calls) I got:

The java installation you have is not up to date
requires at least version 1.6+, you have
version 11.0.30

That provided the clue to the missing -no-version-check (which we originally figured out in hbz/lobid-resources#1965 (comment)).

@acka47
Copy link
Contributor

acka47 commented Feb 23, 2026

I quickly did some functional review and things are looking fine to me. +1

(I only noticed one thing: The manifest URL in https://test.lobid.org/gnd/reconcile/ is not correct. It should be https://test.lobid.org/gnd/reconcile/ instead of https://reconcile-gnd-test.lobid.org but I guess it has been like this for a long time.)

Co-authored-by: Fabian Steeg <steeg@hbz-nrw.de>
@Phu2 Phu2 assigned Phu2 and unassigned TobiasNx Feb 23, 2026
@Phu2 Phu2 merged commit 00c1aeb into master Feb 23, 2026
1 check passed
@Phu2 Phu2 deleted the 455-updateJavaVersionTo11 branch February 23, 2026 15:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Upgrade Java Version to 11

4 participants