> We cannot commit multiple patches for the same files when we have more than > 50 features to document. In that case, instead of uploading patch having multiple unrelated changes to reviewboard, attaching relevant documents using “Add File” with details of sections to look for would help. A ‘ship it’ would apply to every line of modification in the patch, not sure if specific chunks in patch diff can be marked for ‘ship it’.
Regards, Sateesh From: Radhika Puthiyetath Sent: 10 April 2013 12:02 To: Sateesh Chodapuneedi; David Nalley; Jessica Tomechak; ilya musayev; Chip Childers; Pranav Saxena Cc: cloudstack Subject: RE: Review Request: Documentation changes for VMware dvSwitch and Nexus dvSwitch The particular screen (Add-Cluster) has those topic as well. Please neglect what you do not want to see. We cannot commit multiple patches for the same files when we have more than 50 features to document. From: Sateesh Chodapuneedi [mailto:nore...@reviews.apache.org] On Behalf Of Sateesh Chodapuneedi Sent: Wednesday, April 10, 2013 11:58 AM To: David Nalley; Jessica Tomechak; Sateesh Chodapuneedi; ilya musayev; Chip Childers; Pranav Saxena Cc: cloudstack; Radhika Puthiyetath Subject: Re: Review Request: Documentation changes for VMware dvSwitch and Nexus dvSwitch This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/10366/ docs/en-US/add-clusters-vsphere.xml<https://reviews.apache.org/r/10366/diff/1/?file=279196#file279196line102> (Diff revision 1) None 89 <para>vCenter Password. Enter the password for the user named above</para> 102 <para><emphasis role="bold">RAM overcommit ratio</emphasis>: Enter the RAM overcommit Can this patch be cleaned to just reflect VMware dvSwitch feature documentation? Seeing text related to other features like cpu/memory over commit ratio support. Also it helps to move out nexus dvSwitch specific documentation from this patch as this patch is addressing only CLOUDSTACK-772. - Sateesh On April 9th, 2013, 7:31 a.m., Radhika PC wrote: Review request for cloudstack, David Nalley, Chip Childers, Jessica Tomechak, Pranav Saxena, Sateesh Chodapuneedi, and ilya musayev. By Radhika PC. Updated April 9, 2013, 7:31 a.m. Description Documentation on Distributed Switches: nexus and dvSwitch. Prerequisites part of VMware dvSwitch is still unclear. Please provide necessary suggestions. Testing Publican builds, patch applies. Bugs: CLOUDSTACK-772 Diffs * docs/en-US/Book_Info.xml (c125ab8) * docs/en-US/add-clusters-vsphere.xml (6b2dff2) * docs/en-US/images/add-cluster.png (383f375ebedd62d9b294a56f777ed4b8c0d92e10) * docs/en-US/images/dvswitch-config.png (PRE-CREATION) * docs/en-US/images/dvswitchconfig.png (PRE-CREATION) * docs/en-US/vmware-cluster-config-dvswitch.xml (PRE-CREATION) * docs/en-US/vmware-install.xml (467e135) View Diff<https://reviews.apache.org/r/10366/diff/>
