modify the doc with comment

docs/source/advanced/networks/switchdiscover/switch_based_switch_discovery.rst

@@ -8,7 +8,7 @@ Pre-requirement
 
 In order to do switch-based switch discovery, the admin 
 
-1.  Needs to manually setup and configure core-switch, SNMP v3 needs to be enabled in order for xCAT access to it. Example of core-switch definition:   
+1.  Needs to manually setup and configure core-switch, SNMP v3 needs to be enabled in order for xCAT access to it. username and userpassword attribute are for the remote login. It can be for ssh or telnet. If it is for telnet, please set protocol to “telnet”. If the sshusername is blank, the username, password and protocol will be retrieved from the passwd table with “switch” as the key. SNMP attributes will used for SNMPv3 communication.  nodetype has to be set to **switch** to differentiate between switch-based node discovery or switch-based switch discovery. Please refer switches table attributes.  Example of core-switch definition:   
 
 ::
 
@@ -33,7 +33,7 @@ In order to do switch-based switch discovery, the admin
 
 
 
-2.  Then pre-define all the top-rack switches which connect to core-switch. 
+2.  Then pre-define all the top-rack switches which connect to core-switch.  The attribute **ip** is static ip address for the switch.  If ``switchdiscover --setup`` specified, this ip address will replace dhcp ip address on the switch. ``nodetype=switch`` needs to be set to differentiate between switch-based node discovery or switch-based switch discovery during discover process.  the ``switch`` is hostname of core-switch and ``switchport`` is the port number in the core-switch that top-rack switch is connected to.    
 
 ::
 
@@ -48,7 +48,14 @@ In order to do switch-based switch discovery, the admin
       switchtype=BNT
 
 
-3.  Setup Dynamic ip range to network table for discovery switches to use.
+3.  Setup Dynamic ip range to network table for discovery switches to use. ::
+
+     # tabdump networks
+      #netname,net,mask,mgtifname,gateway,dhcpserver,tftpserver,nameservers,ntpservers,logservers,dynamicrange,staticrange,staticrangeincrement,nodehostname,ddnsdomain,vlanid,domain,comments,disable
+      "192_168_0_0-255_255_0_0","192.168.0.0","255.255.0.0","enP4p1s0f2","<xcatmaster>",,"192.168.3.29",,,,"192.168.5.150-192.168.5.170",,,,,,,,
+
+
+dhcp should restarted after set up dynamic ip address.
 
 
 Discover Switches
@@ -80,6 +87,34 @@ if ``–setup`` flag is specified, it will process following steps:
 
 3.  after switches matched, will call config files to set up static ip address, hostname and enable the snmpv3.  currently, BNT and Mellanox switches are supported.  The two config files are located in the ** /opt/xcat/share/xcat/scripts/config.BNT** and **/opt/xcat/share/xcat/scripts/config.Mellanox**.  the log message ``the switch type is not support for config`` if switchtype other than BNT and Mellanox.
 
+4.  After discover process, the pre-defined node attribute in the xCATdb should be updated. 
+
+::
+
+    lsdef switch-192-168-5-22
+       groups=switch
+       ip=192.168.5.22
+       mac=a8:97:dc:02:92:00
+       mgt=switch
+       nodetype=switch
+       password=admin
+       postbootscripts=otherpkgs
+       postscripts=syslog,remoteshell,syncfiles
+       protocol=telnet
+       snmpauth=sha
+       snmppassword=xcatadminpassw0rd@snmp
+       snmpusername=xcatadmin
+       snmpversion=3
+       status=hostname_configed
+       statustime=08-31-2016 15:35:49
+       supportedarchs=ppc64
+       switch=switch-10-5-23-1
+       switchport=45
+       switchtype=BNT
+       usercomment=IBM Networking Operating System RackSwitch G8052
+       username=root
+
+
 
 Configure switches
 ~~~~~~~~~~~~~~~~~~
@@ -121,11 +156,13 @@ These two config files are located in the **/opt/xcat/share/xcat/scripts**.  The
 switch status
 ~~~~~~~~~~~~~
 
+During the switch-based switch discovery process, there are four status introduced.  User may only see **switch_configed** status on node definition if discovery process succefully finished.  
+
 **Matched** --- Discover switch is matched to pre-define switch, otherinterfaces attribute is updated to dhcp ip address, and mac address, switch type and usercomment also updated with vendor information for the predefined switch.
 
-**ip_configed** --- switches are set up to static ip address based on pre-define switch ip address
+**ip_configed** --- switches are set up to static ip address based on pre-define switch ip address.  If failed to set up ip address, the status will keep as **Matched**.
 
-**hostname_configed** -- switches host name changed based on pre-define switch hostname.
+**hostname_configed** -- switches host name changed based on pre-define switch hostname. If failed to change hostname on the switch, the status will keep as **ip_configed**.
 
-**switch_configed** -- snmpv3 is setup for the switches.  this should be finial status after run ``switchdiscover --setup`` command.
+**switch_configed** -- snmpv3 is setup for the switches.  this should be finial status after run ``switchdiscover --setup`` command. If failed to setup snmpv3,  the status will keep as **hostname_configed**.