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

Add 21 and 22 as supported versions in external tests #4950

Merged
merged 1 commit into from
Jan 9, 2024

Conversation

llxia
Copy link
Contributor

@llxia llxia commented Jan 9, 2024

related: #4949

related: adoptium#4949
Signed-off-by: Lan Xia <Lan_Xia@ca.ibm.com>
@llxia
Copy link
Contributor Author

llxia commented Jan 9, 2024

Grinder
build_image passed. The job failed at a later stage for something else that is not related to this PR.

00:01:12.822  build_image:
00:01:12.823       [echo] Executing external.sh --build --dir criu-ubi-portable-checkpoint --tag nightly --version 21 --impl openj9 --base_docker_registry_dir 'ubi8-with-criu/linux_ppc-64_le-ubi8-criu' --docker_registry_url docker-na.artifactory.swg-devops.com/sys-rt-docker-local --docker_args -v /home/jenkins/workspace/Grinder/jdkbinary/j2sdk-image:/opt/java/openjdk 
00:01:12.823       [exec] The test here is criu-ubi-portable-checkpoint
00:01:12.823       [exec] The directory in the external.sh is criu-ubi-portable-checkpoint
00:01:12.823       [exec] DOCKERIMAGE_TAG nightly has been recognized.
00:01:12.823       [exec] build_image.sh criu-ubi-portable-checkpoint 21 openj9 ubi jdk full linux_x86-64 ubi8-with-criu/linux_ppc-64_le-ubi8-criu 0 
00:01:12.823       [exec] /home/jenkins/workspace/Grinder/aqa-tests/external
00:01:12.823       [exec] 
00:01:12.823       [exec] Writing /home/jenkins/workspace/Grinder/jvmtest/external/criu-ubi-portable-checkpoint/dockerfile/21/jdk/ubi/Dockerfile.openj9.full ... done
00:01:12.823       [exec] 
00:01:12.823       [exec] The test in the build_image() function is criu-ubi-portable-checkpoint

@llxia llxia marked this pull request as ready for review January 9, 2024 14:25
Copy link
Contributor

@JasonFengJ9 JasonFengJ9 left a comment

Choose a reason for hiding this comment

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

LGTM

@smlambert smlambert merged commit 37b48e3 into adoptium:master Jan 9, 2024
3 checks passed
sophia-guo pushed a commit to sophia-guo/openjdk-tests that referenced this pull request Jan 11, 2024
smlambert added a commit that referenced this pull request Jan 11, 2024
* Use all cores for jtreg instead of halving (#4933)

Signed-off-by: Stewart X Addison <sxa@redhat.com>

* Enable headless mode on RISC-V (#4935)

* Exclude java/lang/StringBuilder/HugeCapacity.java on OpenJ9 zlinux (#4940)

It's already excluded on jdk17+ for other reasons.

Issue eclipse-openj9/openj9#17323

Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>

* Move custom targets to special level (#4941)

Signed-off-by: Shelley Lambert <slambert@gmail.com>

* Add 21 and 22 as supported versions in external tests (#4950)

related: #4949

Signed-off-by: Lan Xia <Lan_Xia@ca.ibm.com>

* cherry-pick

e37ba10 c5e7de8 0d7c837 76695ca 37b48e3 a45c03c

Signed-off-by: Sophia Guo <sophia.gwf@gmail.com>

---------

Signed-off-by: Stewart X Addison <sxa@redhat.com>
Signed-off-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
Signed-off-by: Shelley Lambert <slambert@gmail.com>
Signed-off-by: Lan Xia <Lan_Xia@ca.ibm.com>
Signed-off-by: Sophia Guo <sophia.gwf@gmail.com>
Co-authored-by: Stewart X Addison <6487691+sxa@users.noreply.github.com>
Co-authored-by: Ludovic Henry <git@ludovic.dev>
Co-authored-by: Peter Shipton <Peter_Shipton@ca.ibm.com>
Co-authored-by: Shelley Lambert <slambert@gmail.com>
Co-authored-by: Lan Xia <Lan_Xia@ca.ibm.com>
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.

3 participants