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

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_r62503852
  
    --- Diff: 
server/src/com/cloud/network/router/VirtualNetworkApplianceManagerImpl.java ---
    @@ -1588,6 +1588,16 @@ protected StringBuilder 
createGuestBootLoadArgs(final NicProfile guestNic, final
             return buf;
         }
     
    +    /**
    +     * 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