Hugo, Murali, Thank you for the explanation.
Jessica From: Murali Reddy Sent: Tuesday, September 18, 2012 4:06 AM To: Hugo Trippaers; Jessica Wang; [email protected] Cc: Alena Prokharchyk; Sangeetha Hariharan Subject: Re: when OVS tunnel manager is not used, isolationMethods parameter passed to createPhysicalNetwork API is being ignored. (CLOUDSTACK-129) introduce listIsolationMethods API I opened this bug for listing supported isolation methods. This should avoid any assumptions in the UI. From: Hugo Trippaers <[email protected]<mailto:[email protected]>> Date: Tuesday, 18 September 2012 3:04 AM To: Jessica Wang <[email protected]<mailto:[email protected]>> Cc: Murali Reddy <[email protected]<mailto:[email protected]>>, Alena Prokharchyk <[email protected]<mailto:[email protected]>>, Sangeetha Hariharan <[email protected]<mailto:[email protected]>>, "[email protected]<mailto:[email protected]>" <[email protected]<mailto:[email protected]>> Subject: Re: when OVS tunnel manager is not used, isolationMethods parameter passed to createPhysicalNetwork API is being ignored. Hey Jessica, The drops own is now also used to select the STT isolation type. This one does not depend on any flags and is an implementation that is separate from the OVS implementation. The end user should alway be allowed to select either vlan isolation or STT based isolation and later configure the Nicira support on the physical network. So I removed the flag to check for OVS support at that is no longer the only implementation of SDN that is supported by CloudStack. Hope this explains the change. Cheers, Hugo Verstuurd vanaf mijn iPad Op 17 sep. 2012 om 20:58 heeft "Jessica Wang" <[email protected]<mailto:[email protected]>> het volgende geschreven: Hugo, Murali, Your UI change at 2012-07-11 ("Phase 1 of Nicira integration") made Zone Wizard skip the checking of global setting "sdn.ovs.controller" and show Isolation Method dropdown regardless of value of global setting "sdn.ovs.controller". Is there any specific reason to do so? We use value of global setting "sdn.ovs.controller" (true/false) to determine whether to show Isolation Method dropdown during Advanced zone creation. Because when OVS tunnel manager is not used (i.e. when "sdn.ovs.controller"=false), isolationMethods parameter passed to createPhysicalNetwork API is being ignored. Thus, Isolation Method dropdown shouldn't show when "sdn.ovs.controller"=false. Jessica ================================================================================================ Author: Hugo Trippaers <[email protected]<mailto:[email protected]>> 2012-07-11 16:14:28 Committer: Murali Reddy <[email protected]<mailto:[email protected]>> 2012-07-11 17:16:06 Parent: 968752071291f3cc47b86ff67289d7cfc7cac3dd (applying Chip Childer's patches for license headers in the ui directory) Child: 1075344f03e836de5b11222aaf68b90bdd4a9531 (CS-15353: Larger click boxes in UI.) Branches: autoscale, master, remotes/cloudcom/asf_autoscale, remotes/origin/4.0, remotes/origin/asf-4.0, remotes/origin/autoscale, remotes/origin/deps-ctrl, remotes/origin/gradle, remotes/origin/javelin, remotes/origin/master, remotes/origin/maven, remotes/origin/maven-waf, remotes/origin/rbd, remotes/origin/storagerefactor, remotes/origin/tampa, remotes/origin/vm-snapshot, remotes/origin/vpc, vpc Follows: tag-3.0.1-prerelease-1 Precedes: Phase 1 of Nicira integration =================================================================================================
