Skip to content

Conversation

@p-schlickmann
Copy link
Contributor

@p-schlickmann p-schlickmann commented Oct 20, 2025

Second attempt to reduce the flakiness of processes_lifecycle_test.rb.

I started using the same logic from #L65-L85.

According to the comment, we should actually check for :ready rather than :claimed.

Issue #602

@p-schlickmann p-schlickmann changed the title Attempt reduce flakiness of processes_lifecycle_test.rb Second attempt reduce flakiness of processes_lifecycle_test.rb Oct 20, 2025
@p-schlickmann p-schlickmann force-pushed the flaky-processes-lifecycle-test branch from c1e5f34 to 613a110 Compare October 20, 2025 02:12
@p-schlickmann p-schlickmann force-pushed the flaky-processes-lifecycle-test branch from 0144b86 to 0d2f05f Compare October 20, 2025 03:03
@p-schlickmann
Copy link
Contributor Author

@rosa Seems to be working now! The failed test is unrelated to processes_lifecycle_test.rb 🙏🏻

Copy link
Member

@rosa rosa left a comment

Choose a reason for hiding this comment

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

Nice! Thank you!

@rosa rosa merged commit c51ae60 into rails:main Oct 21, 2025
42 of 43 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.

2 participants