hi Alin, I second Shashank’s thoughts. I prefer the name “OVS-Extended-Switch”, it makes documentation explicit. I’d prefer to keep it that way.
I’m sure that once users are familiar with the documentation, they can pretty much pick whatever name they want. Thanks, -- Nithin On Feb 4, 2017, at 11:20 PM, Shashank Ram <r...@vmware.com<mailto:r...@vmware.com>> wrote: 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 ________________________________ From: ovs-dev-boun...@openvswitch.org<mailto:ovs-dev-boun...@openvswitch.org> <ovs-dev-boun...@openvswitch.org<mailto:ovs-dev-boun...@openvswitch.org>> on behalf of Alin Serdean <aserd...@cloudbasesolutions.com<mailto:aserd...@cloudbasesolutions.com>> Sent: Saturday, February 4, 2017 12:21 AM To: d...@openvswitch.org<mailto:d...@openvswitch.org> Subject: [ovs-dev] [PATCH 4/4] doc-windows: Consolidate switch names in documentation Use shorter name for the Hyper-V vSwitch for simplicity of typing it later on. Signed-off-by: Alin Gabriel Serdean <aserd...@cloudbasesolutions.com<mailto:aserd...@cloudbasesolutions.com>> --- Documentation/intro/install/windows.rst | 8 ++++---- 1 file changed, 4 insertions(+), 4 deletions(-) diff --git a/Documentation/intro/install/windows.rst b/Documentation/intro/install/windows.rst index ece207d..2341b5c 100644 --- a/Documentation/intro/install/windows.rst +++ b/Documentation/intro/install/windows.rst @@ -274,12 +274,12 @@ Enforcement' during boot. The following commands can be used: In the Virtual Switch Manager configuration you can enable the Open vSwitch Extension on an existing switch or create a new switch. -The command to create a new switch named 'OVS-Extended-Switch' using a physical -NIC named 'Ethernet0' is: +The command to create a new switch named 'external' using a physical NIC named +'Ethernet0' is: .. code-block:: ps1con - 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? .. note:: @@ -293,7 +293,7 @@ An alternative way to do the same is to run the following command: .. code-block:: ps1con - PS > Enable-VMSwitchExtension "Open vSwitch Extension" OVS-Extended-Switch + PS > Enable-VMSwitchExtension "Open vSwitch Extension" external .. note:: -- 2.10.2.windows.1 _______________________________________________ dev mailing list d...@openvswitch.org<mailto:d...@openvswitch.org> https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.org_mailman_listinfo_ovs-2Ddev&d=DwICAg&c=uilaK90D4TOVoH58JNXRgQ&r=6OuVHk-mnufSWzkKa74UkQ&m=ANt1MyWGWPCfbl8o7d8EK5wC138sS4Uuzm8XX4IsouA&s=WtqG4fs_RRxuNxJrrZZ7cK19KcdlqwhuzYpesaRapeE&e= _______________________________________________ dev mailing list d...@openvswitch.org https://urldefense.proofpoint.com/v2/url?u=https-3A__mail.openvswitch.org_mailman_listinfo_ovs-2Ddev&d=DwICAg&c=uilaK90D4TOVoH58JNXRgQ&r=-xl6DPE_Y3uQD-mpZD7osBo2iL4s3jwdmSjTlGgjlsQ&m=ZJZdXeUyJK0yT5jDOnWAeAQaGx-FHZF4KNy12XRtQaM&s=3N8RJlWhGyZ1TYeGFMgqeb57WFfsO10eg2H7J2erhg0&e= _______________________________________________ dev mailing list d...@openvswitch.org https://mail.openvswitch.org/mailman/listinfo/ovs-dev