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

pdctl: support health check #792

Merged
merged 1 commit into from Oct 12, 2017

Conversation

Projects
None yet
3 participants
@Connor1996
Member

Connor1996 commented Oct 11, 2017

As mentioned in #750

@siddontang

This comment has been minimized.

Show comment
Hide comment
@siddontang

siddontang Oct 11, 2017

Member

what is the output for this command in pd-ctl?

Member

siddontang commented Oct 11, 2017

what is the output for this command in pd-ctl?

@Connor1996

This comment has been minimized.

Show comment
Hide comment
@Connor1996

Connor1996 Oct 11, 2017

Member

@siddontang {"health": "true"} or {"health": "false"}

Member

Connor1996 commented Oct 11, 2017

@siddontang {"health": "true"} or {"health": "false"}

@siddontang

LGTM

@disksing

This comment has been minimized.

Show comment
Hide comment
@disksing

disksing Oct 12, 2017

Member

LGTM.

Member

disksing commented Oct 12, 2017

LGTM.

@disksing disksing merged commit f5bf59e into master Oct 12, 2017

4 checks passed

ci/circleci Your tests passed on CircleCI!
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
jenkins-ci-pd/build Jenkins job succeeded.
Details
license/cla Contributor License Agreement is signed.
Details

@disksing disksing deleted the connor/pdctl-support-health branch Oct 12, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment