From dev-return-52604-apmail-cloudstack-dev-archive=cloudstack.apache.org@cloudstack.apache.org Mon Feb 24 15:06:15 2014 Return-Path: X-Original-To: apmail-cloudstack-dev-archive@www.apache.org Delivered-To: apmail-cloudstack-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 3D16810476 for ; Mon, 24 Feb 2014 15:06:15 +0000 (UTC) Received: (qmail 90682 invoked by uid 500); 24 Feb 2014 15:06:14 -0000 Delivered-To: apmail-cloudstack-dev-archive@cloudstack.apache.org Received: (qmail 90268 invoked by uid 500); 24 Feb 2014 15:06:13 -0000 Mailing-List: contact dev-help@cloudstack.apache.org; run by ezmlm Precedence: bulk List-Help: List-Unsubscribe: List-Post: List-Id: Reply-To: dev@cloudstack.apache.org Delivered-To: mailing list dev@cloudstack.apache.org Received: (qmail 90252 invoked by uid 99); 24 Feb 2014 15:06:12 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Mon, 24 Feb 2014 15:06:12 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 593AE1C764C; Mon, 24 Feb 2014 15:06:11 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============3813119411340968972==" MIME-Version: 1.0 Subject: Review Request 17995: Added fix for CLOUDSTACK - 6164 From: "Santhosh Edukulla" To: "Girish Shilamkar" Cc: "Santhosh Edukulla" , "cloudstack" Date: Mon, 24 Feb 2014 15:06:11 -0000 Message-ID: <20140224150611.21193.8948@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "Santhosh Edukulla" X-ReviewGroup: cloudstack X-ReviewRequest-URL: https://reviews.apache.org/r/17995/ X-Sender: "Santhosh Edukulla" Reply-To: "Santhosh Edukulla" X-ReviewRequest-Repository: cloudstack-git --===============3813119411340968972== 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/17995/ ----------------------------------------------------------- Review request for cloudstack and Girish Shilamkar. Bugs: CLOUDSTACK-6164 https://issues.apache.org/jira/browse/CLOUDSTACK-6164 Repository: cloudstack-git Description ------- Currently,Host class has a create method, uses addHost Command, but 1. It does not verify the output of addHost command is valid output in all forms. 2. Tests some times directly goes to using Host post Host.create, but we are not sure whether the host is up or not, they have to explicitly call again listHosts api call, so every where host is added list host is called. Instead, the create method now returns the FAILED or host object based upon the api success and whether the host is up. 3. Added a try\except block and return FAILED to tests in case of an exception. Diffs ----- tools/marvin/marvin/config/test_data.py PRE-CREATION tools/marvin/marvin/deployDataCenter.py c8feaaf Diff: https://reviews.apache.org/r/17995/diff/ Testing ------- Tested indentation and import changes. Thanks, Santhosh Edukulla --===============3813119411340968972==--