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

Quality reinforcement: fix unstable test cases (#916) #953

Merged
merged 3 commits into from
Aug 28, 2020

Conversation

csuzhangxc
Copy link
Member

@csuzhangxc csuzhangxc commented Aug 28, 2020

Co-authored-by: lance6716 [email protected]
manually cherry-pick #916

What problem does this PR solve?

fix unstable test cases in CI.

What is changed and how it works?

  • /run-all-tests to find unstable cases.
  • push commit in this pr to fix the cases.

Co-authored-by: lance6716 <[email protected]>
(cherry picked from commit 10e739c)
@csuzhangxc csuzhangxc requested review from GMHDBJD and lance6716 and removed request for GMHDBJD August 28, 2020 10:46
@csuzhangxc csuzhangxc added the type/test Changes related to test cases label Aug 28, 2020
@csuzhangxc csuzhangxc added this to the v2.0.0 RC.2 milestone Aug 28, 2020
@lance6716
Copy link
Collaborator

LGTM

@ti-srebot ti-srebot added the status/LGT1 One reviewer already commented LGTM label Aug 28, 2020
Copy link
Collaborator

@GMHDBJD GMHDBJD left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@GMHDBJD GMHDBJD left a comment

Choose a reason for hiding this comment

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

LGTM

@GMHDBJD GMHDBJD added status/LGT2 Two reviewers already commented LGTM, ready for merge and removed status/LGT1 One reviewer already commented LGTM labels Aug 28, 2020
@csuzhangxc
Copy link
Member Author

/run-all-tests

@csuzhangxc csuzhangxc merged commit 535bdf0 into pingcap:release-2.0 Aug 28, 2020
@csuzhangxc csuzhangxc deleted the cp-916 branch August 28, 2020 12:55
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
status/LGT2 Two reviewers already commented LGTM, ready for merge type/test Changes related to test cases
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants