-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/30901/#review72375
-----------------------------------------------------------
I was about to push this, but it looks like there's a test failure:
======================================================================
FAIL: test_web_alert (TestAlerts.TestAlerts)
----------------------------------------------------------------------
Traceback (most recent call last):
File
"/Users/jhurley/src/hwx/ambari/ambari-common/src/test/python/mock/mock.py",
line 1199, in patched
return func(*args, **keywargs)
File
"/Users/jhurley/src/hwx/ambari/ambari-agent/src/test/python/ambari_agent/TestAlerts.py",
line 440, in test_web_alert
self.assertEquals('OK', alerts[0]['state'])
AssertionError: 'OK' != 'UNKNOWN'
- Jonathan Hurley
On Feb. 12, 2015, 6:36 p.m., Yurii Shylov wrote:
>
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/30901/
> -----------------------------------------------------------
>
> (Updated Feb. 12, 2015, 6:36 p.m.)
>
>
> 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
>
>
