Skip to content

Added the Liberty tests for jfr#23388

Draft
annaibm wants to merge 3 commits intoeclipse-openj9:masterfrom
annaibm:addJFR_OpenLiberty
Draft

Added the Liberty tests for jfr#23388
annaibm wants to merge 3 commits intoeclipse-openj9:masterfrom
annaibm:addJFR_OpenLiberty

Conversation

@annaibm
Copy link
Contributor

@annaibm annaibm commented Feb 23, 2026

Update the jfr_liberty_tests.xml file to include the new JFR command line tests.

Copy link
Contributor

@tajila tajila left a comment

Choose a reason for hiding this comment

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

I dont think the FindJFRFile and VerifyJfrRecording classes are need as we can just use the existing cmdline tester result parser to verify just as how its currently done.

@annaibm
Copy link
Contributor Author

annaibm commented Mar 2, 2026

Thanks @tajila , Sure, I will review the current workflow and update the changes to use the existing cmdline tester parser.

@annaibm annaibm force-pushed the addJFR_OpenLiberty branch 2 times, most recently from f0db9ad to cd5e860 Compare March 2, 2026 18:30
@annaibm
Copy link
Contributor Author

annaibm commented Mar 2, 2026

@annaibm annaibm requested a review from tajila March 2, 2026 20:20
@tajila
Copy link
Contributor

tajila commented Mar 9, 2026

jenkins test sanity.functional xlinux jdk17

@annaibm
Copy link
Contributor Author

annaibm commented Mar 9, 2026

The Personal build failure is due to the TKG dependency update (adoptium/TKG#811) not being merged yet. Once that's in, re-triggering should pass.

@annaibm annaibm force-pushed the addJFR_OpenLiberty branch 2 times, most recently from 0c0eb0d to cab8156 Compare March 10, 2026 19:33
Update the jfr_liberty_tests.xml file to include the new JFR command line tests.

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

 Signed-off-by: Anna Babu Palathingal <anna.bp@ibm.com>
@annaibm annaibm force-pushed the addJFR_OpenLiberty branch from cf48836 to 121b562 Compare March 10, 2026 19:41
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.

2 participants