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

kvserver: remove multiTestContext #61561

Merged
merged 1 commit into from
Mar 8, 2021

Conversation

lunevalex
Copy link
Collaborator

Closes #8299

With all the tests converted to use TestCluster/TestServer,
we can finally remove multiTestContext and all of the dependent
code.

Release note: None
Release justification: Cleans up unused test code.

@cockroach-teamcity
Copy link
Member

This change is Reviewable

@lunevalex lunevalex requested a review from andreimatei March 5, 2021 19:49
Closes cockroachdb#8299

With all the tests converted to use TestCluster/TestServer,
we can finally remove multiTestContext and all of the dependent
code.

Release note: None
Release justification: Cleans up unused test code.
@lunevalex lunevalex force-pushed the alex/delete-mtc-for-realz branch from 67404eb to 8560678 Compare March 5, 2021 20:27
@lunevalex lunevalex requested a review from tbg March 8, 2021 15:28
@lunevalex
Copy link
Collaborator Author

bors r+

@craig
Copy link
Contributor

craig bot commented Mar 8, 2021

Build succeeded:

@craig craig bot merged commit e9592f5 into cockroachdb:master Mar 8, 2021
lunevalex added a commit to lunevalex/cockroach that referenced this pull request Mar 25, 2021
PR cockroachdb#61561 removed multiTestContext. This PR cleans up comments
and code references that still referred to it.

Release note: None
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.

kvserver: migrate multiTestContext to TestCluster
3 participants