smallyao commented on code in PR #3378:
URL: https://github.com/apache/ambari/pull/3378#discussion_r990759181
##########
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 @kevinw66 It's also okay to make http response code 302 pass if just
simply resolve this issue. I will modify it.
--
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]
