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

Disable tests for CUDA configurations in CI #1264

Merged
merged 1 commit into from
Jan 29, 2025

Conversation

msimberg
Copy link
Collaborator

We can't yet have #1225, so disable CUDA tests for now and leave the CUDA builds in place in CI.

@msimberg msimberg requested review from rasolca and albestro January 27, 2025 16:08
@msimberg
Copy link
Collaborator Author

I've simply commented out the test stages with CUDA now. Let me know if you'd prefer completely deleting the blocks.

@msimberg msimberg marked this pull request as draft January 27, 2025 16:08
@msimberg
Copy link
Collaborator Author

cscs-ci run

@msimberg
Copy link
Collaborator Author

msimberg commented Jan 28, 2025

There were two timeouts on test_bt_reduction_to_band:

The eigensolver test times out after that as well, but my guess is that it's just a consequence of MPI going into an inconsistent state (the test hangs in the beginning).

I think the test_bt_reduction_to_band tests timing out is still the same old scoped trace issue, though without segfaults in these particular instances. #1258 and #838 will help diagnose and improve, respectively, the situation. We still need pika 0.31.0 to fix the last (known) source of yielding, but those two PRs will already help quite a bit.

Edit: I've restarted the two failed test jobs.

@msimberg msimberg marked this pull request as ready for review January 28, 2025 11:07
@msimberg
Copy link
Collaborator Author

All remaining tests passed after one rerun.

@rasolca rasolca merged commit acb0b83 into eth-cscs:master Jan 29, 2025
5 checks passed
@msimberg msimberg deleted the disable-cuda-tests branch January 29, 2025 15:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

2 participants