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

[CI] Use correct SNAPSHOT version for Quarkus release branches #673

Merged
merged 2 commits into from
Jan 26, 2024

Conversation

@zakkak zakkak added the testing CI and testing related issues and PRs label Jan 24, 2024
@zakkak zakkak requested a review from Karm January 24, 2024 14:09
@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Jan 24, 2024
@zakkak zakkak force-pushed the 2024-01-24-fix-quarkus-snapshots branch from 8c7ddb2 to 1e3c6b3 Compare January 24, 2024 14:09
@zakkak
Copy link
Collaborator Author

zakkak commented Jan 25, 2024

I re-scheduled the CI run since the original was canceled due to a newer Nightly run.

@zakkak
Copy link
Collaborator Author

zakkak commented Jan 25, 2024

Failures are due to oracle#8075 / quarkusio/quarkus#37657

@zakkak
Copy link
Collaborator Author

zakkak commented Jan 25, 2024

Weekly CI was not used on this repository because of:

zakkak/mandrel/.github/workflows/base.yml@2024-01-24-fix-quarkus-snapshots is not allowed to be used in graalvm/mandrel. Reusable workflows in this workflow must be: within a repository owned by graalvm or matching the following: docker/*, graalvm/*, oracle/*, ruby/setup-ruby@*.

But it run in my fork: https://github.com/zakkak/mandrel/actions/runs/7641569142 and indicates that there is a mistake in my bash conditionals.

@zakkak zakkak marked this pull request as draft January 25, 2024 13:34
@zakkak zakkak force-pushed the 2024-01-24-fix-quarkus-snapshots branch from 69e6a5c to e9d51eb Compare January 25, 2024 13:41
@zakkak
Copy link
Collaborator Author

zakkak commented Jan 25, 2024

The PR depends on Karm/mandrel-integration-tests#247, without it the tests fail with:

java.lang.ExceptionInInitializerError
	at org.graalvm.tests.integration.DebugSymbolsTest.<clinit>(DebugSymbolsTest.java:97)
Caused by: java.lang.NumberFormatException: For input string: "3.2.999"
	at org.graalvm.tests.integration.DebugSymbolsTest.<clinit>(DebugSymbolsTest.java:97)

See https://github.com/zakkak/mandrel/actions/runs/7655379970/job/20862301754#step:11:4258

@zakkak
Copy link
Collaborator Author

zakkak commented Jan 26, 2024

The changes work as expected, see https://github.com/zakkak/mandrel/actions/runs/7655379970 now that Karm/mandrel-integration-tests#247 is merged.

@zakkak zakkak marked this pull request as ready for review January 26, 2024 12:30
@zakkak zakkak merged commit 0e71dc5 into graalvm:default Jan 26, 2024
25 of 46 checks passed
@zakkak zakkak deleted the 2024-01-24-fix-quarkus-snapshots branch January 26, 2024 12:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
OCA Verified All contributors have signed the Oracle Contributor Agreement. testing CI and testing related issues and PRs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant