Skip to content

disable failing testcases related to Loom - #15237

Merged
llxia merged 1 commit into
eclipse-openj9:masterfrom
fengxue-IS:exclude19
Jun 8, 2022
Merged

disable failing testcases related to Loom#15237
llxia merged 1 commit into
eclipse-openj9:masterfrom
fengxue-IS:exclude19

Conversation

@fengxue-IS

Copy link
Copy Markdown
Contributor

Related: #15152

Signed-off-by: Jack Lu Jack.S.Lu@ibm.com

@JasonFengJ9

Copy link
Copy Markdown
Member

01:54:32 FAILURE - Copyright date in file: 'test/functional/cmdLineTests/jvmtitests/jvmtitests_excludes_19.xml' appears to be incorrect

Are we looking for more excludes to be added for a green sanity.functional?

@fengxue-IS

Copy link
Copy Markdown
Contributor Author

Are we looking for more excludes to be added for a green sanity.functional?

The only test failure in sanity.functional which I did not exclude is JDWP error which I am looking into fixing by adding the vmhook/event mapping for new JVMTI capability.

My build last night failed due to machine issue, will try to rerun the personal build to confirm

@fengxue-IS
fengxue-IS force-pushed the exclude19 branch 2 times, most recently from 3c3fa5a to 62f5691 Compare June 7, 2022 14:53
@fengxue-IS

Copy link
Copy Markdown
Contributor Author

Personal build still in progress: Test_grinder/job/Grinder/24577/
will update PR once it completes

list of tests failed due to JDWP requiring new capability can_support_virtual_threads for reference:

cmdLineTester_jvmtitests_debug_1/2/3/4/5/6/7/8/9/10/11
cmdLineTester_jvmtitests_debug_openj9_none_SCC_1/2/3/4/5/6/7/8/9/10/11
decompileAtMethodResolve_0
cmdLineTester_jython_1/3/4/6
TestFlushReflectionCache_2

@tajila

tajila commented Jun 7, 2022

Copy link
Copy Markdown
Contributor

Okay, you can exlclude those as well, @babsingh is looking into JVMTI failures

@fengxue-IS
fengxue-IS force-pushed the exclude19 branch 2 times, most recently from 2d2db9c to 824bf35 Compare June 8, 2022 05:58
Signed-off-by: Jack Lu <Jack.S.Lu@ibm.com>
<version>19+</version>
<variation>Mode607-OSR</variation>
</disable>
</disables>

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@llxia is there a regex to disable test modes?

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.

No, it is not supported atm. @renfeiw maybe we should add support for this?

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.

Opened an issue for this: adoptium/TKG#329 Will work on it later on.

@fengxue-IS
fengxue-IS marked this pull request as ready for review June 8, 2022 15:19
@fengxue-IS

Copy link
Copy Markdown
Contributor Author

personal build passed on xlinux

@tajila

tajila commented Jun 8, 2022

Copy link
Copy Markdown
Contributor

jenkins test sanity amac jdknext

@tajila

tajila commented Jun 8, 2022

Copy link
Copy Markdown
Contributor

@llxia Please review these changes

@llxia llxia 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.

LGTM

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.

5 participants