Skip to content

roachtest: misc fixes#94361

Merged
craig[bot] merged 3 commits intocockroachdb:masterfrom
tbg:follower-reads-again
Dec 29, 2022
Merged

roachtest: misc fixes#94361
craig[bot] merged 3 commits intocockroachdb:masterfrom
tbg:follower-reads-again

Conversation

@tbg
Copy link
Copy Markdown
Member

@tbg tbg commented Dec 28, 2022

Fixes #94272.
Fixes #94266.

Epic: none
Release note: None

Epic: none
Release note: None
@tbg tbg requested a review from a team as a code owner December 28, 2022 09:43
@tbg tbg requested review from erikgrinaker, herkolategan and renatolabs and removed request for a team December 28, 2022 09:43
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

Fixes cockroachdb#94204.

Epic: none
Release note: None
@tbg tbg changed the title roachtest: fix follower-reads roachtest: misc fixes Dec 28, 2022
Errors were reported like this:

> (test_impl.go:291).Fatal: pq: invalid syntax @1

but that line is the impl of `Fatal`. We want to walk up one more frame.

Epic: none
Release note: None
@tbg
Copy link
Copy Markdown
Member Author

tbg commented Dec 28, 2022

bors r=erikgrinaker
Tftr!

@craig
Copy link
Copy Markdown
Contributor

craig Bot commented Dec 28, 2022

🕐 Waiting for PR status (Github check) to be set, probably by CI. Bors will automatically try to run when all required PR statuses are set.

Copy link
Copy Markdown
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.

:lgtm:

Reviewable status: :shipit: complete! 1 of 0 LGTMs obtained (waiting on @herkolategan and @renatolabs)

@healthy-pod
Copy link
Copy Markdown
Contributor

Re-borsing after license/cla re-delivery.

bors r=erikgrinaker

@craig
Copy link
Copy Markdown
Contributor

craig Bot commented Dec 29, 2022

Build succeeded:

@craig craig Bot merged commit f756b21 into cockroachdb:master Dec 29, 2022
@tbg tbg deleted the follower-reads-again branch January 4, 2023 08:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

5 participants