Skip to content

Commit 84b4b91

Browse files
craig[bot]spilchen
andcommitted
Merge #154543
154543: workload: remove SCRUB test r=spilchen a=spilchen The existing SCRUB test was considered for conversion to use INSPECT, but this test is outdated and lacks metric exports. Our goal is to build a roachtest that can measure throughput and the impact on foreground traffic. Both of which will be addressed in #154457. We also plan to add INSPECT coverage to multiple existing roachtests (#154049), rendering this standalone test unnecessary. Additionally, SCRUB itself is scheduled for removal (#148274), so maintaining or converting this test is not worthwhile. Release note: none Epic: CRDB-30356 Closes #149810 Co-authored-by: Matt Spilchen <[email protected]>
2 parents 2e14011 + 924bf19 commit 84b4b91

File tree

3 files changed

+0
-100
lines changed

3 files changed

+0
-100
lines changed

pkg/cmd/roachtest/tests/BUILD.bazel

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -183,7 +183,6 @@ go_library(
183183
"s3_minio.go",
184184
"schemachange.go",
185185
"schemachange_random_load.go",
186-
"scrub.go",
187186
"secondary_indexes.go",
188187
"sequelize.go",
189188
"slow_drain.go",

pkg/cmd/roachtest/tests/registry.go

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -154,8 +154,6 @@ func RegisterTests(r registry.Registry) {
154154
registerSchemaChangeRandomLoad(r)
155155
registerLargeSchemaBackupRestores(r)
156156
registerLargeSchemaBenchmarks(r)
157-
registerScrubAllChecksTPCC(r)
158-
registerScrubIndexOnlyTPCC(r)
159157
registerSecondaryIndexesMultiVersionCluster(r)
160158
registerSequelize(r)
161159
registerSlowDrain(r)

pkg/cmd/roachtest/tests/scrub.go

Lines changed: 0 additions & 97 deletions
This file was deleted.

0 commit comments

Comments
 (0)