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

RISC-V: Disable some known failing tests #4977

Merged
merged 1 commit into from
Jan 17, 2024

Conversation

luhenry
Copy link
Contributor

@luhenry luhenry commented Jan 16, 2024

The tests are broken and should be disabled until fixed.

Based on results from https://ci.adoptium.net/job/build-scripts/job/jobs/job/evaluation/job/jobs/job/jdk21u/job/jdk21u-evaluation-linux-riscv64-temurin/112

Relates to #4976

The tests are broken and should be disabled until fixed.

Relates to adoptium#4976
@luhenry luhenry changed the title RISC-V: Disable known failing system tests RISC-V: Disable known failing tests Jan 16, 2024
@luhenry luhenry changed the title RISC-V: Disable known failing tests RISC-V: Disable some known failing tests Jan 16, 2024
@luhenry luhenry marked this pull request as draft January 16, 2024 17:51
@luhenry luhenry marked this pull request as ready for review January 17, 2024 07:58
Copy link
Contributor

@smlambert smlambert left a comment

Choose a reason for hiding this comment

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

Tried to run Grinders 8519 and 8520 as a little check, but hit issues.

That being said, the changes are straight-forward and look fine. Approving.

@luhenry
Copy link
Contributor Author

luhenry commented Jan 17, 2024

@smlambert I reran it with https://ci.adoptium.net/job/Grinder/8537/console. My fork is on luhenry/adoptium-aqa-tests, which is why your run didn't succeed.

@sophia-guo sophia-guo merged commit 7575985 into adoptium:master Jan 17, 2024
18 of 27 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