----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/11708/ -----------------------------------------------------------
Review request for cloudstack and Devdeep Singh. Description ------- When multiple vlan ranges are added to a physical networks the vlan ranges displayed in the output of the listPhysicalNetworks api displays the vlan range in the order the ranges were added,Instead if they are displayed in the ascending order range this would make it easy for the end user. This addresses bug CLOUDSTACK-2167. Diffs ----- server/src/com/cloud/api/ApiResponseHelper.java bcc1605 Diff: https://reviews.apache.org/r/11708/diff/ Testing ------- The response of list api is now enhanced: <listphysicalnetworksresponse cloud-stack-version="4.2.0-SNAPSHOT"> <count>1</count> <physicalnetwork> <id>49e5cdfc-2c14-415a-9dd3-38ac2fdeef54</id> <name>Physical Network 1</name> <broadcastdomainrange>ZONE</broadcastdomainrange> <zoneid>0bd17058-2931-479b-98b5-29c8c91c24d3</zoneid> <state>Enabled</state> <vlan>480-504;910-914;916-918;920-923;925-934;936-940</vlan> <isolationmethods>VLAN</isolationmethods> </physicalnetwork> </listphysicalnetworksresponse> Build passes successfully. Thanks, Saksham Srivastava
