New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

build(deps): upgrade quarkus to 3.2.9 #352

Merged

andrewazores merged 2 commits into cryostatio:main from andrewazores:quarkus-version

Apr 11, 2024

Member

andrewazores commented Apr 5, 2024

No description provided.

andrewazores requested a review from a team as a code owner

April 5, 2024 15:11

github-actions bot added the needs-triage label

andrewazores added dependencies build chore safe-to-test and removed needs-triage labels

Member Author

andrewazores commented Apr 5, 2024

/build_test

github-actions bot commented Apr 5, 2024

Workflow started at 4/5/2024, 11:12:26 AM. View Actions Run.

github-actions bot commented Apr 5, 2024

No OpenAPI schema changes detected.

github-actions bot commented Apr 5, 2024

CI build and push: All tests pass ✅ (JDK17)
https://github.com/cryostatio/cryostat3/actions/runs/8571934405

Member Author

andrewazores commented Apr 5, 2024

This causes bytebuddy to be upgraded:

(main) $ mvn dependency:tree | grep bytebuddy
[INFO] |  |  +- net.bytebuddy:byte-buddy:jar:1.12.18:compile
[INFO] |  |     +- net.bytebuddy:byte-buddy-agent:jar:1.14.4:test

(quarkus-version) $ mvn dependency:tree | grep bytebuddy
[INFO] |  |  +- net.bytebuddy:byte-buddy:jar:1.14.7:compile
[INFO] |  |     +- net.bytebuddy:byte-buddy-agent:jar:1.14.4:test

The newer version here should support JDK 21. The CI runner for 21 looks like it's hanging however, but it's unclear why or if that's actually due to some problem with the application using JDK 21. It got further than it normally did with the new bytebuddy version, anyway.

raphw/byte-buddy#1396

Member Author

andrewazores commented Apr 5, 2024

I see the same "JDP prune thread" shutdown hang behaviour when running tests locally using JDK 21 now.

andrewazores marked this pull request as draft

April 5, 2024 15:32

andrewazores mentioned this pull request

build(deps): bump quarkus.platform.version from 3.2.6.Final to 3.9.2 #351

Closed

andrewazores force-pushed the quarkus-version branch 2 times, most recently from e6bc66b to cbd3a63 Compare

April 5, 2024 16:32

Member Author

andrewazores commented Apr 5, 2024

/build_test

github-actions bot commented Apr 5, 2024

Workflow started at 4/5/2024, 12:34:15 PM. View Actions Run.

github-actions bot commented Apr 5, 2024

CI build and push: At least one test failed ❌ (JDK21)
https://github.com/cryostatio/cryostat3/actions/runs/8571934405

github-actions bot commented Apr 5, 2024

No OpenAPI schema changes detected.

github-actions bot commented Apr 5, 2024

CI build and push: All tests pass ✅ (JDK17)
https://github.com/cryostatio/cryostat3/actions/runs/8572918569

github-actions bot commented Apr 5, 2024

CI build and push: At least one test failed ❌ (JDK21)
https://github.com/cryostatio/cryostat3/actions/runs/8572918569

Member Author

andrewazores commented Apr 5, 2024

The JDK 21 thread hang doesn't appear to happen with #351 , so it looks like some threading interaction between older Quarkus and/or how we're using it, and JDK 21. Upgrading the Quarkus version and JDK version at the same time should get around it.


          build(deps): upgrade quarkus to 3.2.9

37dfc5b

andrewazores force-pushed the quarkus-version branch from cbd3a63 to 37dfc5b Compare

April 11, 2024 14:12


          run CI only for JDK 17

d7a1698

andrewazores marked this pull request as ready for review

April 11, 2024 14:13

andrewazores merged commit bbbbf38 into cryostatio:main

6 of 7 checks passed

andrewazores deleted the quarkus-version branch

April 11, 2024 14:14

This was referenced Apr 11, 2024

[Request] Upgrade to Quarkus 3.8 LTS and JDK 21 #364

Closed

[Bug] Report generation failure #367

Closed

andrewazores added a commit to andrewazores/cryostat3 that referenced this pull request


          Revert "build(deps): upgrade quarkus to 3.2.9 (cryostatio#352)"

d4dfe75

This reverts commit bbbbf38.

andrewazores added a commit to andrewazores/cryostat3 that referenced this pull request


          Reapply "build(deps): upgrade quarkus to 3.2.9 (cryostatio#352)"

3da1b29

This reverts commit d4dfe75.

andrewazores added a commit to andrewazores/cryostat3 that referenced this pull request


          Revert "build(deps): upgrade quarkus to 3.2.9 (cryostatio#352)"

4ade1dd

This reverts commit bbbbf38.

andrewazores added a commit to andrewazores/cryostat3 that referenced this pull request


          Reapply "build(deps): upgrade quarkus to 3.2.9 (cryostatio#352)"

d063761

This reverts commit d4dfe75.

andrewazores added a commit to andrewazores/cryostat3 that referenced this pull request


          Revert "build(deps): upgrade quarkus to 3.2.9 (cryostatio#352)"

f5f59c6

This reverts commit bbbbf38.

andrewazores added a commit to andrewazores/cryostat3 that referenced this pull request


          Reapply "build(deps): upgrade quarkus to 3.2.9 (cryostatio#352)"

This reverts commit d4dfe75.

andrewazores added a commit to andrewazores/cryostat3 that referenced this pull request


          Revert "build(deps): upgrade quarkus to 3.2.9 (cryostatio#352)"

65bf1d9

This reverts commit bbbbf38.

andrewazores added a commit to andrewazores/cryostat3 that referenced this pull request


          Reapply "build(deps): upgrade quarkus to 3.2.9 (cryostatio#352)"

db43a36

This reverts commit d4dfe75.

andrewazores added a commit to andrewazores/cryostat3 that referenced this pull request


          Revert "build(deps): upgrade quarkus to 3.2.9 (cryostatio#352)"

b909061

This reverts commit bbbbf38.

andrewazores added a commit to andrewazores/cryostat3 that referenced this pull request


          Reapply "build(deps): upgrade quarkus to 3.2.9 (cryostatio#352)"

15ef07d

This reverts commit d4dfe75.

andrewazores added a commit to andrewazores/cryostat3 that referenced this pull request


          Revert "build(deps): upgrade quarkus to 3.2.9 (cryostatio#352)"

a91cbbf

This reverts commit bbbbf38.

andrewazores added a commit to andrewazores/cryostat3 that referenced this pull request


          Reapply "build(deps): upgrade quarkus to 3.2.9 (cryostatio#352)"

dd34820

This reverts commit d4dfe75.

andrewazores added a commit to andrewazores/cryostat3 that referenced this pull request


          Revert "build(deps): upgrade quarkus to 3.2.9 (cryostatio#352)"

bc494c9

This reverts commit bbbbf38.

andrewazores added a commit to andrewazores/cryostat3 that referenced this pull request


          Reapply "build(deps): upgrade quarkus to 3.2.9 (cryostatio#352)"

ed89d2a

This reverts commit d4dfe75.

andrewazores added a commit to andrewazores/cryostat3 that referenced this pull request


          Revert "build(deps): upgrade quarkus to 3.2.9 (cryostatio#352)"

e29d0d1

This reverts commit bbbbf38.

andrewazores added a commit to andrewazores/cryostat3 that referenced this pull request


          Reapply "build(deps): upgrade quarkus to 3.2.9 (cryostatio#352)"

This reverts commit d4dfe75.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build chore dependencies safe-to-test