> On Sept. 1, 2017, 12:41 p.m., Robert Levas wrote: > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/service_check.py > > Lines 182-190 (patched) > > <https://reviews.apache.org/r/61998/diff/1/?file=1807987#file1807987line185> > > > > This can be simplified to > > > > ``` > > return params.security_enabled \ > > and 'core-site' in params.config['configurations'] \ > > and 'hadoop.http.authentication.type' in > > params.config['configurations']['core-site'] \ > > and > > params.config['configurations']['core-site']['hadoop.http.authentication.type'] > > == "kerberos" \ > > and 'hadoop.http.filter.initializers' in > > params.config['configurations']['core-site'] \ > > and > > params.config['configurations']['core-site']['hadoop.http.filter.initializers'] > > == "org.apache.hadoop.security.AuthenticationFilterInitializer" > > ```
Hi Robert, Thank you so much for the review! I will fix it! - Qin ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61998/#review184351 ----------------------------------------------------------- On Aug. 30, 2017, 7:07 p.m., Qin Liu wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61998/ > ----------------------------------------------------------- > > (Updated Aug. 30, 2017, 7:07 p.m.) > > > Review request for Ambari, Aravindan Vijayan, Dmytro Sen, Eric Yang, Robert > Levas, and Sid Wagle. > > > Bugs: AMBARI-21821 > https://issues.apache.org/jira/browse/AMBARI-21821 > > > Repository: ambari > > > Description > ------- > > This is a subtask of AMBARI-14384 "Ambari Metrics doesn't use SPNEGO to > authenticate". > In a Kerberos enabled cluster with SPNEGO enabled on Hadoop APIs, Ambari > Metrics Collector web-console will be Kerberos HTTP SPNEGO enabled too. But > Ambari Metrics service check currently does not support Kerberos HTTP SPNEGO > authentication. > > 2017-08-24 16:13:44,078 - Ambari Metrics service check was started. > 2017-08-24 16:13:44,109 - Generated metrics for host amshost.example.com : > { > "metrics": [ > { > "metricname": "AMBARI_METRICS.SmokeTest.FakeMetric", > "appid": "amssmoketestfake", > "hostname": "host.example.com", > "timestamp": 1503616424000, > "starttime": 1503616424000, > "metrics": > { "1503616424000": 0.164134766408, "1503616425000": 1503616424000 } > > } > ] > } > 2017-08-24 16:13:44,114 - Connecting (POST) to > amshost.example.com:6188/ws/v1/timeline/metrics/ > 2017-08-24 16:13:44,117 - Http response for host amshost.example.com: 401 > Authentication required > > > Diffs > ----- > > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/params.py > 6975bec > > ambari-server/src/main/resources/common-services/AMBARI_METRICS/0.1.0/package/scripts/service_check.py > 2b3dfa9 > > > Diff: https://reviews.apache.org/r/61998/diff/1/ > > > Testing > ------- > > 1. run the Hadoop QA testing twice and the testcase failures has nothing to > do with the patch > > 2. manually tested AMS service check on HDP/trunk cluster with combination of > HTTP SPNEGO/Non-HTTP SPNEGO, SSL/Non-SSL, AMS distributed/embedded. > > > Thanks, > > Qin Liu > >