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

ASF subversion and git services commented on CLOUDSTACK-3753:
-------------------------------------------------------------

Commit 4f51d21908a17d6b46ea0dfd1a3d5fa344a14dec in branch refs/heads/master 
from [~jessicawang]
[ https://git-wip-us.apache.org/repos/asf?p=cloudstack.git;h=4f51d21 ]

CLOUDSTACK-3753: UI > Infrastructure > zone detail > physical network > guest > 
(1) remove Delete VLAN Range action. (2) change Add VLAN Range action to pass 
new VLAN range + exisitng VLAN range(s) to API. This is corresponding UI change 
to updatePhysicalNetwork API change that removevlan property has been removed 
and definition of vlan property has been changed.

                
> Multiple VLAN range API need to accept a list rather than "add" or "remove" 
> per command
> ---------------------------------------------------------------------------------------
>
>                 Key: CLOUDSTACK-3753
>                 URL: https://issues.apache.org/jira/browse/CLOUDSTACK-3753
>             Project: CloudStack
>          Issue Type: Bug
>      Security Level: Public(Anyone can view this level - this is the 
> default.) 
>          Components: Network Controller
>    Affects Versions: 4.2.0
>            Reporter: Sheng Yang
>            Assignee: Jessica Wang
>            Priority: Blocker
>             Fix For: 4.2.0
>
>
> https://cwiki.apache.org/confluence/display/CLOUDSTACK/Non+contiguous+vlan+ranges
> "vlan parameter will add a new vlan range to the existing vlan range (new 
> behavior). if the new vlan range overlaps the existing vlan range it will 
> extend that vlan (This is the existing behavior.).
> removevlan parameter will remove the mentioned vlan. The removevlan and vlan 
> parameters can be used together. If the vlan range we are trying to remove is 
> in use, the operation will not succeed."
> I haven't seen such API in the CloudStack. It's much more clean to use a list 
> here(even for ranges, can be extended from one element, like "1-2, 4-5, 
> 6-7"), rather than newly defined "vlan"/"removevlan" behavior. It broke API 
> compatibility(e.g. the previous API potentially able to change vlan directly, 
> rather than add it), and is also user unfriendly.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators
For more information on JIRA, see: http://www.atlassian.com/software/jira

Reply via email to