cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From "ASF Subversion and Git Services" <asf...@urd.zones.apache.org>
Subject Re: Review Request: CLOUDSTACK-702: Tests for Multiple IP Ranges
Date Fri, 21 Jun 2013 17:01:48 GMT

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


Commit a6a49490fbcc344f8a37d8c2ef0254da3ae39d89 in branch refs/heads/master from Sheng Yang
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=a6a4949 ]

Fix baremetal functionality

1. Baremetal doesn't have secondary storage, so we don't need check them.

2. The new "AddBaremetalHostCmd" hasn't been used by UI, so keep the validity
checking out for now. "AddHostCmd" would still works.

3. Baremetal haven't implemented multiple ip range feature(CLOUDSTACK-702),
return true for now for single range.


- ASF Subversion and Git Services


On May 9, 2013, 1:23 p.m., sanjeev n wrote:
> 
> -----------------------------------------------------------
> This is an automatically generated e-mail. To reply, visit:
> https://reviews.apache.org/r/11026/
> -----------------------------------------------------------
> 
> (Updated May 9, 2013, 1:23 p.m.)
> 
> 
> Review request for cloudstack, Prasanna Santhanam and SrikanteswaraRao Talluri.
> 
> 
> Description
> -------
> 
> CLOUDSTACK-702: Tests for Multiple IP Ranges
> 1. Add ip range super set to existing CIDR
> 2. Add ip range subset to existing CIDR
> 
> 
> This addresses bug CLOUDSTACK-702.
> 
> 
> Diffs
> -----
> 
>   test/integration/component/test_multiple_ip_ranges.py 7e9eeb1 
> 
> Diff: https://reviews.apache.org/r/11026/diff/
> 
> 
> Testing
> -------
> 
> yes
> 
> 
> Thanks,
> 
> sanjeev n
> 
>


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