Skip to content

release-25.4: roachtest: add flaky npgsql and hibernate tests to ignore lists#163797

Open
rafiss wants to merge 1 commit intocockroachdb:release-25.4from
rafiss:blathers/backport-release-25.4-163783
Open

release-25.4: roachtest: add flaky npgsql and hibernate tests to ignore lists#163797
rafiss wants to merge 1 commit intocockroachdb:release-25.4from
rafiss:blathers/backport-release-25.4-163783

Conversation

@rafiss
Copy link
Collaborator

@rafiss rafiss commented Feb 17, 2026

Backport 1/1 commits from #163783 on behalf of @rafiss.


Add Npgsql.Tests.NotificationTests.WaitAsync_with_timeout to the npgsql ignore list and update hibernate ignore list entries for JtaTransactionJoiningTest.

These tests have been observed to be flaky in the upstream testing suites.

Resolves: #163604
Resolves: #163481

Release note: None


Release justification:

Add Npgsql.Tests.NotificationTests.WaitAsync_with_timeout to the npgsql
ignore list and update hibernate ignore list entries for
JtaTransactionJoiningTest.

These tests have been observed to be flaky in the upstream testing
suites.

Resolves: cockroachdb#163604
Resolves: cockroachdb#163481

Release note: None
@rafiss rafiss force-pushed the blathers/backport-release-25.4-163783 branch from 7270b06 to d344edd Compare February 17, 2026 22:14
@blathers-crl blathers-crl bot added blathers-backport This is a backport that Blathers created automatically. O-robot Originated from a bot. labels Feb 17, 2026
@blathers-crl blathers-crl bot requested a review from fqazi February 17, 2026 22:14
@blathers-crl
Copy link

blathers-crl bot commented Feb 17, 2026

Thanks for opening a backport.

Before merging, please confirm that it falls into one of the following categories (select one):

  • Non-production code changes OR fixes for serious issues. Non-production includes test-only changes, build system changes, etc. Serious issues are defined in the policy as correctness, stability, or security issues, data corruption/loss, significant performance regressions, breaking working and widely used functionality, or an inability to detect and debug production issues.
  • Other approved changes. These changes must be gated behind a disabled-by-default feature flag unless there is a strong justification not to. Reference the approved ENGREQ ticket in the PR body (e.g., "Fixes ENGREQ-123").

Add a brief release justification to the PR description explaining your selection.

Also, confirm that the change does not break backward compatibility and complies with all aspects of the backport policy.

All backports must be reviewed by the TL and EM for the owning area.

@blathers-crl blathers-crl bot added backport Label PR's that are backports to older release branches T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions) labels Feb 17, 2026
@blathers-crl
Copy link

blathers-crl bot commented Feb 17, 2026

✅ PR #163797 is compliant with backport policy

Confidence: high
Backward compatible: true
Explanation: The pull request modifies files that are part of the non-production testing infrastructure specifically for handling flaky tests in CockroachDB's roachtest suite. The file paths pkg/cmd/roachtest/tests/hibernate_blocklist.go and pkg/cmd/roachtest/tests/npgsql_blocklist.go clearly fall under the defined non-production file patterns (listed as pkg/cmd/roachtest/). Since the changes affect only test definitions (ignore lists) and do not alter production code, they are automatically exempt from the standard requirements like addressing a critical bug or being gated behind a feature flag.

ENGREQ Check Failed: Please check one of the backport category checkboxes.

🦉 Hoot! I am a Blathers, a bot for CockroachDB. My owner is dev-inf.

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@blathers-crl blathers-crl bot added the backport-test-only Used to denote the backport has only non-production changes label Feb 17, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport Label PR's that are backports to older release branches backport-test-only Used to denote the backport has only non-production changes blathers-backport This is a backport that Blathers created automatically. O-robot Originated from a bot. T-sql-foundations SQL Foundations Team (formerly SQL Schema + SQL Sessions)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments