Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

CRDB Check Broken due to missing the latest (v23.1.14) binary #3300

Closed
roger2hk opened this issue Jan 17, 2024 · 1 comment · Fixed by #3301
Closed

CRDB Check Broken due to missing the latest (v23.1.14) binary #3300

roger2hk opened this issue Jan 17, 2024 · 1 comment · Fixed by #3301

Comments

@roger2hk
Copy link
Contributor

roger2hk commented Jan 17, 2024

Error Log

2024/01/17 21:59:52 cockroach logs directory: /tmp/cockroach-testserver1754551371
2024/01/17 21:59:52 GET https://binaries.cockroachdb.com/cockroach-v23.1.14.linux-amd64.tgz
2024/01/17 21:59:52 cockroach-go testserver: Failed to fetch latest binary: error downloading https://binaries.cockroachdb.com/cockroach-v23.1.14.linux-amd64.tgz: 404 (404 Not Found) attempting to use cockroach binary from your PATH
E0117 21:59:52.931905    3719 common_test.go:29] Failed to start test server: cockroach-go testserver command cockroach version failed: exec: "cockroach": executable file not found in $PATH
FAIL	github.com/google/trillian/quota/crdbqm	0.175s
FAIL
Error: Process completed with exit code 1.

Root Cause

The test server version is currently pinned to 22.2.7 for /storage/crdb but not /quota/crdbqm/. The latest version is automatically picked for /quota/crdbqm/ test. However, the latest version (v23.1.14) is only available only for CockroachDB Cloud clusters. Thus, the CRDB test checks always failed.

ts, err := testserver.NewTestServer(testserver.CustomVersionOpt("22.2.7"))

ts, err := testserver.NewTestServer()

@roger2hk roger2hk changed the title CRDB Check Broken due to CockroachDB v22.1 EOL CRDB Check Broken due to missing v23.1.14 binary Jan 17, 2024
@roger2hk roger2hk changed the title CRDB Check Broken due to missing v23.1.14 binary CRDB Check Broken due to missing the latest (v23.1.14) binary Jan 17, 2024
@roger2hk
Copy link
Contributor Author

cc @pav-kv

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant