Review Request 25390: CLOUDSTACK-7500 - Add an addition simulated host for basic mode tests

2014-09-05 Thread John Dilley
--- Thanks, John Dilley

Re: Review Request 25235: vGPU service offering upgrade test automation

2014-09-04 Thread John Dilley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25235/#review52287 --- Ship it! Ship It! - John Dilley On Sept. 3, 2014, 6:12 p.m

Re: Review Request 24696: Only attempt to shrink volume in test if CLVM is in use

2014-09-02 Thread John Dilley
- test/integration/smoke/test_volumes.py e2e0d76 Diff: https://reviews.apache.org/r/24696/diff/ Testing --- Tested on XenServer to check that the test now passes. Thanks, John Dilley

Re: Review Request 24740: CLOUDSTACK-7354: Set isdynamicallyscalable before attempting to scale the VM

2014-09-02 Thread John Dilley
to scale the VM, so it had no effect. Diffs - test/integration/smoke/test_scale_vm.py b3804dc Diff: https://reviews.apache.org/r/24740/diff/ Testing --- Tested on VMWare Thanks, John Dilley

Re: Review Request 25235: vGPU service offering upgrade test automation

2014-09-02 Thread John Dilley
so that if the testcase fails, it's clear whether it also failed on the first service offering. - John Dilley On Sept. 1, 2014, 5:34 p.m., sailaja mada wrote: > > --- > This is an automatically generated e-mail. To reply,

RE: Unable to connect to management server on recent builds

2014-09-02 Thread John Dilley
by: Rohit Yadav -Original Message----- From: John Dilley [mailto:john.dil...@citrix.com] Sent: Tuesday, 2 September 2014 12:23 PM To: dev@cloudstack.apache.org Subject: Unable to connect to management server on recent builds Hi all, The latest build seems to have broken starting the managemen

Unable to connect to management server on recent builds

2014-09-01 Thread John Dilley
Hi all, The latest build seems to have broken starting the management server - I've raised https://issues.apache.org/jira/browse/CLOUDSTACK-7465 http://jenkins.buildacloud.org/job/simulator-singlerun/259/ has failed, and I'm assuming it's the same cause, but I can't see a lot of information on

Re: Review Request 24420: vGPU Test Automation ( Check for vGPU resources & VM lifecycle tests)

2014-08-26 Thread John Dilley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24420/#review51527 --- Ship it! Ship It! - John Dilley On Aug. 22, 2014, 2:09 p.m

Re: Review Request 25017: Disabling VPC tests on Hyper-v

2014-08-26 Thread John Dilley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/25017/#review51526 --- VPC is supported for Hyper-V in ACS - John Dilley On Aug. 26

Re: Review Request 24420: vGPU Test Automation ( Check for vGPU resources & VM lifecycle tests)

2014-08-22 Thread John Dilley
likely to be fragile - XenServer doesn't guarentee the order of the parameters. Future versions could add a parameter and break this test/integration/component/test_deploy_vgpu_vm.py <https://reviews.apache.org/r/24420/#comment89423> Give the test a different nam

Review Request 24740: CLOUDSTACK-7354: Set isdynamicallyscalable before attempting to scale the VM

2014-08-15 Thread John Dilley
/integration/smoke/test_scale_vm.py b3804dc Diff: https://reviews.apache.org/r/24740/diff/ Testing --- Tested on VMWare Thanks, John Dilley

Re: Regarding Shrinking of Volume

2014-08-14 Thread John Dilley
Or a new API listDiskOfferingsVolumeCanBeResizedTo. But probably with a more concise name :) John Sent from my Samsung Galaxy S5 - powered by Three Original message From: Mike Tutkowski Date:14/08/2014 17:43 (GMT+00:00) To: John Dilley Cc: dev@cloudstack.apache.org

Re: Review Request 24648: CLOUDSTACK-7310 - don't try and shink a disk in the volume resize test

2014-08-14 Thread John Dilley
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24648/#review50657 --- Don't submit this one, see review 24696 instead - John Dilley

Re: Review Request 24696: Only attempt to shrink volume in test if CLVM is in use

2014-08-14 Thread John Dilley
block. - John --- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24696/#review50572 --- On Aug. 14, 2014, 10:42 a.m

Re: Review Request 24696: Only attempt to shrink volume in test if CLVM is in use

2014-08-14 Thread John Dilley
ike this if xenserver: stop vm grow volume if possible to shrink: shrink volume if xenserver: start vm again to check it can still run - John Dilley On Aug. 14, 2014, 10:42 a.

RE: Regarding Shrinking of Volume

2014-08-14 Thread John Dilley
CLOUDSTACK/Resize+Data+Volumes > claims that XenServer can shrink data volumes. Can I edit it and remove >that claim from FS? Kindly let me know, > >Thank you, >Chandan. > >-Original Message- >From: Marcus [mailto:shadow...@gmail.com<mailto:shadow...@gmail

Review Request 24696: Only attempt to shrink volume in test if CLVM is in use

2014-08-14 Thread John Dilley
://reviews.apache.org/r/24696/diff/ Testing --- Tested on XenServer to check that the test now passes. Thanks, John Dilley

Review Request 24695: CLOUDSTACK-7346: Skip iSCSI primary storage test for VMWare

2014-08-14 Thread John Dilley
org/r/24695/diff/ Testing --- Thanks, John Dilley

Re: Regarding Shrinking of Volume

2014-08-13 Thread John Dilley
Hi, I'll put a condition around the block that does the shrinking only in the case of CLVM. Would the disk image type be "CLVM" instead of "qcow2"? Thanks, John > On 13 Aug 2014, at 20:39, "Mike Tutkowski" > wrote: > > This is how Marcus responded: > > Yes, this is true. Ideally other stor

Re: Review Request 24648: CLOUDSTACK-7310 - don't try and shink a disk in the volume resize test

2014-08-13 Thread John Dilley
. I have therefore removed it. Diffs - test/integration/smoke/test_volumes.py e2e0d76 Diff: https://reviews.apache.org/r/24648/diff/ Testing --- Thanks, John Dilley

Review Request 24648: CLOUDSTACK-7310 - don't try and shink a disk in the volume resize test

2014-08-13 Thread John Dilley
- test/integration/smoke/test_volumes.py e2e0d76 Diff: https://reviews.apache.org/r/24648/diff/ Testing --- Thanks, John Dilley

Re: Review Request 24378: CLOUDSTACK-7268: Ignore "already exists" error in createEgressFirewallRule

2014-08-07 Thread John Dilley
(updated) - tools/marvin/marvin/lib/base.py 3a1f7e6 Diff: https://reviews.apache.org/r/24378/diff/ Testing --- Tested on KVM advanced zone Thanks, John Dilley

Re: Review Request 24378: CLOUDSTACK-7268: Ignore "already exists" error in createEgressFirewallRule

2014-08-07 Thread John Dilley
o reply, visit: https://reviews.apache.org/r/24378/#review49877 ------- On Aug. 7, 2014, 7:41 a.m., John Dilley wrote: > > --- > This is an automatically

Re: Review Request 24378: CLOUDSTACK-7268: Ignore "already exists" error in createEgressFirewallRule

2014-08-06 Thread John Dilley
- tools/marvin/marvin/lib/base.py 3a1f7e6 Diff: https://reviews.apache.org/r/24378/diff/ Testing --- Tested on KVM advanced zone Thanks, John Dilley

Review Request 24383: CLOUDSTACK-7271: Accept any hypervisor in error message for integration.smoke.test_deploy_vm_root_resize.TestDeployVM.test_00_deploy_vm_root_resize

2014-08-06 Thread John Dilley
/integration/smoke/test_deploy_vm_root_resize.py 029e7db Diff: https://reviews.apache.org/r/24383/diff/ Testing --- Tested on VMWare Thanks, John Dilley

Re: Review Request 23982: CLOUDSTACK-7192: Skip tests on Hyper-V which don't apply

2014-08-06 Thread John Dilley
d1 test/integration/smoke/test_vm_snapshots.py 01626f5 test/integration/smoke/test_volumes.py 6e9ea75 Diff: https://reviews.apache.org/r/23982/diff/ Testing --- Thanks, John Dilley

Review Request 24378: CLOUDSTACK-7268: Ignore "already exists" error in createEgressFirewallRule

2014-08-06 Thread John Dilley
/base.py 3a1f7e6 Diff: https://reviews.apache.org/r/24378/diff/ Testing --- Tested on KVM advanced zone Thanks, John Dilley

Review Request 24304: CLOUDSTACK-7247: Default hypervisor to None in deployVirtualMachine call in marvin base.py

2014-08-05 Thread John Dilley
24304/diff/ Testing --- In progress on VMWare Thanks, John Dilley

Re: Review Request 24182: CLOUDSTACK-7223 Wait for System VM agent state to be connected before verication

2014-08-04 Thread John Dilley
This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/24182/#review49455 --- On Aug. 4, 2014, 2:50 p.m., John Dilley wrote: > > ---

Re: Review Request 24182: CLOUDSTACK-7223 Wait for System VM agent state to be connected before verication

2014-08-04 Thread John Dilley
/destroy system VMs before verifying that they are functioning properly. Diffs (updated) - test/integration/smoke/test_ssvm.py ac4079e Diff: https://reviews.apache.org/r/24182/diff/ Testing --- Tested against KVM advanced zone Thanks, John Dilley

Re: Review Request 24182: CLOUDSTACK-7223 Wait for System VM agent state to be connected before verication

2014-08-03 Thread John Dilley
properly. Diffs - test/integration/smoke/test_ssvm.py ac4079e Diff: https://reviews.apache.org/r/24182/diff/ Testing --- Tested against KVM advanced zone Thanks, John Dilley

Re: Review Request 24182: CLOUDSTACK-7223 Wait for System VM agent state to be connected before verication

2014-08-01 Thread John Dilley
advanced zone Thanks, John Dilley

Review Request 24182: CLOUDSTACK-7223 Wait for System VM agent state to be connected before verication

2014-08-01 Thread John Dilley
/integration/smoke/test_ssvm.py ac4079e Diff: https://reviews.apache.org/r/24182/diff/ Testing --- Tested against KVM advanced zone Thanks, John Dilley

Review Request 23982: CLOUDSTACK-7192: Skip tests on Hyper-V which don't apply

2014-07-28 Thread John Dilley
ts.py 8b6fdd1 test/integration/smoke/test_vm_snapshots.py 01626f5 test/integration/smoke/test_volumes.py 6e9ea75 Diff: https://reviews.apache.org/r/23982/diff/ Testing --- Thanks, John Dilley

Review Request 23976: CLOUDSTACK-7190: Fix Hyper-V issue when attempting SSH to System VMs

2014-07-28 Thread John Dilley
/component/test_persistent_networks.py b90d621 test/integration/smoke/test_routers.py f77d11a test/integration/smoke/test_ssvm.py 9b8d19d Diff: https://reviews.apache.org/r/23976/diff/ Testing --- Thanks, John Dilley

Re: Review Request 23169: CLOUDSTACK-7010: Add domain to createZones in marvin

2014-07-08 Thread John Dilley
c097238 Diff: https://reviews.apache.org/r/23169/diff/ Testing --- Tested deployDataCenter, both with and without specifying a domain Thanks, John Dilley

Re: Review Request 23169: CLOUDSTACK-7010: Add domain to createZones in marvin

2014-07-08 Thread John Dilley
> As well, please test it on both 4.4-forward and master, if it need to > > go to both? > > John Dilley wrote: > > Does cfg, files used has these option of domain? > > Hi Santhosh, > > Not quite sure what you mean here - can you clarify? &

Re: Review Request 23169: CLOUDSTACK-7010: Add domain to createZones in marvin

2014-07-07 Thread John Dilley
> As well, please test it on both 4.4-forward and master, if it need to > > go to both? > > John Dilley wrote: > > Does cfg, files used has these option of domain? > > Hi Santhosh, > > Not quite sure what you mean here - can you clarify? &

Re: Review Request 23169: CLOUDSTACK-7010: Add domain to createZones in marvin

2014-07-07 Thread John Dilley
view47065 --- On June 30, 2014, 2:30 p.m., John Dilley wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://revie

Re: Review Request 23169: CLOUDSTACK-7010: Add domain to createZones in marvin

2014-07-07 Thread John Dilley
This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/23169/#review47065 ------- On June 30, 2014, 2:30 p.m., John Dilley wrote: > > ---

Review Request 23169: CLOUDSTACK-7010: Add domain to createZones in marvin

2014-06-30 Thread John Dilley
specifying a domain Thanks, John Dilley

Re: Review Request 22554: CLOUDSTACK-6909 - fix marvin's handling of SMB credentials for storage

2014-06-25 Thread John Dilley
enerated e-mail. To reply, visit: https://reviews.apache.org/r/22554/#review46495 --- On June 23, 2014, 3:53 p.m., John Dilley wrote: > > --- > This is an automatically generated e-mail. To re

Re: Review Request 22554: CLOUDSTACK-6909 - fix marvin's handling of SMB credentials for storage

2014-06-23 Thread John Dilley
} ], "clustertype":"CloudManaged", "hosts":[ { "url":"http://10.220.163.45";, "username":"root", "password":"xenroot" } ], "XRT_Hosts":1 } ], "gateway":"10.220.160.1" } ], "internaldns1":"10.220.160.11", "secondaryStorages":[ { "url":"cifs://10.220.163.45/storage/secondary", "details":{ "domain":"XSQA", "password":"xenroot01T", "user":"Administrator" }, "provider":"SMB" } ] } ], "mgtSvr":[ { "mgtSvrIp":"10.220.137.39", "passwd":"xenroot", "port":8096, "user":"root" } ] } Thanks, John Dilley

