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.

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

Disable JFR tests on AIX. #2532

Merged
merged 1 commit into from
Apr 27, 2021
Merged

Disable JFR tests on AIX. #2532

merged 1 commit into from
Apr 27, 2021

Conversation

sophia-guo
Copy link
Contributor

@sophia-guo sophia-guo commented Apr 21, 2021 •
edited
Loading

Disable JFR tests on AIX.
JFR tests don't run on AIX with JDK11+, which are actually disabled by require property @requires vm.hasJFR. As no test selected test jobs still fail. Will just disable for all version.

https://ci.adoptopenjdk.net/job/Test_openjdk11_hs_extended.openjdk_ppc64_aix/22/console

01:47:30  Test results: no tests selected
01:47:39  Report written to /home/jenkins/workspace/Test_openjdk11_hs_extended.openjdk_ppc64_aix/jvmtest/openjdk/report/html/report.html
01:47:39  Results written to /home/jenkins/workspace/Test_openjdk11_hs_extended.openjdk_ppc64_aix/openjdk-tests/TKG/output_16189619473542/jdk_jfr_1/work
01:47:39  
01:47:39  jdk_jfr_1_FAILED

Correct two disable comment link.

Close #2256
Close #2218

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

@sophia-guo
Disable JFR tests on AIX.
5cad7fa 
Disable JFR tests on AIX.
Correct two disable comment link.

Signed-off-by: Sophia Guo <sophia.gwf@gmail.com>
smlambert
smlambert approved these changes Apr 21, 2021
View reviewed changes
Copy link
Contributor

@smlambert smlambert left a comment

Choose a reason for hiding this comment

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

thanks @sophia-guo !

@karianna karianna added this to the April 2021 milestone Apr 23, 2021
@smlambert smlambert requested a review from lumpfish April 27, 2021 00:54
lumpfish
lumpfish approved these changes Apr 27, 2021
View reviewed changes
Copy link
Contributor

@lumpfish lumpfish left a comment

Choose a reason for hiding this comment

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

Looks good

@lumpfish lumpfish merged commit 1a02140 into adoptium:master Apr 27, 2021
@sophia-guo sophia-guo deleted the JFR branch May 13, 2022 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@smlambert smlambert smlambert approved these changes

@lumpfish lumpfish lumpfish approved these changes

Assignees
No one assigned
Labels
enhancement
Projects
None yet
Milestone
April 2021
4 participants
@sophia-guo @smlambert @lumpfish @karianna