> 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 from what I can see, in case of this test condition the sub-domains(subdomain1 and subdomain2) that we are trying to move the VM’s between are under two different domains. Is that right? If we are trying to test TP 8 in this test then we can change line 155 from cls.sdomain_account_user2 = create_domain_account_user(cls.domain_account_user2['domain']) to cls.sdomain_account_user2 = create_domain_account_user(cls.domain_account_user1['domain']). With that change the test condition can pass the VM move. - Likitha ----------------------------------------------------------- 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 > >