> On Aug. 1, 2013, 4:57 p.m., Sanjay Tripathi wrote: > > test/integration/component/memory_limits/test_domain_limits.py, line 302 > > <https://reviews.apache.org/r/11626/diff/2/?file=328254#file328254line302> > > > > Migration of a VM to another host will not affect/update any count > > because VM still belongs to the same account; instead of doing migration > > test, you should test assignVM to other account in which VM get assigned to > > other account and affects/updates the count for both the accounts.
Will add separate test for assigning vm. This test case is targeted at migrate scenario. - Gaurav ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11626/#review24360 ----------------------------------------------------------- On Aug. 9, 2013, 1:21 p.m., Gaurav Aradhye wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/11626/ > ----------------------------------------------------------- > > (Updated Aug. 9, 2013, 1:21 p.m.) > > > Review request for cloudstack, sailaja mada, Sanjay Tripathi, and Prasanna > Santhanam. > > > Repository: cloudstack-git > > > Description > ------- > > Adding resource limit tests related to Memory. Changes suggested in CPU > resource limit tests which are applicable here are incorporated too. > > Updated test plan is available here: > https://cwiki.apache.org/confluence/download/attachments/30757590/LimitResourcesTestPlanUpdate5.xlsx?version=1&modificationDate=1366952352000 > > > Diffs > ----- > > test/integration/component/memory_limits/test_domain_limits.py PRE-CREATION > test/integration/component/memory_limits/test_maximum_limits.py > PRE-CREATION > test/integration/component/memory_limits/test_memory_limits.py PRE-CREATION > test/integration/component/memory_limits/test_project_limits.py > PRE-CREATION > tools/marvin/marvin/integration/lib/base.py 695edf7 > tools/marvin/marvin/integration/lib/common.py 4f5acef > > Diff: https://reviews.apache.org/r/11626/diff/ > > > Testing > ------- > > > Thanks, > > Gaurav Aradhye > >
