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

healthz API returns OK not 200 #3398

Closed
chrishare08 opened this issue Nov 21, 2019 · 5 comments · May be fixed by sumodgeorge/graphql-engine#30, Allymahmoud/graphql-engine#386 or ptakpatryk/graphql-engine#27
Labels
c/server Related to server k/question support/needs-more-info Needs more details/info/repro instructions

Comments

@chrishare08
Copy link

Documentation says the healthz API returns a 200 on success and a 500 on failure. This is not what happens. The API returns the text string "ok".

@lexi-lambda
Copy link
Contributor

The healthz endpoint returns an HTTP status code of 200 on success and a status code of 500 on failure—the actual response body is not especially meaningful. Are you seeing something else? And if so, what status code are you seeing?

@lexi-lambda lexi-lambda added c/server Related to server k/question support/needs-more-info Needs more details/info/repro instructions labels Nov 21, 2019
@chrishare08
Copy link
Author

MacBook-Pro-2:.ssh robert.hare$ curl http://172.31.3.214:8080/healthz
OK
MacBook-Pro-2:.ssh robert.hare$ curl -v http://172.31.3.214:8080/healthz

  • Trying 172.31.3.214...
  • TCP_NODELAY set
  • Connected to 172.31.3.214 (172.31.3.214) port 8080 (#0)

GET /healthz HTTP/1.1
Host: 172.31.3.214:8080
User-Agent: curl/7.54.0
Accept: /

< HTTP/1.1 200 OK
< Transfer-Encoding: chunked
< Date: Fri, 22 Nov 2019 03:52:43 GMT
< Server: Warp/3.2.27
<

  • Connection #0 to host 172.31.3.214 left intact
    MacBook-Pro-2:.ssh robert.hare$

the health check for the AWS target group always fails, even though I can get an okay response from teh EC2 instance

@amrnt
Copy link

amrnt commented Nov 22, 2019

< HTTP/1.1 200 OK
< Transfer-Encoding: chunked

It says 200 OK whats the problem?

@lexi-lambda
Copy link
Contributor

I agree with @amrnt—I’m not totally sure what the issue is here/what you think should work differently. I’m going to close this, but feel free to reopen it if you think it’s still an issue and can provide a little more information.

@chrishare08
Copy link
Author

no worries. I think we were grasping at straws when we couldn't get the health checks reporting correctly.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
c/server Related to server k/question support/needs-more-info Needs more details/info/repro instructions
Projects
None yet
3 participants