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

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

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-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-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

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 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: > > ---

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
> 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-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-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

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

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 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

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

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-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-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: > > ---

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

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

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 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 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

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-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

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 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

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

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

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

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

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: 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 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: 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

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: 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

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-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

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: 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

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: 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 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 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

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

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