Skip to content

Commit

Permalink
application_api,lease: skip some tests under deadlock
Browse files Browse the repository at this point in the history
These tests are all likely to time out.

Epic: CRDB-8308
Release note: None
  • Loading branch information
rickystewart committed Dec 11, 2023
1 parent 4bdde4c commit f9b20ae
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 0 deletions.
2 changes: 2 additions & 0 deletions pkg/server/application_api/schema_inspection_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -480,6 +480,8 @@ func TestAdminAPITableStats(t *testing.T) {
defer leaktest.AfterTest(t)()
defer log.Scope(t).Close(t)

skip.UnderDeadlockWithIssue(t, 115914, "likely to time out")

skip.UnderStress(t, "flaky under stress #107156")
skip.UnderRace(t, "flaky under race #107156")

Expand Down
3 changes: 3 additions & 0 deletions pkg/server/application_api/stats_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -30,6 +30,7 @@ import (
"github.com/cockroachdb/cockroach/pkg/testutils"
"github.com/cockroachdb/cockroach/pkg/testutils/diagutils"
"github.com/cockroachdb/cockroach/pkg/testutils/serverutils"
"github.com/cockroachdb/cockroach/pkg/testutils/skip"
"github.com/cockroachdb/cockroach/pkg/testutils/sqlutils"
"github.com/cockroachdb/cockroach/pkg/util/leaktest"
"github.com/cockroachdb/cockroach/pkg/util/log"
Expand All @@ -41,6 +42,8 @@ func TestTelemetrySQLStatsIndependence(t *testing.T) {
defer leaktest.AfterTest(t)()
defer log.Scope(t).Close(t)

skip.UnderDeadlockWithIssue(t, 115914, "likely to time out")

ctx := context.Background()
var params base.TestServerArgs
params.Knobs.SQLStatsKnobs = sqlstats.CreateTestingKnobs()
Expand Down
3 changes: 3 additions & 0 deletions pkg/server/application_api/stmtdiag_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -24,6 +24,7 @@ import (
"github.com/cockroachdb/cockroach/pkg/sql/sessiondata"
"github.com/cockroachdb/cockroach/pkg/testutils"
"github.com/cockroachdb/cockroach/pkg/testutils/serverutils"
"github.com/cockroachdb/cockroach/pkg/testutils/skip"
"github.com/cockroachdb/cockroach/pkg/testutils/sqlutils"
"github.com/cockroachdb/cockroach/pkg/util/leaktest"
"github.com/cockroachdb/cockroach/pkg/util/log"
Expand Down Expand Up @@ -231,6 +232,8 @@ func TestStatementDiagnosticsDoesNotReturnExpiredRequests(t *testing.T) {
defer leaktest.AfterTest(t)()
defer log.Scope(t).Close(t)

skip.UnderDeadlockWithIssue(t, 115914, "likely to time out")

s, sqlDB, _ := serverutils.StartServer(t, base.TestServerArgs{})
defer s.Stopper().Stop(context.Background())
db := sqlutils.MakeSQLRunner(sqlDB)
Expand Down
2 changes: 2 additions & 0 deletions pkg/sql/catalog/lease/lease_test.go
Original file line number Diff line number Diff line change
Expand Up @@ -1828,6 +1828,8 @@ func TestLeaseRenewedPeriodically(testingT *testing.T) {
defer leaktest.AfterTest(testingT)()
defer log.Scope(testingT).Close(testingT)

skip.UnderDeadlockWithIssue(testingT, 115910, "likely to time out")

ctx := context.Background()

var mu syncutil.Mutex
Expand Down

0 comments on commit f9b20ae

Please sign in to comment.