----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/61605/#review182859 -----------------------------------------------------------
Can you provide a test case for this? - Jonathan Hurley On Aug. 12, 2017, 9:43 a.m., Eugene Chekanskiy wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/61605/ > ----------------------------------------------------------- > > (Updated Aug. 12, 2017, 9:43 a.m.) > > > Review request for Ambari, Jonathan Hurley and Nate Cole. > > > Bugs: AMBARI-21712 > https://issues.apache.org/jira/browse/AMBARI-21712 > > > Repository: ambari > > > Description > ------- > > Sometimes WEB alert can guaranty that codes more than 400 will guaranty that > WEB working properly, but ambari has hardcoded values of acceptable HTTP > codes. This patch allows for WEB alerts to define custom list of acceptable > codes that will be accepted alongside with codes less than 400. > > > Diffs > ----- > > ambari-agent/src/main/python/ambari_agent/alerts/base_alert.py add29fc > ambari-agent/src/main/python/ambari_agent/alerts/web_alert.py 8ce4405 > > ambari-server/src/main/java/org/apache/ambari/server/state/alert/AlertUri.java > dea105e > > > Diff: https://reviews.apache.org/r/61605/diff/1/ > > > Testing > ------- > > mvn clean test > > > Thanks, > > Eugene Chekanskiy > >
