Exclude failed dacapo tests #6338
Conversation
karianna
left a comment
There was a problem hiding this comment.
Whitespace needs fixing.
|
Leading whitespace fixed |
| <disable> | ||
| <comment>https://github.com/adoptium/aqa-tests/issues/4858#issuecomment-2968494739</comment> | ||
| </disable> | ||
| </disables> |
There was a problem hiding this comment.
Instead of excluding all cases, we need to exclude per JDK version and platform. For dacapo-luindex, please exclude for JDK8 only (i.e., <version>8</version>).
Please update the rest of the exclusions. Thanks
There was a problem hiding this comment.
Updated exclusions
|
This PR can only be fully tested if adoptium/TKG#710 (comment) is resolved. |
|
I feel this PR as well as #6341 and #6337 are mixed up.
Since we are very close to getting #6337 done, we can skip #6341. I will leave it up to you. |
Reset to current upstream and only added the playlist.xml changes. Fixes: adoptium#6331 Signed-off-by: Matthew Wei <mwei2@andrew.cmu.edu>
|
Let's skip #6341, sorry for the confusion. I have created a PR for adoptium/TKG#710 and cleaned up #6337. |
|
Please use the git keyword to relate to the issue. See https://github.com/adoptium/aqa-tests/blob/master/Contributing.md#submitting-a-contribution-to-aqavit for details. Thanks |
I believe my most recent commit has Fixes:#6331 within the description, is this incorrect? |
|
Yes, it is correct. Please also add it to the PR description. Thanks |
Exclude decapo tests that failed. Using 23.11 minimal (Disabled pmd, fop, jython, h2, eclipse, luindex)
Fixes: #6331