Re: Review Request: CS-2273: Automation: Change account membership for VMs

2013-05-29 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11307/#review21177 --- Ship it! Shipped with 85062a75ae363ec37d00cd1beb92a29b408cd238 - P

Re: Review Request: CS-2273: Automation: Change account membership for VMs

2013-05-28 Thread Ashutosh Kelkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11307/ --- (Updated May 28, 2013, 10:22 a.m.) Review request for cloudstack, Likitha Shett

Re: Review Request: CS-2273: Automation: Change account membership for VMs

2013-05-28 Thread Ashutosh Kelkar
> On May 28, 2013, 9:58 a.m., Likitha Shetty wrote: > > test/integration/component/test_assign_vm.py, line 382 > > > > > > Even the root disk snapshots are deleted. > > And since the VM will be successfully moved i

Re: Review Request: CS-2273: Automation: Change account membership for VMs

2013-05-28 Thread Likitha Shetty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11307/#review21084 --- test/integration/component/test_assign_vm.py

Re: Review Request: CS-2273: Automation: Change account membership for VMs

2013-05-28 Thread Ashutosh Kelkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11307/ --- (Updated May 28, 2013, 9:45 a.m.) Review request for cloudstack, Likitha Shetty

Re: Review Request: CS-2273: Automation: Change account membership for VMs

2013-05-28 Thread Ashutosh Kelkar
> On May 28, 2013, 8:22 a.m., Likitha Shetty wrote: > > test/integration/component/test_assign_vm.py, line 360 > > > > > > create_vm with pf=true has been implemented? Yes, VirtualMachine.create() checks for mode if m

Re: Review Request: CS-2273: Automation: Change account membership for VMs

2013-05-28 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11307/ --- (Updated May 28, 2013, 8:28 a.m.) Review request for cloudstack, Likitha Shetty

Re: Review Request: CS-2273: Automation: Change account membership for VMs

2013-05-28 Thread Likitha Shetty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11307/#review21081 --- First two comments have been repeated, please ignore the duplicates.

Re: Review Request: CS-2273: Automation: Change account membership for VMs

2013-05-28 Thread Likitha Shetty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11307/#review21077 --- test/integration/component/test_assign_vm.py

Re: Review Request: CS-2273: Automation: Change account membership for VMs

2013-05-27 Thread Likitha Shetty
> On May 23, 2013, 6:14 a.m., Likitha Shetty wrote: > > Ashutosh, are you referring to TP 8 in the Test-Plan? If yes, then the TP is still valid because in the TP the sub-domains (SubDomain1 and SubDomain2) that we are trying to move the VM’s between are under the same domain(Doamin1). But fro

Re: Review Request: CS-2273: Automation: Change account membership for VMs

2013-05-27 Thread Ashutosh Kelkar
> On May 23, 2013, 6:14 a.m., Likitha Shetty wrote: > > test/integration/component/test_assign_vm.py, line 321 > > > > > > If a domain admin tries to move a VM to an account in a subdomain that > > belongs to a differ

Re: Review Request: CS-2273: Automation: Change account membership for VMs

2013-05-27 Thread Ashutosh Kelkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11307/ --- (Updated May 28, 2013, 5:29 a.m.) Review request for cloudstack, Likitha Shetty

Re: Review Request: CS-2273: Automation: Change account membership for VMs

2013-05-22 Thread Likitha Shetty
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11307/#review20929 --- test/integration/component/test_assign_vm.py

Re: Review Request: CS-2273: Automation: Change account membership for VMs

2013-05-22 Thread Ashutosh Kelkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11307/ --- (Updated May 23, 2013, 3:56 a.m.) Review request for cloudstack, Likitha Shetty

Re: Review Request: CS-2273: Automation: Change account membership for VMs

2013-05-21 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11307/ --- (Updated May 21, 2013, 3 p.m.) Review request for cloudstack, Likitha Shetty, P

Re: Review Request: CS-2273: Automation: Change account membership for VMs

2013-05-21 Thread Ashutosh Kelkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11307/ --- (Updated May 21, 2013, 1:18 p.m.) Review request for cloudstack, Prasanna Santh

Review Request: CS-2273: Automation: Change account membership for VMs

2013-05-21 Thread Ashutosh Kelkar
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11307/ --- Review request for cloudstack, Prasanna Santhanam and Girish Shilamkar. Descrip