-
Notifications
You must be signed in to change notification settings - Fork 4.1k
bazel: add more detail to ./dev test --help #82411
Copy link
Copy link
Closed
Labels
A-build-systemC-enhancementSolution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)T-dev-inf
Description
-
Currently, the cobra message does not describe the formatting or potential arguments for
--test-argsand-stress-args. I think the following could be specified like:stress-args: additional arguments to pass to stress, prefixed with-. E.g.--stress-args='-maxtime=30s'. See all stress args here https://github.com/cockroachdb/stress/blob/master/main.gotest-args: additional arguments to pass to the go test binary, prefixed with-test.{arg}. For example:--test-args='-test.timeout 1s'. Learn about go test args here [??]. (I'm not sure if the msg should point to a CRDB specific list of go test args)
-
Currently, the cobra message does not note that the user can pass bazel specific args behind
--. Perhaps in the flag list, we could add something like:--: add additional arguments to bazel, prefixed with--, e.g.-- --sandbox_debug. Note: thedevmaintainers would like all commonly used bazel args to get passed directly to dev. Please file an issue if you're commonly reaching for the bazel flag!
-
Currently, its unclear which flags operate on the dev level or go test level. It could be nice to specify which dev flags are merely shortcuts to go test, (e.g.
-verbose)
Jira issue: CRDB-16361
Epic CRDB-15061
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-build-systemC-enhancementSolution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)Solution expected to add code/behavior + preserve backward-compat (pg compat issues are exception)T-dev-inf