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.

Sign up for GitHub

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

Jump to bottom

fix scheduler ha bug #443

Merged
merged 1 commit into from
May 1, 2019
Merged

fix scheduler ha bug #443

merged 1 commit into from
May 1, 2019

Conversation

weekface
Copy link
Contributor

What problem does this PR solve?

this pr fixes #442

What is changed and how it works?

Check List

Tests

  • Unit test
  • E2E test
  • Stability test
  • No code

Code changes

  • Has Go code change

Side effects

Related changes

  • Need to cherry-pick to the release branch

@weekface
Copy link
Contributor Author

/run-e2e-tests

@weekface weekface added type/bug Something isn't working test/stability stability tests labels Apr 30, 2019
tennix
tennix approved these changes May 1, 2019
View reviewed changes
Copy link
Member

@tennix tennix left a comment

Choose a reason for hiding this comment

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

LGTM

@tennix tennix merged commit f9166c3 into pingcap:master May 1, 2019
@weekface weekface deleted the fix-scheduler-ha-bug branch May 1, 2019 00:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tennix tennix tennix approved these changes

@xiaojingchen xiaojingchen xiaojingchen approved these changes

@cofyc cofyc Awaiting requested review from cofyc

@onlymellb onlymellb Awaiting requested review from onlymellb

Assignees
No one assigned
Labels
test/stability stability tests type/bug Something isn't working
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

tidb-scheduler can't scheduling pd
3 participants
@weekface @xiaojingchen @tennix