Re: Review Request 22554: CLOUDSTACK-6909 - fix marvin's handling of SMB credentials for storage

2014-06-23 Thread John Dilley
Managed", "hosts":[ { "url":"http://10.220.163.45";, "username":"root", "password":"xenroot" } ], "XRT_Hosts":1 } ], "gateway":"10.220.160.1" } ], "internaldns1":"10.220.160.11", "secondaryStorages":[ { "url":"cifs://10.220.163.45/storage/secondary", "details":{ "domain":"XSQA", "password":"xenroot01T", "user":"Administrator" }, "provider":"SMB" } ] } ], "mgtSvr":[ { "mgtSvrIp":"10.220.137.39", "passwd":"xenroot", "port":8096, "user":"root" } ] } Thanks, John Dilley

Re: Review Request 22554: CLOUDSTACK-6909 - fix marvin's handling of SMB credentials for storage

2014-06-18 Thread John Dilley
--- On June 13, 2014, 4:53 p.m., John Dilley wrote: > > --- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/22554/ > -

Re: Review Request 22554: CLOUDSTACK-6909 - fix marvin's handling of SMB credentials for storage

2014-06-13 Thread John Dilley
CloudManaged", "hosts":[ { "url":"http://10.220.163.45";, "username":"root", "password":"xenroot" } ], "XRT_Hosts":1 } ], "gateway":"10.220.160.1" } ], "internaldns1":"10.220.160.11", "secondaryStorages":[ { "url":"cifs://10.220.163.45/storage/secondary", "details":{ "domain":"XSQA", "password":"xenroot01T", "user":"Administrator" }, "provider":"SMB" } ] } ], "mgtSvr":[ { "mgtSvrIp":"10.220.137.39", "passwd":"xenroot", "port":8096, "user":"root" } ] } Thanks, John Dilley

Review Request 22554: CLOUDSTACK-6909 - fix marvin's handling of SMB credentials for storage

2014-06-13 Thread John Dilley
t;domain":"XSQA" } ] } ], "clustertype":"CloudManaged", "hosts":[ { "url":"http://10.220.163.45";, "username":"root", "password":"xenroot" } ], "XRT_Hosts":1 } ], "gateway":"10.220.160.1" } ], "internaldns1":"10.220.160.11", "secondaryStorages":[ { "url":"cifs://10.220.163.45/storage/secondary", "details":{ "domain":"XSQA", "password":"xenroot01T", "user":"Administrator" }, "provider":"SMB" } ] } ], "mgtSvr":[ { "mgtSvrIp":"10.220.137.39", "passwd":"xenroot", "port":8096, "user":"root" } ] } Thanks, John Dilley