----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11307/#review21084 -----------------------------------------------------------
test/integration/component/test_assign_vm.py <https://reviews.apache.org/r/11307/#comment43623> Got it, thanks test/integration/component/test_assign_vm.py <https://reviews.apache.org/r/11307/#comment43622> All snapshots(of both root and attached volumes)are deleted. And the since the VM will be successfully moved shouldn't test condition be different? test/integration/component/test_assign_vm.py <https://reviews.apache.org/r/11307/#comment43624> Even the root disk snapshots are deleted. And since the VM will be successfully moved if both the domains are under the same domain admin shouldn't the test condition change? test/integration/component/test_assign_vm.py <https://reviews.apache.org/r/11307/#comment43627> Oh that makes sense. If the resource type is snapshot shouldn't the value of resourceType parameter be '3' for updateResourceLimit API? I am a little confused. So the test case is that we reduce the snapshot resource limit of subdomain2 to 1 and try to move a VM with 1 snapshot to subdomain1? - Likitha Shetty On May 28, 2013, 9:45 a.m., Ashutosh Kelkar wrote: > > ----------------------------------------------------------- > 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, Prasanna Santhanam, Parth > Jagirdar, and Girish Shilamkar. > > > Description > ------- > > Testcases for changing account membership for vas > > > This addresses bug CLOUDSTACK-2273. > > > Diffs > ----- > > test/integration/component/test_assign_vm.py PRE-CREATION > tools/marvin/marvin/integration/lib/base.py f3a96bd > > Diff: https://reviews.apache.org/r/11307/diff/ > > > Testing > ------- > > > Thanks, > > Ashutosh Kelkar > >