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

Enabling JVMTI tests for ibm jck8 #3703

Merged
merged 3 commits into from
May 27, 2022
Merged

Enabling JVMTI tests for ibm jck8 #3703

merged 3 commits into from
May 27, 2022

Conversation

sobirada
Copy link
Contributor

@sobirada sobirada commented May 25, 2022

This PR enables JVMTI tests on all 64bit platforms for IBM JCK8.

Fixes: runtimes/backlog/issues/607

Signed off by : Soumya Biradar sobirada@in.ibm.com

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.

Given the description in the PR, the changes in ProblemList_openjdk11.txt are not relevant and should be removed from this PR (as that file is related to excluding testcases from the openjdk suite for hotspot implementations.

@llxia
Copy link
Contributor

llxia commented May 25, 2022

Also, the aqaTestPipeline.groovy change is not relevant and should be removed. The change in aqaTestPipeline.groovy was already committed into master dc686eb

@sobirada
Copy link
Contributor Author

@ShelleyLambert @llxia Can you please review it now, earlier had some conflicts with origin

<platform>.*zos.*</platform>
<impl>ibm</impl>
<version>8</version>
</disable>
Copy link
Contributor

Choose a reason for hiding this comment

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

Instead of adding a new <disable>, please update line 99 to <version>8+</version>

@@ -96,14 +96,9 @@
<comment>Temporarily disabled on z/OS for backlog/issues/670</comment>
<platform>.*zos.*</platform>
<impl>ibm</impl>
<version>8+</version>
<version>11</version>
Copy link
Contributor

Choose a reason for hiding this comment

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

<version>11</version> should be removed

Copy link
Contributor

@llxia llxia 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 cb71ac7 into adoptium:master May 27, 2022
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