kevinw66 commented on code in PR #3378:
URL: https://github.com/apache/ambari/pull/3378#discussion_r990657793
##########
ambari-server/src/main/resources/stacks/BIGTOP/3.2.0/services/HDFS/package/files/checkWebUI.py:
##########
@@ -67,19 +67,21 @@ def main():
parser.add_option("-p", "--port", dest="port", help="Port of WEB UI to check
it availability")
parser.add_option("-s", "--https", dest="https", help="\"True\" if value of
dfs.http.policy is \"HTTPS_ONLY\"")
parser.add_option("-o", "--protocol", dest="protocol", help="Protocol to use
when executing https request")
+ parser.add_option("-a", "--path", dest="path", help="Path of WEB UI")
(options, args) = parser.parse_args()
hosts = options.hosts.split(',')
port = options.port
https = options.https
protocol = options.protocol
+ path = options.path
for host in hosts:
- httpCode = make_connection(host, port, https.lower() == "true", protocol)
+ httpCode = make_connection(host, port, https.lower() == "true", protocol,
path)
if httpCode != 200:
Review Comment:
Hi @smallyao , after I take a deep look at this bug, I found the error only
occurs when Kerberos is disabled and HA is enabled, it reports an error when
http response code is 200, but now it returns 302, which I think also should be
healthy status.
Can we simply add one more condition here to make 302 passed too?
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
