Hi Shashank, Thanks for reviewing the patch! The main problem is we have two switch names defined currently in the documentation. I wanted to change it to have only one. Sorry I wasn't clear in the commit message. The idea is we instruct to create a switch named `OVS-Extended-Switch`. For a sysadmin typing that name later on would be annoying when they want to do operation on the switch. I.e.: if they want to check if the extension is enabled they would have to issue the following command: Get-VMSwitchExtension -VMSwitchName "OVS-Extended-Switch" I usually create a switch with the name = switch type so it is easy to distinguish which does what, but point taken, some can find that confusing. "ovsext-switch" is still long IMO and "ovsext" is the name of the running driver.
What about "ovs_switch" ? Response to the other comment inlined. Thanks, Alin. From: Shashank Ram [mailto:[email protected]] Sent: Sunday, February 5, 2017 9:21 AM To: Alin Serdean <[email protected]<mailto:[email protected]>>; [email protected]<mailto:[email protected]> Subject: Re: [PATCH 4/4] doc-windows: Consolidate switch names in documentation Hi Alin, I don't see a big value from this patch if I understand it correctly. Firstly, your commit is not clear about what it means when it says "typing it later on". Are you referring to typing it later on in the documentation? Secondly, since Hyper-V has different switch types such as "internal", "external", "private", replacing the switch name in the documentation with "external" might confuse readers unnecessarily, since its also a type. If you want to rename it, you could just call it something like "ovsext-switch" or just "ovsext". Please find the other comments inline. Thanks, Shashank - PS > New-VMSwitch "OVS-Extended-Switch" -NetAdapterName "Ethernet0" ` + PS > New-VMSwitch external -NetAdapterName "Ethernet0" ` -AllowManagementOS $false [SR]: The commit msgh does not mention anything about the "AllowManagementOS" flag being added. What's the reason for this? [Alin Serdean]: Sorry that was intended to be included in patch 1 _______________________________________________ dev mailing list [email protected] https://mail.openvswitch.org/mailman/listinfo/ovs-dev
