cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "Ashutosh Kelkar" <ashut...@clogeny.com>
Subject Re: Review Request 13523: 7 New tests for affinity groups
Date Thu, 15 Aug 2013 04:48:02 GMT


> On Aug. 14, 2013, 2:46 p.m., Prasanna Santhanam wrote:
> > test/integration/component/test_affinity_groups.py, line 507
> > <https://reviews.apache.org/r/13523/diff/1/?file=340307#file340307line507>
> >
> >     I didn't quite understand this assert. What hostid are we talking about?

The host id is from the VM we created and associated with the group. The test ensures the
listing of the group also lists the host id of the vms associated with that affinity group.
Updated the assert to better reflect the check being carried out.


> On Aug. 14, 2013, 2:46 p.m., Prasanna Santhanam wrote:
> > test/integration/component/test_affinity_groups.py, line 1594
> > <https://reviews.apache.org/r/13523/diff/1/?file=340307#file340307line1594>
> >
> >     Can we be explicit? 
> >     "Missing affinity group %s .."

Updated assert to make the check more clear


> On Aug. 14, 2013, 2:46 p.m., Prasanna Santhanam wrote:
> > test/integration/component/test_affinity_groups.py, line 1636
> > <https://reviews.apache.org/r/13523/diff/1/?file=340307#file340307line1636>
> >
> >     remove multihost tag

Dropped multihost tag from test


> On Aug. 14, 2013, 2:46 p.m., Prasanna Santhanam wrote:
> > test/integration/component/test_affinity_groups.py, line 1659
> > <https://reviews.apache.org/r/13523/diff/1/?file=340307#file340307line1659>
> >
> >     "%s affinity groups listed by admin with id %s. Expected 1" % (count, id)
> >     
> >     ^ This is much better for debugging and takes the form of expected against observed.

Updated assert to be inline with review comment


> On Aug. 14, 2013, 2:46 p.m., Prasanna Santhanam wrote:
> > test/integration/component/test_affinity_groups.py, line 1688
> > <https://reviews.apache.org/r/13523/diff/1/?file=340307#file340307line1688>
> >
> >     Improve the assertion to show expected vs observed.

Updated assert to be inline with review comment.


- Ashutosh


-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
https://reviews.apache.org/r/13523/#review25146
-----------------------------------------------------------


On Aug. 15, 2013, 4:41 a.m., Ashutosh Kelkar wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/13523/
> -----------------------------------------------------------
> 
> (Updated Aug. 15, 2013, 4:41 a.m.)
> 
> 
> Review request for cloudstack, Girish Shilamkar and Prasanna Santhanam.
> 
> 
> Repository: cloudstack-git
> 
> 
> Description
> -------
> 
> New tests for affinity groups
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_affinity_groups.py 39baf3e 
> 
> Diff: https://reviews.apache.org/r/13523/diff/
> 
> 
> Testing
> -------
> 
> 
> Thanks,
> 
> Ashutosh Kelkar
> 
>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message