Skip to content
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.

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

Exclude new GraalVM dependencies that are part of GraalVM/Mandrel #35873

Merged
merged 1 commit into from
Sep 13, 2023

Conversation

zakkak
Copy link
Contributor

@zakkak zakkak commented Sep 12, 2023

oracle/graal#7171 splits graal-sdk in 4 new
modules/artifacts. As a result starting with GraalVM for JDK 21 (23.1)
we need to remove those as well. Note that it's OK to have them marked
for removal even when using GraalVM < 23.1 despite them not actually
being present.

Closes #35872

Marked as draft till CI run https://github.com/graalvm/mandrel/actions/runs/6158348245 completes

Update: CI run looks good

@zakkak zakkak marked this pull request as ready for review September 12, 2023 13:17
@zakkak zakkak requested a review from geoand September 12, 2023 13:17
@zakkak
Copy link
Contributor Author

zakkak commented Sep 12, 2023

@jerboaa please review

Copy link
Contributor

@jerboaa jerboaa left a comment

Choose a reason for hiding this comment

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

Almost. We need to account for the fact that polyglot is part of the dependencies too.

@zakkak zakkak force-pushed the 2023-09-12-rm-more-arteftacts branch from 78031de to f1476be Compare September 12, 2023 14:18
oracle/graal#7171 splits graal-sdk in 4 new
modules/artifacts. As a result starting with GraalVM for JDK 21 (23.1)
we need to remove those as well. Note that it's OK to have them marked
for removal even when using GraalVM < 23.1 despite them not actually
being present.

Closes quarkusio#35872
@zakkak zakkak force-pushed the 2023-09-12-rm-more-arteftacts branch from f1476be to 9838aae Compare September 12, 2023 14:25
@zakkak zakkak added the triage/waiting-for-ci Ready to merge when CI successfully finishes label Sep 12, 2023
@quarkus-bot

This comment has been minimized.

@quarkus-bot

This comment has been minimized.

@zakkak
Copy link
Contributor Author

zakkak commented Sep 13, 2023

Native Tests - Virtual Thread - Main times out, but I fail to see how it could be caused by this PR.

@zakkak zakkak removed the triage/waiting-for-ci Ready to merge when CI successfully finishes label Sep 13, 2023
@quarkus-bot
Copy link

quarkus-bot bot commented Sep 13, 2023

Failing Jobs - Building 9838aae

Status Name Step Failures Logs Raw logs Build scan
JVM Tests - JDK 11 Build ⚠️ Check → Logs Raw logs
✔️ JVM Tests - JDK 17
JVM Tests - JDK 17 Windows Build ⚠️ Check → Logs Raw logs
✔️ JVM Tests - JDK 20

@zakkak zakkak merged commit 08961a5 into quarkusio:main Sep 13, 2023
@zakkak zakkak deleted the 2023-09-12-rm-more-arteftacts branch September 13, 2023 07:30
@quarkus-bot quarkus-bot bot added this to the 3.5 - main milestone Sep 13, 2023
zakkak added a commit to zakkak/quarkus that referenced this pull request Nov 30, 2023
When quarkusio#35873 was merged
`org.graalvm.polyglot:polyglot` was part of the GraalVM dev
builds/releases, but was later completely removed from them.
holly-cummins pushed a commit to holly-cummins/quarkus that referenced this pull request Feb 8, 2024
When quarkusio#35873 was merged
`org.graalvm.polyglot:polyglot` was part of the GraalVM dev
builds/releases, but was later completely removed from them.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[GraalVM 23.1+] Several quarkus native tests fail with locally installed GraalVM 23.1+ maven artefacts
3 participants