Skip to content

roachtest: add async association tests to activerecord ignore list#168318

Merged
trunk-io[bot] merged 1 commit intocockroachdb:masterfrom
spilchen:gh-164804/260414/0859/fnd/activerecord-flake-fix
Apr 14, 2026
Merged

roachtest: add async association tests to activerecord ignore list#168318
trunk-io[bot] merged 1 commit intocockroachdb:masterfrom
spilchen:gh-164804/260414/0859/fnd/activerecord-flake-fix

Conversation

@spilchen
Copy link
Copy Markdown
Contributor

The AsyncBelongsToAssociationsTest, AsyncHasManyAssociationsTest, and AsyncHasOneAssociationsTest tests all use a wait_for_async_query helper with a 5-second timeout that flakes on CI when CockroachDB query execution exceeds the deadline. These are the same class of flake as the five LoadAsyncTest entries already on the ignore list (added in d5431d9 for #162147), but the association test variants were missed at that time.

Closes: #164804
Epic: none

Release note: None

The `AsyncBelongsToAssociationsTest`, `AsyncHasManyAssociationsTest`, and
`AsyncHasOneAssociationsTest` tests all use a `wait_for_async_query`
helper with a 5-second timeout that flakes on CI when CockroachDB query
execution exceeds the deadline. These are the same class of flake as
the five `LoadAsyncTest` entries already on the ignore list (added in
d5431d9 for cockroachdb#162147), but the association test variants were missed
at that time.

Closes: cockroachdb#164804
Epic: none

Release note: None

Co-Authored-By: roachdev-claude <roachdev-claude-bot@cockroachlabs.com>
@spilchen spilchen self-assigned this Apr 14, 2026
@trunk-io
Copy link
Copy Markdown
Contributor

trunk-io bot commented Apr 14, 2026

😎 Merged successfully - details.

@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

@spilchen spilchen marked this pull request as ready for review April 14, 2026 13:37
@spilchen spilchen requested review from a team and bghal April 14, 2026 13:37
@spilchen spilchen added the backport-all Flags PRs that need to be backported to all supported release branches label Apr 14, 2026
@spilchen
Copy link
Copy Markdown
Contributor Author

TFTR!

/trunk merge

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport-all Flags PRs that need to be backported to all supported release branches target-release-26.3.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

roachtest: activerecord failed

3 participants