> On May 23, 2013, 6:14 a.m., Likitha Shetty wrote: > > test/integration/component/test_assign_vm.py, line 321 > > <https://reviews.apache.org/r/11307/diff/3/?file=295683#file295683line321> > > > > If a domain admin tries to move a VM to an account in a subdomain that > > belongs to a different domain, the request should fail. This is because a > > domain admin can only move VM's between 2 accounts that are in the domain > > path. > > So shouldn't the check condition of test_08 be changed?
As per https://cwiki.apache.org/confluence/display/CLOUDSTACK/Change+account+membership+Test+Plan this test condition should pass the vm move. Shall i change the TP and test check condition? - Ashutosh ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11307/#review20929 ----------------------------------------------------------- On May 28, 2013, 5:29 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, 5:29 a.m.) > > > Review request for cloudstack, Likitha Shetty, Prasanna Santhanam, 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 > >