Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-08-09 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11626/ --- (Updated Aug. 9, 2013, 7:51 a.m.) Review request for cloudstack, sailaja mada,

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-08-26 Thread Sanjay Tripathi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11626/#review25218 --- test/integration/component/memory_limits/test_domain_limits.py

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-08-26 Thread Sanjay Tripathi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11626/#review25546 --- Can you please remove the trailing white spaces from the patch so th

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-08-29 Thread Gaurav Aradhye
> On June 17, 2013, 10:32 a.m., Prasanna Santhanam wrote: > > test/integration/component/test_memory_limits.py, line 184 > > > > > > what if networks is an empty list? [] Removed this method as it is no longer used an

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-08-29 Thread Gaurav Aradhye
> On Aug. 1, 2013, 4:57 p.m., Sanjay Tripathi wrote: > > test/integration/component/memory_limits/test_domain_limits.py, line 302 > > > > > > Migration of a VM to another host will not affect/update any count > > beca

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-08-29 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11626/ --- (Updated Aug. 29, 2013, 1:02 p.m.) Review request for cloudstack, sailaja mada,

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-08-29 Thread Sanjay Tripathi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11626/#review25723 --- Diff for common.py is not available in the patch, update the patch a

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-08-29 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11626/ --- (Updated Aug. 29, 2013, 4:24 p.m.) Review request for cloudstack, sailaja mada,

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-08-29 Thread Sanjay Tripathi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11626/#review25788 --- Ship it! Looks good to me. - Sanjay Tripathi On Aug. 29, 2013, 1

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-09-12 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11626/ --- (Updated Sept. 12, 2013, 2:53 p.m.) Review request for cloudstack, sailaja mada

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-09-12 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11626/ --- (Updated Sept. 12, 2013, 3:49 p.m.) Review request for cloudstack, sailaja mada

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-09-13 Thread sailaja mada
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11626/#review26076 --- Ship it! Ship It! - sailaja mada On Sept. 12, 2013, 10:19 a.m.,

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-09-16 Thread sailaja mada
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11626/#review26136 --- Ship it! https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;a

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-07-26 Thread Gaurav Aradhye
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11626/ --- (Updated July 26, 2013, 11:27 a.m.) Review request for cloudstack and Prasanna

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-07-26 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11626/#review23945 --- Review 11626 PASSED the build test The url of build cloudstack-maste

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-07-27 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11626/ --- (Updated July 27, 2013, 1:21 p.m.) Review request for cloudstack, sailaja mada,

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-08-01 Thread Sanjay Tripathi
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11626/#review24360 --- test/integration/component/memory_limits/test_domain_limits.py

Re: Review Request 11626: Automation: Memory limits tests from Resource Limits Tests

2013-08-01 Thread Sanjay Tripathi
> On Aug. 1, 2013, 11:27 a.m., Sanjay Tripathi wrote: > > test/integration/component/memory_limits/test_domain_limits.py, line 429 > > > > > > use listAccounts and assert to verify count. Also, can you close the issue