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.

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

Added /health endpoint which returns the version + status of the proxy and its netmaster #75

Merged
merged 1 commit into from
Jan 21, 2017

Conversation

dseevr
Copy link
Contributor

@dseevr dseevr commented Jan 21, 2017

Example responses

These are pretty-printed here, but the actual raw responses are not.

Healthy response

{
    "netmaster": {
        "status": "healthy",
        "version": "pv0.1-01-05-2017.23-41-24.UTC"
    },
    "status": "healthy",
    "version": "devbuild"
}

Unhealthy response

{
    "netmaster": {
        "reason": "failed to connect to netmaster: Get http://localhost:12345/version: dial tcp 127.0.0.1:12345: getsockopt: connection refused",
        "status": "unhealthy"
    },
    "status": "unhealthy",
    "version": "devbuild"
}

Fixes https://github.com/contiv/ccn/issues/154

@dseevr dseevr requested review from rhim and neelimamukiri January 21, 2017 00:16
Copy link
Contributor

@rhim rhim left a comment

Choose a reason for hiding this comment

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

LGTM.

@dseevr dseevr merged commit d791afb into contiv:master Jan 21, 2017
@dseevr dseevr deleted the obamacare branch January 21, 2017 01:24
dseevr pushed a commit to dseevr-dev/auth_proxy that referenced this pull request Dec 21, 2017
* organizations tab ui

* fixed organization url

* added organization icon

* CIDR regular expression not needed

* Removed commented coded
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants