> On Aug. 26, 2013, 10:09 a.m., Sanjay Tripathi wrote: > > test/integration/component/cpu_limits/test_project_limits.py, line 275 > > <https://reviews.apache.org/r/13001/diff/3/?file=338935#file338935line275> > > > > use listResourceLimits API to get the resource limits of respective > > account/domain.
Removed the configuration as it was not being used in the test case. The project limits are tested in maximum limits test cases. These test cases are used to test the resource count. > On Aug. 26, 2013, 10:09 a.m., Sanjay Tripathi wrote: > > test/integration/component/cpu_limits/test_project_limits.py, line 316 > > <https://reviews.apache.org/r/13001/diff/3/?file=338935#file338935line316> > > > > use listResourceLimits API to get the resource limit of respective > > account/domain. Same reason. - Gaurav ----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13001/#review25541 ----------------------------------------------------------- On Aug. 8, 2013, 12:51 p.m., Gaurav Aradhye wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/13001/ > ----------------------------------------------------------- > > (Updated Aug. 8, 2013, 12:51 p.m.) > > > Review request for cloudstack, Girish Shilamkar, sailaja mada, Sanjay > Tripathi, and Prasanna Santhanam. > > > Repository: cloudstack-git > > > Description > ------- > > Adding test cases for CPU limits from Limit Resources. > Marvin changes have to be picked up from Memory Limits patch > (https://reviews.apache.org/r/11626/). Those are not included in this patch. > > > Diffs > ----- > > test/integration/component/cpu_limits/test_cpu_limits.py PRE-CREATION > test/integration/component/cpu_limits/test_domain_limits.py PRE-CREATION > test/integration/component/cpu_limits/test_maximum_limits.py PRE-CREATION > test/integration/component/cpu_limits/test_project_limits.py PRE-CREATION > tools/marvin/marvin/integration/lib/base.py b5d086b > tools/marvin/marvin/integration/lib/common.py 4f5acef > > Diff: https://reviews.apache.org/r/13001/diff/ > > > Testing > ------- > > > Thanks, > > Gaurav Aradhye > >
