cloudstack-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Sateesh Chodapuneedi <sateesh.chodapune...@citrix.com>
Subject RE: Review Request: Documentation changes for VMware dvSwitch and Nexus dvSwitch
Date Wed, 10 Apr 2013 07:02:27 GMT
> 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:noreply@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/>


Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message