Re: Review Request 13109: CLOUDSTACK-3594: Fixes related to affinity groups

2013-08-02 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13109/#review24513 --- Ship it! Ship It! - Prasanna Santhanam On July 31, 2013, 12:13

Re: Review Request 13109: CLOUDSTACK-3594: Fixes related to affinity groups

2013-08-01 Thread Prasanna Santhanam
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13109/#review24426 --- test/integration/component/test_affinity_groups.py

Re: Review Request 13109: CLOUDSTACK-3594: Fixes related to affinity groups

2013-08-01 Thread Gaurav Aradhye
On Aug. 1, 2013, 8:26 a.m., Prasanna Santhanam wrote: tools/marvin/marvin/integration/lib/base.py, line 574 https://reviews.apache.org/r/13109/diff/1/?file=331286#file331286line574 For every API in base, just use the prefix verb before the Cloud resource as the name of the

Re: Review Request 13109: CLOUDSTACK-3594: Fixes related to affinity groups

2013-08-01 Thread Gaurav Aradhye
On Aug. 1, 2013, 8:26 a.m., Prasanna Santhanam wrote: test/integration/component/test_affinity_groups.py, line 1444 https://reviews.apache.org/r/13109/diff/1/?file=331285#file331285line1444 use utils.randomgen() Currently the function random_gen is in utils only. It's not defined

Re: Review Request 13109: CLOUDSTACK-3594: Fixes related to affinity groups

2013-07-31 Thread Jenkins Cloudstack.org
--- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/13109/#review24346 --- Review 13109 failed the build test : FAILURE The url of build