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

check_task: cherrypick ping db when apply dbconfig #778

Merged
merged 2 commits into from
Jul 2, 2020

Conversation

GMHDBJD
Copy link
Collaborator

@GMHDBJD GMHDBJD commented Jul 2, 2020

manually cherrypick #769

What problem does this PR solve?

ping database when apply db config

Tests

  • Integration test

@GMHDBJD GMHDBJD added priority/normal Minor change, requires approval from ≥1 primary reviewer status/PTAL This PR is ready for review. Add this label back after committing new changes type/cherry-pick This PR is just a cherry-pick (backport) labels Jul 2, 2020
Copy link
Contributor

@WangXiangUSTC WangXiangUSTC left a comment

Choose a reason for hiding this comment

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

LGTM

@WangXiangUSTC WangXiangUSTC added status/LGT1 One reviewer already commented LGTM and removed status/PTAL This PR is ready for review. Add this label back after committing new changes labels Jul 2, 2020
@lance6716
Copy link
Collaborator

LGTM

@ti-srebot
Copy link

@lance6716, Thanks for your review, however we are sorry that your vote won't be count. You are not a reviewer or committer or co-leader or leader for the related sigs:tools(slack).

@lance6716 lance6716 added status/LGT2 Two reviewers already commented LGTM, ready for merge and removed status/LGT1 One reviewer already commented LGTM labels Jul 2, 2020
@GMHDBJD GMHDBJD merged commit d5d1c07 into pingcap:release-1.0 Jul 2, 2020
@GMHDBJD GMHDBJD deleted the cherrypickPingDB branch July 2, 2020 11:24
@csuzhangxc csuzhangxc added this to the v1.0.7 milestone Jul 15, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
priority/normal Minor change, requires approval from ≥1 primary reviewer status/LGT2 Two reviewers already commented LGTM, ready for merge type/cherry-pick This PR is just a cherry-pick (backport)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants