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

Clean up Excludes Test List jdk21- 1st pass #5903

Merged
merged 1 commit into from
Jan 24, 2025

Conversation

dennycjacob
Copy link
Contributor

@dennycjacob dennycjacob commented Jan 23, 2025

@dennycjacob
Copy link
Contributor Author

dennycjacob commented Jan 23, 2025

Grinder Tests:

  1. java/lang/ProcessBuilder/Basic.java

https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/46851/ - aix-ppc64

  1. java/lang/System/LoggerFinder/modules/JDKLoggerForImageTest.java

https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/46866/ -aarch64_mac
https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/46867/ -x86-64_mac

  1. java/lang/System/LoggerFinder/modules/LoggerInImageTest.java

https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/46872/ -aarch64_mac
https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/46873/ -x86-64_mac

  1. java/lang/System/LoggerFinder/modules/NamedLoggerForImageTest.java

https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/46875/ -aarch64_mac
https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/46876/ -x86-64_mac

  1. java/lang/System/LoggerFinder/modules/UnnamedLoggerForImageTest.java

https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/46878/ -x86-64_mac
https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/46879/ -aarch64_mac

  1. jdk/modules/incubator/ImageModules.java - mac

https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/46883/ -x86-64_mac
https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/46882/ -aarch64_mac

  1. java/net/ipv6tests/TcpTest.java -mac
    https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/46898/ -aarch64_mac
    https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/46898/ -x86-64_mac

  2. java/net/ipv6tests/UdpTest.java
    https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/46901/ -aarch64_mac
    https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/46902/ -x86-64_mac
    https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/46904/ -aarch64_linux
    https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/46905/ -ppc64le_linux
    https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/46906/ -s390x_linux
    https://hyc-runtimes-jenkins.swg-devops.com/view/Test_grinder/job/Grinder/46908/ -x86-64_linux

  3. java/net/MulticastSocket/Promiscuous.java
    Networking failures on Marist/s390x systems infrastructure#699 linux-s390x,macosx-all,aix-all

FAIL:
https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/46935 - s390x_linux
https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/46938 - ppc64_aix

PASS:
https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/46936- x86-64_mac
https://hyc-runtimes-jenkins.swg-devops.com/job/Grinder/46937 - aarch64_mac

@llxia
Copy link
Contributor

llxia commented Jan 23, 2025

Thanks @dennycjacob

  • java/net/ipv6tests/UdpTest.java - please test it on all linux, not just mac.
  • java/net/MulticastSocket/Promiscuous.java - we can enable it on macosx-all. Please open an issue in runtimes/backlog and describe the issue. Further analysis is needed.

@dennycjacob
Copy link
Contributor Author

Created the issue here https://github.ibm.com/runtimes/backlog/issues/1591
The linux tests have been added.

java/net/MulticastSocket/B6427403.java https://github.ibm.com/runtimes/backlog/issues/715 macosx-all,aix-all
java/net/MulticastSocket/MulticastAddresses.java https://github.ibm.com/runtimes/backlog/issues/715 macosx-all
java/net/MulticastSocket/NoSetNetworkInterface.java https://github.ibm.com/runtimes/backlog/issues/655 windows-all
java/net/MulticastSocket/Promiscuous.java https://github.com/adoptium/infrastructure/issues/699 linux-s390x,macosx-all,aix-all
java/net/MulticastSocket/Promiscuous.java https://github.com/adoptium/infrastructure/issues/699 linux-s390x,aix-all
Copy link
Contributor

Choose a reason for hiding this comment

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

Please update this issue link to the new backlog issue

Copy link
Contributor Author

Choose a reason for hiding this comment

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

added the link to the issue

@dennycjacob dennycjacob marked this pull request as ready for review January 24, 2025 16:29
Update ProblemList_openjdk21-openj9.txt to add the link to a new issue

Added link to https://github.ibm.com/runtimes/backlog/issues/1591 for java/net/MulticastSocket/Promiscuous.java because of failures in linux-s390x and aix-all platforms
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

@llxia llxia requested a review from JasonFengJ9 January 24, 2025 16:41
@llxia
Copy link
Contributor

llxia commented Jan 24, 2025

Just for the record, some tests are also excluded at the test target level (i.e., jdk_net), so re-enable its subtests will not enable them in our nightly/weekly run.

@llxia llxia requested a review from LongyuZhang January 24, 2025 17:04
Copy link
Contributor

@LongyuZhang LongyuZhang left a comment

Choose a reason for hiding this comment

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

LGTM

@LongyuZhang LongyuZhang merged commit 1dedab8 into adoptium:master Jan 24, 2025
3 checks passed
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