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

Unit tests for wrangler version of TabletExternallyReparented #5292

Merged
merged 3 commits into from
Oct 15, 2019

Conversation

deepthi
Copy link
Member

@deepthi deepthi commented Oct 11, 2019

NOTE: This PR is targeting the reparent-refactor feature branch, not master.

Duplicated relevant RPC tests for wrangler.
Moved unrelated tests to a different file, fixed RPC tests to not error out during SetMaster.

Signed-off-by: deepthi deepthi@planetscale.com

Duplicated relevant RPC tests for wrangler.
Moved unrelated tests to a different file, fixed RPC tests to not error
out during SetMaster

Signed-off-by: deepthi <deepthi@planetscale.com>
@deepthi deepthi requested a review from sougou as a code owner October 11, 2019 18:16
@deepthi deepthi requested a review from enisoc October 11, 2019 18:16
@deepthi deepthi changed the title Unit tests for wrangler version of TabletExternallyReparented. Unit tests for wrangler version of TabletExternallyReparented Oct 11, 2019
…otected by mutex

Signed-off-by: deepthi <deepthi@planetscale.com>
…Cancel

Signed-off-by: deepthi <deepthi@planetscale.com>
Copy link
Member

@enisoc enisoc left a comment

Choose a reason for hiding this comment

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

LGTM

@deepthi
Copy link
Member Author

deepthi commented Oct 15, 2019

I'm going to merge this into the feature branch, since the only failure is unit_race timeout.

@deepthi deepthi merged commit b43e6f5 into vitessio:reparent-refactor Oct 15, 2019
@deepthi deepthi deleted the ds-ter-tests branch October 15, 2019 03:15
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.

2 participants