cockroachdb / cockroachdb/cockroach

workload: expand INSPECT validation to more roachtests

Open
#155,704 0 comments 0 reactions 0 assignees View on GitHub
C-enhancement T-sql-foundations
Dominant language
Go
Stars
32.5k
Forks
4.1k
PR merge metrics
PR metrics pending

Description

We recently added INSPECT validation after roachtest runs in https://github.com/cockroachdb/cockroach/pull/155438. For now, this is limited to a small set of test suites to avoid a thundering herd of all tests running INSPECT validation.

This issue tracks expanding coverage by gradually removing entries from `inspectBlocklistRegex` in `pkg/cmd/roachtest/test_runner.go`, increasing the number of roachtests that perform INSPECT validation.

Jira issue: CRDB-55583

Epic CRDB-55075

Contributor guide

Open the contributing guide

Research direction

Start by reviewing the INSPECT validation added in PR 155438 and the inspectBlocklistRegex entries in pkg/cmd/roachtest/test_runner.go. Identify a small, safe group of blocked roachtests to remove while avoiding the thundering herd described in the issue. Done means more roachtests perform INSPECT validation without overwhelming test runs.

Written by the indexing model from the issue text.

Assessment

Tech stack
go
Domain
testing
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.