Return-Path: 
 <dev-return-31182-apmail-ambari-dev-archive=ambari.apache.org@ambari.apache.org>
X-Original-To: apmail-ambari-dev-archive@www.apache.org
Delivered-To: apmail-ambari-dev-archive@www.apache.org
Received: from mail.apache.org (hermes.apache.org [140.211.11.3])
	by minotaur.apache.org (Postfix) with SMTP id 273BB11794
	for <apmail-ambari-dev-archive@www.apache.org>;
 Fri, 13 Jun 2014 13:58:31 +0000 (UTC)
Received: (qmail 27538 invoked by uid 500); 13 Jun 2014 13:58:30 -0000
Delivered-To: apmail-ambari-dev-archive@ambari.apache.org
Received: (qmail 27509 invoked by uid 500); 13 Jun 2014 13:58:30 -0000
Mailing-List: contact dev-help@ambari.apache.org; run by ezmlm
Precedence: bulk
List-Help: <mailto:dev-help@ambari.apache.org>
List-Unsubscribe: <mailto:dev-unsubscribe@ambari.apache.org>
List-Post: <mailto:dev@ambari.apache.org>
List-Id: <dev.ambari.apache.org>
Reply-To: dev@ambari.apache.org
Delivered-To: mailing list dev@ambari.apache.org
Received: (qmail 27497 invoked by uid 99); 13 Jun 2014 13:58:30 -0000
Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40)
    by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 13 Jun 2014 13:58:30 +0000
Received: from reviews.apache.org (localhost [127.0.0.1])
	by reviews.apache.org (Postfix) with ESMTP id 4BE831D9D05;
	Fri, 13 Jun 2014 13:58:21 +0000 (UTC)
Content-Type: multipart/alternative;
 boundary="===============8243777566808366272=="
MIME-Version: 1.0
Subject: Re: Review Request 22541: Request and Stage should not throw
 RuntimeException when cluster could not be loaded
From: "Nate Cole" <ncole@hortonworks.com>
To: "Sid Wagle" <swagle@hortonworks.com>
Cc: "Nate Cole" <ncole@hortonworks.com>, "Ambari" <dev@ambari.apache.org>
Date: Fri, 13 Jun 2014 13:58:21 -0000
Message-ID: <20140613135821.22323.51376@reviews.apache.org>
X-ReviewBoard-URL: https://reviews.apache.org
Auto-Submitted: auto-generated
Sender: "Nate Cole" <noreply@reviews.apache.org>
X-ReviewGroup: Ambari
X-ReviewRequest-URL: https://reviews.apache.org/r/22541/
X-Sender: "Nate Cole" <noreply@reviews.apache.org>
References: <20140612223148.22324.67722@reviews.apache.org>
In-Reply-To: <20140612223148.22324.67722@reviews.apache.org>
Reply-To: "Nate Cole" <ncole@hortonworks.com>
X-ReviewRequest-Repository: ambari

--===============8243777566808366272==
Content-Type: text/plain; charset="utf-8"
MIME-Version: 1.0
Content-Transfer-Encoding: 7bit


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/22541/
-----------------------------------------------------------

(Updated June 13, 2014, 9:58 a.m.)


Review request for Ambari and Sid Wagle.


Changes
-------

Updated patch to change from LOG.error to LOG.debug (log gets filled up fast in the case of that initial failure).


Bugs: AMBARI-6105
    https://issues.apache.org/jira/browse/AMBARI-6105


Repository: ambari


Description
-------

Steps to reproduce:
* Install cluster, and service install (Nagios) fails
* Return to de-select Nagios
* Continue to install, everything green
* Restart ambari-server, and get error 500: Cluster <id> not found


Diffs (updated)
-----

  ambari-server/src/main/java/org/apache/ambari/server/actionmanager/Request.java d1b7e5c 
  ambari-server/src/main/java/org/apache/ambari/server/actionmanager/Stage.java 98ca13b 

Diff: https://reviews.apache.org/r/22541/diff/


Testing
-------

[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESS
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 15:24.685s
[INFO] Finished at: Thu Jun 12 18:21:40 EDT 2014
[INFO] Final Memory: 21M/123M
[INFO] ------------------------------------------------------------------------


Thanks,

Nate Cole


--===============8243777566808366272==--