Skip to content

Commit c421c85

Browse files
committed
dev: use bazel test to run acceptance tests, not bazel run
This is probably some oversight or historical artifact, but `bazel test` is conceptually correct and what we should be using in CI. Epic: CRDB-17171 Release note: None
1 parent c3a0838 commit c421c85

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

dev

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,7 @@ fi
88
set -euo pipefail
99

1010
# Bump this counter to force rebuilding `dev` on all machines.
11-
DEV_VERSION=77
11+
DEV_VERSION=78
1212

1313
THIS_DIR=$(cd "$(dirname "$0")" && pwd)
1414
BINARY_DIR=$THIS_DIR/bin/dev-versions

pkg/cmd/dev/acceptance.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -76,7 +76,7 @@ func (d *dev) acceptance(cmd *cobra.Command, commandLine []string) error {
7676
cockroachBin := filepath.Join(workspace, "artifacts", "cockroach-short")
7777

7878
var args []string
79-
args = append(args, "run", "//pkg/acceptance:acceptance_test", "--config=test")
79+
args = append(args, "test", "//pkg/acceptance:acceptance_test")
8080
if numCPUs != 0 {
8181
args = append(args, fmt.Sprintf("--local_cpu_resources=%d", numCPUs))
8282
}

0 commit comments

Comments
 (0)