Skip to content
This repository has been archived by the owner on Jul 24, 2024. It is now read-only.

tests: wait tikv 100 seconds to init #288

Merged
merged 2 commits into from
May 15, 2020
Merged

Conversation

3pointer
Copy link
Collaborator

What problem does this PR solve?

try to fix integration_test failed with Failed to initialize TiKV cluster

What is changed and how it works?

wait tikv 100 seconds to start

Check List

Tests

  • Integration test

Related changes

  • Need to cherry-pick to the release branch

Release Note

  • No release note

@codecov
Copy link

codecov bot commented May 14, 2020

Codecov Report

Merging #288 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #288   +/-   ##
=======================================
  Coverage   71.41%   71.41%           
=======================================
  Files          48       48           
  Lines        5013     5013           
=======================================
  Hits         3580     3580           
  Misses        973      973           
  Partials      460      460           

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b334a0f...7457248. Read the comment docs.

Copy link
Collaborator

@kennytm kennytm left a comment

Choose a reason for hiding this comment

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

LGTM

@3pointer
Copy link
Collaborator Author

/run-cherry-picker

@3pointer 3pointer merged commit 00fe08c into pingcap:master May 15, 2020
@sre-bot
Copy link
Contributor

sre-bot commented May 15, 2020

cherry pick to release-3.1 in PR #290

@sre-bot
Copy link
Contributor

sre-bot commented May 15, 2020

cherry pick to release-4.0 in PR #291

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants