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

fix: Comment out tests relating to component iteration until figure out why miri hangs on it #467

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

MaxCWhitehead
Copy link
Collaborator

@MaxCWhitehead MaxCWhitehead commented Sep 17, 2024

Relates to issue: #466

Possibly an issue with #454 ?

All of these tests are hanging in miri locally - will see if CI completes now.

@MaxCWhitehead
Copy link
Collaborator Author

am realizing some them don't hang locally, just take a long time, will revise.

@MaxCWhitehead MaxCWhitehead marked this pull request as draft September 17, 2024 00:49
@MaxCWhitehead
Copy link
Collaborator Author

first run passed with all these commented out after 40min, will incrementally push back in what I can verify locally until find a smoking gun (or find nothing, heh)

@MaxCWhitehead
Copy link
Collaborator Author

MaxCWhitehead commented Sep 17, 2024

Restored some tests I had pass locally and it timed out again on CI here. Going to bump nightly version locally and see if can repro, but probably not an issue with tests, but regression in miri.

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.

1 participant