Skip to content

roachtest: deflake acceptance/version-upgrade#47690

Merged
craig[bot] merged 1 commit intocockroachdb:masterfrom
tbg:version-upgrade-text-file-busy
Apr 21, 2020
Merged

roachtest: deflake acceptance/version-upgrade#47690
craig[bot] merged 1 commit intocockroachdb:masterfrom
tbg:version-upgrade-text-file-busy

Conversation

@tbg
Copy link
Copy Markdown
Member

@tbg tbg commented Apr 20, 2020

Remove a workaround that wasn't necessary any more (since I regenerated
the fixtures last week) but which caused flakes of its own because it
was re-uploading the binary for predecessorVersion while processes were
running using that binary (resulting in occasional 'text file busy' on
linux).

Closes #44732.
Touches #47024.

Release note: None

Remove a workaround that wasn't necessary any more (since I regenerated
the fixtures last week) but which caused flakes of its own because it
was re-uploading the binary for predecessorVersion while processes were
running using that binary (resulting in occasional 'text file busy' on
linux).

Closes cockroachdb#44732.
Touches cockroachdb#47024.

Release note: None
@tbg tbg requested a review from spaskob April 20, 2020 10:35
@cockroach-teamcity
Copy link
Copy Markdown
Member

This change is Reviewable

Copy link
Copy Markdown
Contributor

@spaskob spaskob 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! 0 of 0 LGTMs obtained (waiting on @spaskob)

@tbg
Copy link
Copy Markdown
Member Author

tbg commented Apr 20, 2020

TFTR!

bors r=spaskob

@irfansharif irfansharif mentioned this pull request Apr 20, 2020
20 tasks
@craig
Copy link
Copy Markdown
Contributor

craig bot commented Apr 20, 2020

Build failed

@tbg
Copy link
Copy Markdown
Member Author

tbg commented Apr 21, 2020

Timed out on a gossip-related roachtest. I think the test flaked and then the consistency check got stuck forever. I'll send a PR for that.

bors r=spaskob

@craig
Copy link
Copy Markdown
Contributor

craig bot commented Apr 21, 2020

Build succeeded

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 this pull request may close these issues.

roachtest: acceptance/version-upgrade failed

3 participants