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 7954710A62 for ; Tue, 1 Apr 2014 10:47:49 +0000 (UTC) Received: (qmail 52166 invoked by uid 500); 1 Apr 2014 10:47:46 -0000 Delivered-To: apmail-cloudstack-dev-archive@cloudstack.apache.org Received: (qmail 50979 invoked by uid 500); 1 Apr 2014 10:47:44 -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 50819 invoked by uid 99); 1 Apr 2014 10:47:43 -0000 Received: from reviews-vm.apache.org (HELO reviews.apache.org) (140.211.11.40) by apache.org (qpsmtpd/0.29) with ESMTP; Tue, 01 Apr 2014 10:47:43 +0000 Received: from reviews.apache.org (localhost [127.0.0.1]) by reviews.apache.org (Postfix) with ESMTP id 1DC161C011D; Tue, 1 Apr 2014 10:47:40 +0000 (UTC) Content-Type: multipart/alternative; boundary="===============0818346063049313637==" MIME-Version: 1.0 Subject: Re: Review Request 19866: Added Fixes for CLOUDSTACK-6316 From: "ASF Subversion and Git Services" To: "SrikanteswaraRao Talluri" Cc: "Santhosh Edukulla" , "ASF Subversion and Git Services" , "cloudstack" Date: Tue, 01 Apr 2014 10:47:40 -0000 Message-ID: <20140401104740.28608.33098@reviews.apache.org> X-ReviewBoard-URL: https://reviews.apache.org Auto-Submitted: auto-generated Sender: "ASF Subversion and Git Services" X-ReviewGroup: cloudstack X-ReviewRequest-URL: https://reviews.apache.org/r/19866/ X-Sender: "ASF Subversion and Git Services" References: <20140401100446.2770.31994@reviews.apache.org> In-Reply-To: <20140401100446.2770.31994@reviews.apache.org> Reply-To: "ASF Subversion and Git Services" X-ReviewRequest-Repository: cloudstack-git --===============0818346063049313637== 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/19866/#review39148 ----------------------------------------------------------- Commit 63d36883605e42cf48eec1f50e2496f43ea6a496 in cloudstack's branch refs/heads/marvin from Santhosh Edukulla [ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=63d3688 ] Added fix for CLOUDSTACK-6316 Added changes for CLOUDSTACK-6316. More details in the bug Signed-off-by: Santhosh Edukulla - ASF Subversion and Git Services On April 1, 2014, 10:04 a.m., Santhosh Edukulla wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/19866/ > ----------------------------------------------------------- > > (Updated April 1, 2014, 10:04 a.m.) > > > Review request for cloudstack and SrikanteswaraRao Talluri. > > > Bugs: cloudstack-6316 > https://issues.apache.org/jira/browse/cloudstack-6316 > > > Repository: cloudstack-git > > > Description > ------- > > 1. Earlier, i added a facility called DeleteDataCenter to clean up a datacenter, it has some issues for deleting, fixed them. This change is under deployDataCenter.py > 2. Earlier Added a new facility, where we can deploy multiple zones and run regression simultaneously on all zones with multiple hypervisors, > 2.1 So, user creates the data center with multiple zones and hypervisors. > 2.2 Provides switches like zone and hypervisor to marvinPlugin and all tests are run against that zone. With this patch, added hypervisor switch to nose\marvinplugin. > 3. Fixed an issue in one of the test suite under maint, it is using wait_for_vm function, which is not available. > 4. Added time taken for async job, post some stress scenarios, we want to collect logs and analyze the time taken for various commands, their jobid and time taken, added time taken > facility. > 5. Added pep8 changes for marvin directory > > > Diffs > ----- > > test/integration/component/maint/test_host_high_availability.py f81bff2 > tools/marvin/marvin/__init__.py 7af168e > tools/marvin/marvin/asyncJobMgr.py ee3ae5a > tools/marvin/marvin/cloudstackConnection.py 26116cc > tools/marvin/marvin/cloudstackException.py 3da28b7 > tools/marvin/marvin/cloudstackTestCase.py 303f7ad > tools/marvin/marvin/cloudstackTestClient.py 0dac5f4 > tools/marvin/marvin/codegenerator.py d4a81cf > tools/marvin/marvin/config/test_data.py 563bc07 > tools/marvin/marvin/configGenerator.py 605832c > tools/marvin/marvin/dbConnection.py ba7bac9 > tools/marvin/marvin/deployAndRun.py 56747a7 > tools/marvin/marvin/deployDataCenter.py d4a0b85 > tools/marvin/marvin/jsonHelper.py f7db5ba > tools/marvin/marvin/marvinInit.py 7ceba78 > tools/marvin/marvin/marvinLog.py f62c35a > tools/marvin/marvin/marvinPlugin.py 6525cd2 > tools/marvin/marvin/sandbox/advanced/advanced_env.py 1728e61 > tools/marvin/marvin/sandbox/advancedsg/advancedsg_env.py 9cf4a0a > tools/marvin/marvin/sandbox/basic/basic_env.py bf106fc > tools/marvin/marvin/sandbox/demo/simulator/simulator_setup.py d45d482 > tools/marvin/marvin/sshClient.py c24477c > tools/marvin/marvin/tcExecuteEngine.py c437337 > tools/marvin/marvin/testSetupSuccess.py 1701626 > > Diff: https://reviews.apache.org/r/19866/diff/ > > > Testing > ------- > > Tested running creating and deleting deployDataCenter. > > > Thanks, > > Santhosh Edukulla > > --===============0818346063049313637==--