[email protected] has posted comments on this change. ( 
http://gerrit.cloudera.org:8080/17962 )

Change subject: KUDU-3308: Add health check endpoint to webserver
......................................................................


Patch Set 1:

> Patch Set 1: Code-Review+1
>
> Is this API easily extendable to add a request param to return health like 
> ksck output?

It should be possible to pass additional arguments via HTTP Get params and 
change the output according to that. However that might prove limited use for 
HTTP request based Liveness Probe purposes. From the kubernetes docs:

"Any code greater than or equal to 200 and less than 400 indicates success. Any 
other code indicates failure."

If we need a more comprehensive check for liveness probe, then most likely a 
command based Liveness Probe.


--
To view, visit http://gerrit.cloudera.org:8080/17962
To unsubscribe, visit http://gerrit.cloudera.org:8080/settings

Gerrit-Project: kudu
Gerrit-Branch: master
Gerrit-MessageType: comment
Gerrit-Change-Id: Ic735e210d95aeec1dfd1335ee972318f12d52475
Gerrit-Change-Number: 17962
Gerrit-PatchSet: 1
Gerrit-Owner: Anonymous Coward <[email protected]>
Gerrit-Reviewer: Anonymous Coward <[email protected]>
Gerrit-Reviewer: Attila Bukor <[email protected]>
Gerrit-Reviewer: Bankim Bhavsar <[email protected]>
Gerrit-Reviewer: Kudu Jenkins (120)
Gerrit-Comment-Date: Tue, 26 Oct 2021 18:09:43 +0000
Gerrit-HasComments: No

Reply via email to