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

    https://github.com/apache/cloudstack/pull/1579#discussion_r90236739
  
    --- Diff: api/src/com/cloud/network/Network.java ---
    @@ -215,6 +216,8 @@ public static Provider getProvider(String providerName) 
{
             public static final Capability DistributedRouter = new 
Capability("DistributedRouter");
             public static final Capability StretchedL2Subnet = new 
Capability("StretchedL2Subnet");
             public static final Capability RegionLevelVpc = new 
Capability("RegionLevelVpc");
    +        public static final Capability NoVlan = new Capability("NoVlan");
    +        public static final Capability PublicAccess = new 
Capability("PublicAccess");
    --- End diff --
    
    NoVlan is a Capability we're currently using when adding a new IP range to 
an existing shared network. Without this, the Broadcast URI would become 
vlan://untagged, while the shared network has a Broadcast URI of 
vsp://network-uuid. When the CIDR of the new ip range is the same as one of the 
already existing ip ranges, it would still not be allowed as the vlan id is 
different. 


---
If your project is set up for it, you can reply to this email and have your
reply appear on GitHub as well. If your project does not have this feature
enabled and wishes so, or if the feature is enabled but not working, please
contact infrastructure at infrastruct...@apache.org or file a JIRA ticket
with INFRA.
---

Reply via email to