[ 
https://issues.apache.org/jira/browse/CLOUDSTACK-9165?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=15276400#comment-15276400
 ] 

ASF GitHub Bot commented on CLOUDSTACK-9165:
--------------------------------------------

Github user nlivens commented on a diff in the pull request:

    https://github.com/apache/cloudstack/pull/1246#discussion_r62503776
  
    --- Diff: server/src/com/cloud/network/guru/GuestNetworkGuru.java ---
    @@ -396,6 +396,16 @@ public NicProfile allocate(final Network network, 
NicProfile nic, final VirtualM
             return nic;
         }
     
    +    /**
    +     * Return a string representing network Cidr for the specifeid network
    +     * @param guestNetwork
    +     * @return valid network Cidr for the specified network
    +     */
    +    protected String getValidNetworkCidr(Network guestNetwork){
    --- End diff --
    
    I think you should extract this method to another common class (e.g. 
NetworkModel) since this method can be reused over the different classes.


> unable to use reserved IP range in a network for external VMs
> -------------------------------------------------------------
>
>                 Key: CLOUDSTACK-9165
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-9165
>             Project: CloudStack
>          Issue Type: Bug
>      Security Level: Public(Anyone can view this level - this is the 
> default.) 
>            Reporter: sudharma jain
>




--
This message was sent by Atlassian JIRA
(v6.3.4#6332)

Reply via email to