Skip to content

Commit

Permalink
roachtest: skip restore/tpce/32TB/inc-count=400/gce/nodes=15/cpus=16
Browse files Browse the repository at this point in the history
This weekly test is very costly (leads to cross region egress) and is already
run on aws. We can reconsider unskipping it after cockroachdb#111371 is addressed.

Epic: none

Release note: none
  • Loading branch information
msbutler committed Oct 31, 2023
1 parent 8b34755 commit 285fda2
Showing 1 changed file with 5 additions and 0 deletions.
5 changes: 5 additions & 0 deletions pkg/cmd/roachtest/tests/restore.go
Original file line number Diff line number Diff line change
Expand Up @@ -420,6 +420,7 @@ func registerRestore(r registry.Registry) {
`SET CLUSTER SETTING backup.restore_span.target_size = '0'`,
},
restoreUptoIncremental: 400,
skip: "a recent gcp pricing policy makes this test very expensive. unskip after #111371 is addressed",
},
{
// A teeny weeny 15GB restore that could be used to bisect scale agnostic perf regressions.
Expand Down Expand Up @@ -453,6 +454,7 @@ func registerRestore(r registry.Registry) {
CompatibleClouds: sp.clouds,
Suites: sp.suites,
Tags: sp.tags,
Skip: sp.skip,
Run: func(ctx context.Context, t test.Test, c cluster.Cluster) {

rd := makeRestoreDriver(t, c, sp)
Expand Down Expand Up @@ -825,6 +827,9 @@ type restoreSpecs struct {

testName string
setUpStmts []string

// skip, if non-empty, skips the test with the given reason.
skip string
}

func (sp *restoreSpecs) initTestName() {
Expand Down

0 comments on commit 285fda2

Please sign in to comment.