> On Фев. 12, 2015, 3:01 п.п., Nate Cole wrote: > > Any new tests?
tests will be added in scope of another technical debt jira > On Фев. 12, 2015, 3:01 п.п., Nate Cole wrote: > > ambari-agent/src/main/python/ambari_agent/alerts/web_alert.py, lines 126-127 > > <https://reviews.apache.org/r/30901/diff/1/?file=861266#file861266line126> > > > > Can this fail? I guess if it does the curl call will bomb out too. in case of unsuccessful execution of kinit command, curl execution will just try to request specified url without spnego authentication and provide proper error message in case of failure - Yurii ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/30901/#review72161 ----------------------------------------------------------- On Фев. 12, 2015, 11:36 п.п., Yurii Shylov wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/30901/ > ----------------------------------------------------------- > > (Updated Фев. 12, 2015, 11:36 п.п.) > > > Review request for Ambari, Jonathan Hurley and Srimanth Gunturi. > > > Bugs: AMBARI-9586 > https://issues.apache.org/jira/browse/AMBARI-9586 > > > Repository: ambari > > > Description > ------- > > A bunch of alerts for web are failing in secured cluster: > - App Timeline Web UI > - Oozie Server Web UI > - WebHCat Server Status > - Storm Web UI > > Looks like the reason is that in secured cluster web services require > authentification (401 HTTP code) which causes exception "urllib2.HTTPError: > HTTP Error 401: Authentication required" for common web check: > {code} > url_response = urllib2.urlopen(query) > {code} > > > Diffs > ----- > > ambari-agent/src/main/python/ambari_agent/alerts/base_alert.py 283da9c > ambari-agent/src/main/python/ambari_agent/alerts/web_alert.py f736135 > > ambari-server/src/main/java/org/apache/ambari/server/state/alert/AlertUri.java > 731e41a > > ambari-server/src/main/resources/common-services/FALCON/0.5.0.2.1/alerts.json > 7843676 > ambari-server/src/main/resources/common-services/HDFS/2.1.0.2.0/alerts.json > 6aa7b2c > > ambari-server/src/main/resources/common-services/OOZIE/4.0.0.2.0/alerts.json > 3eae69f > > ambari-server/src/main/resources/common-services/STORM/0.9.1.2.1/alerts.json > 4c4e027 > ambari-server/src/main/resources/common-services/YARN/2.1.0.2.0/alerts.json > 5361bb5 > > Diff: https://reviews.apache.org/r/30901/diff/ > > > Testing > ------- > > Tests ok > > > Thanks, > > Yurii Shylov > >
