Skip to content

Exclude JFR Liberty test on z/OS - #23600

Merged
keithc-ca merged 1 commit into
eclipse-openj9:masterfrom
annaibm:checkZOS
Mar 31, 2026
Merged

Exclude JFR Liberty test on z/OS#23600
keithc-ca merged 1 commit into
eclipse-openj9:masterfrom
annaibm:checkZOS

Conversation

@annaibm

@annaibm annaibm commented Mar 28, 2026

Copy link
Copy Markdown
Contributor
  • Skip Liberty runtime checksum verification on z/OS temporarily until checksum issue is resolved
  • Add z/OS exclusion to JFR Liberty tests
  • Update failure condition

related: https://github.ibm.com/runtimes/automation/issues/874
Signed-off-by: Anna Babu Palathingal anna.bp@ibm.com

@annaibm

annaibm commented Mar 28, 2026

Copy link
Copy Markdown
Contributor Author

After #23388 was merged, z/OS builds started failing:
https://hyc-runtimes-jenkins.swg-devops.com/job/Pipeline_Build_Test_JDK25_s390x_zos/133

Root cause:

  • Liberty runtime checksum verification fails on z/OS due to
    EDC5122I Input/output error (z/OS file encoding issue)
  • JFR Liberty tests is excluded for now on z/OS, until a z/OS-compatible
    Liberty runtime source is identified and tested to run successfully

Grinder Testing

@annaibm annaibm changed the title Fix JFR Liberty test failures on z/OS Exclude JFR Liberty test on z/OS Mar 28, 2026
@annaibm
annaibm marked this pull request as ready for review March 28, 2026 20:10
@pshipton
pshipton requested a review from tajila March 30, 2026 02:41
@pshipton

Copy link
Copy Markdown
Member

I see a number of "Made with Bob" comments are being removed. Is any of the code created using AI? Pls see https://github.com/eclipse-openj9/openj9/blob/master/CONTRIBUTING.md#generative-artificial-intelligence-usage-guidelines

Comment thread test/functional/cmdLineTests/jfr/build.xml
Comment thread test/functional/cmdLineTests/jfr/jfr_liberty_tests.xml Outdated
Comment thread test/functional/cmdLineTests/jfr/src/org/openj9/test/LibertyJvmOptions.java Outdated
@annaibm
annaibm force-pushed the checkZOS branch 2 times, most recently from 5d6f17a to 240082c Compare March 30, 2026 20:14
Comment thread test/functional/cmdLineTests/jfr/playlist.xml Outdated
@annaibm
annaibm force-pushed the checkZOS branch 2 times, most recently from a873857 to ac160d7 Compare March 30, 2026 21:31

@keithc-ca keithc-ca left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

In the commit message, I think the Co-authored line should be:

Co-authored by: IBM Bob

adding the colon and "IBM".

@keithc-ca

Copy link
Copy Markdown
Contributor

Sorry, but I think the line in commit message is also missing a hyphen:

Co-authored-by: IBM Bob

and the reference in jfr_liberty_tests.xml should say "IBM Bob".

- Skip Liberty runtime checksum verification on z/OS temporarily
  until checksum issue is resolved
- Add z/OS exclusion to JFR Liberty tests
- Update failure condition

related: https://github.ibm.com/runtimes/automation/issues/874

Co-authored-by: IBM Bob
Signed-off-by: Anna Babu Palathingal <anna.bp@ibm.com>
@keithc-ca
keithc-ca merged commit 6e720af into eclipse-openj9:master Mar 31, 2026
3 checks passed
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