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

Fix testSCCacheManagement failure on ibm 11 zos #10368

Merged
merged 1 commit into from
Dec 7, 2020

Conversation

LongyuZhang
Copy link
Contributor

@LongyuZhang LongyuZhang commented Aug 11, 2020

Signed-off-by: Longyu Zhang longyu.zhang@ibm.com

@LongyuZhang
Copy link
Contributor Author

@hangshao0 @llxia Could you review it? Thanks.

@LongyuZhang
Copy link
Contributor Author

LongyuZhang commented Aug 11, 2020

Test link in Internal Grinder:
ibm: zos(9592)
openj9: xlinux(9593), plinux(9594), zlinux(9595), aix(9596), mac(9597), win(9609)

@@ -164,14 +164,18 @@ public static void main(String args[]) {
}
for(String cacheName: nonpersistentList) {
if (cacheName.indexOf("groupaccess") != -1) {
runSimpleJavaProgramWithNonPersistentCache(cacheName, "groupAccess");
if (isOpenJ9()) {
Copy link
Contributor

Choose a reason for hiding this comment

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

This check affects all platforms, but the original issue complains only about z/OS.

@LongyuZhang LongyuZhang marked this pull request as draft August 11, 2020 19:33
@LongyuZhang
Copy link
Contributor Author

This PR has been temporarily changed to Draft mode, since the test failure may be caused by JDK build error, opened another issue on on Internal GitHub runtimes/openj9-openjdk-jdk11-zos/issues/360

- fix cache location change because ibm 11+ use -DOPENJ9_BUILD setting

[ci skip]
Signed-off-by: Longyu Zhang <longyu.zhang@ibm.com>
@LongyuZhang LongyuZhang marked this pull request as ready for review December 5, 2020 02:07
@LongyuZhang
Copy link
Contributor Author

LongyuZhang commented Dec 5, 2020

Test Links:

vmfarm: 461152

jdk 11:

  • j9: aix: 12446 xlinux: 12447 win: 12449
  • ibm: zos: 12463

jdk 8:

  • j9 : aix: 12450 xlinux: 12451 win: 12453
  • ibm: zos: 12464 xlinux: 12444 win: 12445

@pshipton
Copy link
Member

pshipton commented Dec 7, 2020

lgtm. @hangshao0 pls review.

@pshipton pshipton merged commit 2aeffa1 into eclipse-openj9:master Dec 7, 2020
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.

4 participants