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

test case run into docker #5885

Merged
merged 1 commit into from
Mar 9, 2020
Merged

Conversation

princeparmar
Copy link
Contributor

Testcase was failing at Add Cell info.

while it was trying to create cell in etcd it was failing because of invalid error file.
error file was invalid because process was not able to create the error file and there was no check in the error of file creation.

process was not able to create file because the etcd data directory was not avaiable at that time.

@deepthi
Copy link
Member

deepthi commented Mar 3, 2020

@morgo do we want to run all CI tests using docker? that is what this PR seems to be doing.

@morgo
Copy link
Contributor

morgo commented Mar 3, 2020

@morgo do we want to run all CI tests using docker? that is what this PR seems to be doing.

I think I would rather not require Docker. It is okay if it is one specific test, but it makes it easier on contributors if this is optional (with only minimal dependencies)

@deepthi deepthi self-requested a review March 4, 2020 01:05
@princeparmar princeparmar force-pushed the docker-for-e2e branch 3 times, most recently from db971d8 to bf581f3 Compare March 6, 2020 05:41
failing because topo was not able to create err file.

Signed-off-by: pradip parmar <[email protected]>
Copy link
Member

@deepthi deepthi 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 deepthi merged commit b7627dc into vitessio:master Mar 9, 2020
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.

3 participants