Skip to content

Commit

Permalink
Browse files Browse the repository at this point in the history
105758: sql: skip multiregional tests again r=chengxiong-ruan a=chengxiong-ruan

Unfortunately, it's still flaky:
#105346 (comment)

Informs: #98020

Release note: None

105893: persistedsqlstats: skip TestSQLStatsPersistedLimitReached r=zachlite a=zachlite

Flakes in Bazel essential CI.
Reverts #105770

Informs #105846, #97488
Epic: none
Release note: None

Co-authored-by: Chengxiong Ruan <[email protected]>
Co-authored-by: zachlite <[email protected]>
  • Loading branch information
3 people committed Jun 30, 2023
3 parents 66c9f93 + d517243 + 0e46af9 commit 947a636
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 1 deletion.
3 changes: 3 additions & 0 deletions pkg/ccl/multiregionccl/testdata/secondary_region
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
skip issue-num=98020
----

new-cluster localities=us-east-1,us-east-1,us-west-1,us-west-1,us-central-1,us-central-1,us-central-1,eu-west-1,eu-west-1,eu-west-1
----

Expand Down
2 changes: 1 addition & 1 deletion pkg/sql/sqlstats/persistedsqlstats/flush_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -459,7 +459,7 @@ func TestSQLStatsGatewayNodeSetting(t *testing.T) {
}

func TestSQLStatsPersistedLimitReached(t *testing.T) {
skip.UnderStressWithIssue(t, 97488)
skip.WithIssue(t, 97488)
defer leaktest.AfterTest(t)()
defer log.Scope(t).Close(t)

Expand Down

0 comments on commit 947a636

Please sign in to comment.