From 6fc98a84511574da9db82167cd350447e7e67367 Mon Sep 17 00:00:00 2001
From: radhikap <radhika.puthiyetath@citrix.com>
Date: Tue, 27 Aug 2013 14:17:15 +0530
Subject: [PATCH] CLOUDSTACK-772 review comments on DVS fixed

---
 docs/en-US/vmware-cluster-config-dvswitch.xml | 57 ++++++++++++-------
 1 file changed, 36 insertions(+), 21 deletions(-)

diff --git a/docs/en-US/vmware-cluster-config-dvswitch.xml b/docs/en-US/vmware-cluster-config-dvswitch.xml
index a49a1a58c52..311673a0e09 100644
--- a/docs/en-US/vmware-cluster-config-dvswitch.xml
+++ b/docs/en-US/vmware-cluster-config-dvswitch.xml
@@ -78,8 +78,16 @@
     <para>Use this VDS name when you specify the switch name in the traffic label while creating the
       zone. Traffic label format is [["Name of vSwitch/dvSwitch/EthernetPortProfile"][,"VLAN
       ID"[,"vSwitch Type"]]]</para>
-    <para>possible values for networkLabel, "" (empty string) dvSwitch0 dvSwitch0,200
-      dvSwitch0,300,vmwaredvs myEthernetPortProfile,,nexusdvs dvSwitch0,,vmwaredvs</para>
+    <para>The possible values for traffic labels are:</para>
+    <itemizedlist>
+      <listitem><para>empty string</para></listitem>
+      <listitem><para>dvSwitch0</para></listitem>
+      <listitem><para> dvSwitch0,200</para></listitem>
+      <listitem><para>dvSwitch1,300,vmwaredvs</para></listitem>
+      <listitem><para>myEthernetPortProfile,,nexusdvs</para></listitem>
+      <listitem><para>dvSwitch0,,vmwaredvs</para></listitem>
+    </itemizedlist>
+    
     <mediaobject>
       <imageobject>
         <imagedata fileref="./images/traffic-label.png"/>
@@ -105,15 +113,17 @@
             <entry><para>1</para></entry>
             <entry><para>Represents the name of the virtual / distributed virtual switch at
                 vCenter.</para></entry>
-            <entry><para>The default value assumed would depend upon the type of virtual switch.
-                Defaults values are as follows. vSwitch0 if type of virtual switch is "VMware
-                vNetwork Standard virtual switch" dvSwitch0 if type of virtual switch is "VMware
-                vNetwork distributed virtual switch" epp0 if type of virtual switch is "Cisco Nexus
-                1000v distributed virtual switch</para></entry>
+            <entry><para>The default value depends on the type of virtual switch:</para>
+              <para><emphasis role="bold">vSwitch0</emphasis>: If type of virtual switch is VMware
+                vNetwork Standard virtual switch </para>
+              <para><emphasis role="bold">dvSwitch0</emphasis>: If type of virtual switch is VMware
+                vNetwork Distributed virtual switch</para>
+              <para><emphasis role="bold">epp0</emphasis>: If type of virtual switch is Cisco Nexus
+                1000v Distributed virtual switch</para></entry>
           </row>
           <row>
             <entry><para>2</para></entry>
-            <entry><para>VLAN ID to be used for this traffic where ever applicable</para></entry>
+            <entry><para>VLAN ID to be used for this traffic wherever applicable.</para></entry>
             <entry><para>This field would be used for only public traffic as of now. In case of guest traffic this
                 field would be ignored and could be left empty for guest traffic. By default empty
                 string would be assumed which translates to untagged VLAN for that specific traffic
@@ -121,19 +131,24 @@
           </row>
           <row>
             <entry><para>3</para></entry>
-            <entry><para>Type of virtual switch specified as string</para></entry>
-            <entry><para>Possible valid values are vmwaredvs, vmwaresvs, nexusdvs. Each translates as follows.
-                "vmwaresvs" represents "VMware vNetwork Standard virtual switch" "vmwaredvs"
-                represents "VMware vNetwork distributed virtual switch" "nexusdvs" represents "Cisco
-                Nexus 1000v distributed virtual switch" If nothing is specified (left empty) that
-                means zone level default virtual switch (based on value of global parameters) would
-                be assumed. Following are the global configuration parameters. vmware.use.dvswitch -
-                This should be true to enable any kind (VMware DVS / Cisco Nexus 1000v DVS) of
-                distributed virtual switch in cloudstack deployment. If this is false that means
-                default virtual switch in that cloudstack deployment is "standard virtual switch"
-                only. vmware.use.nexus.vswitch - This parameter would be ignored unless
-                "vmware.use.dvswitch" is true. Set this to "true" to enable Cisco Nexus 1000v
-                distributed virtual switch in a cloudstack deployment.</para></entry>
+            <entry><para>Type of virtual switch. Specified as string.</para></entry>
+            <entry><para>Possible valid values are vmwaredvs, vmwaresvs, nexusdvs. </para>
+              <para><emphasis role="bold">vmwaresvs</emphasis>: Represents VMware vNetwork Standard
+                virtual switch</para>
+              <para><emphasis role="bold">vmwaredvs</emphasis>: Represents VMware vNetwork
+                distributed virtual switch</para>
+              <para><emphasis role="bold">nexusdvs</emphasis>: Represents Cisco Nexus 1000v
+                distributed virtual switch.</para>
+              <para>If nothing specified (left empty), zone-level default virtual switch would be
+                defaulted, based on the value of global parameter you specify. </para>
+              <para>Following are the global configuration parameters:</para>
+              <para><emphasis role="bold">vmware.use.dvswitch</emphasis>: Set to true to enable any
+                kind (VMware DVS and Cisco Nexus 1000v) of distributed virtual switch in a &PRODUCT;
+                deployment. If set to false, the virtual switch that can be used in that &PRODUCT;
+                deployment is Standard virtual switch. </para>
+              <para><emphasis role="bold">vmware.use.nexus.vswitch</emphasis>: This parameter is
+                ignored if vmware.use.dvswitch is set to false. Set to true to enable Cisco Nexus
+                1000v distributed virtual switch in a &PRODUCT; deployment.</para></entry>
           </row>
         </tbody>
       </tgroup>