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

roachtest: fix library searching logic #105524

Merged
merged 1 commit into from
Jun 26, 2023

Conversation

rafiss
Copy link
Collaborator

@rafiss rafiss commented Jun 25, 2023

The change in beb35aa refactored this logic. Now we need to combine the name and suffix inside of findBinaryOrLibrary.

Epic: None
Release note: None

Fixes this error:

Locating and verifying binaries for os="linux", arch="amd64"
looking in /home/rafiss/go/src/github.com/cockroachdb/cockroach for cockroach-ea
looking in /home/rafiss/go/src/github.com/cockroachdb/cockroach/artifacts for cockroach-ea
looking in /home/rafiss/go/src/github.com/cockroachdb/cockroach/bin for cockroach-ea
looking in /home/rafiss/go/src/github.com/cockroachdb/cockroach/bin for cockroach-ea
WARN: unable to find "cockroach-ea" for "amd64": binary or library "cockroach-ea" not found (or was not executable)
looking in /home/rafiss/go/src/github.com/cockroachdb/cockroach for libgeos
looking in /home/rafiss/go/src/github.com/cockroachdb/cockroach/artifacts for libgeos
looking in /home/rafiss/go/src/github.com/cockroachdb/cockroach/lib for libgeos
looking in /home/rafiss/go/src/github.com/cockroachdb/cockroach/lib for libgeos
WARN: unable to find library libgeos, ignoring: binary or library "libgeos" not found (or was not executable)
looking in /home/rafiss/go/src/github.com/cockroachdb/cockroach for libgeos_c
looking in /home/rafiss/go/src/github.com/cockroachdb/cockroach/artifacts for libgeos_c
looking in /home/rafiss/go/src/github.com/cockroachdb/cockroach/lib for libgeos_c
looking in /home/rafiss/go/src/github.com/cockroachdb/cockroach/lib for libgeos_c
WARN: unable to find library libgeos_c, ignoring: binary or library "libgeos_c" not found (or was not executable)

Found the following binaries:
	cockroach "amd64" at: /home/rafiss/go/src/github.com/cockroachdb/cockroach/cockroach-short
	workload "amd64" at: /home/rafiss/go/src/github.com/cockroachdb/cockroach/bin/workload
AMD64 clusters will be provisioned with probability 1.00
HTTP server listening on all network interfaces, port 8080.
22:22:22 main.go:637: test runner logs in: artifacts/_runner-logs/test_runner-1687731742.log
test runner logs in: artifacts/_runner-logs/test_runner-1687731742.log
22:22:22 test_runner.go:1399: [w0] Selected test: hibernate run: 1.
22:22:22 test_runner.go:642: [w0] Using randomly chosen arch="amd64", hibernate
22:22:22 test_runner.go:658: [w0] Library verification failed: cluster.VerifyLibraries: missing required library libgeos (arch="amd64")
Library verification failed: cluster.VerifyLibraries: missing required library libgeos (arch="amd64")
22:22:22 test_runner.go:532: [w0] Worker exiting; no cluster to destroy.
22:22:22 test_runner.go:320: Worker 0 returned with error. Quiescing. Error: cluster.VerifyLibraries: missing required library libgeos (arch="amd64")
Worker 0 returned with error. Quiescing. Error: cluster.VerifyLibraries: missing required library libgeos (arch="amd64")
22:22:22 test_runner.go:346: FAIL (err: cluster.VerifyLibraries: missing required library libgeos (arch="amd64"))
FAIL (err: cluster.VerifyLibraries: missing required library libgeos (arch="amd64"))
22:22:22 main.go:539: runTests destroying all clusters

The change in beb35aa refactored this
logic. Now we need to combine the name and suffix inside of
findBinaryOrLibrary.

Release note: None
@rafiss rafiss requested a review from srosenberg June 25, 2023 22:36
@rafiss rafiss requested a review from a team as a code owner June 25, 2023 22:36
@rafiss rafiss requested review from herkolategan and removed request for a team June 25, 2023 22:36
@cockroach-teamcity
Copy link
Member

This change is Reviewable

@rafiss rafiss added backport-22.2.x backport-23.1.x Flags PRs that need to be backported to 23.1 labels Jun 25, 2023
Copy link
Member

@srosenberg srosenberg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix!

@rafiss
Copy link
Collaborator Author

rafiss commented Jun 26, 2023

acceptance/version-upgrade flake is being fixed by #105391

bors r+

@craig
Copy link
Contributor

craig bot commented Jun 26, 2023

Build succeeded:

@craig craig bot merged commit 72f7db9 into cockroachdb:master Jun 26, 2023
@rafiss rafiss deleted the roachtest-fix-lib-not-found branch June 26, 2023 17:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-23.1.x Flags PRs that need to be backported to 23.1
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants