On Tue, Oct 2, 2012 at 6:38 PM, Michael Hanselmann <han...@google.com> wrote:
> 2012/10/2 Bernardo Dal Seno <bdals...@google.com>:
>
>  printed by the ``vcluster-setup`` script together with other useful
> -information, such as the list of virtual nodes. The commands printed are
> -used to configure the enabled hypervisors and other settings.
> -
> -To run commands for a specific virtual node, a script named ``cmd``
> -located in the node directory can be used. Example:
> +information, such as the list of virtual nodes. The commands printed
> +should be used to configure the list of enabled hypervisors and other
> +settings.
>
>>> +To run commands for a specific virtual node, a script named ``cmd``
>>> +located in the node directory can be used. Example:
>>
>> I liked the previous version better (s/a script/the script/). Anyway,
>> the point of the script is to set GANETI_ROOTDIR and GANETI_HOSTNAME
>> before executing the command, but this is not stated in this document.
>
> How about this:
> -To run commands for a specific virtual node, a script named ``cmd``
> -located in the node directory can be used. Example:
> +To run commands for a specific virtual node, the script named ``cmd``
> +located in the node directory can be used. It takes a command as its
> +argument(s), sets the environment variables ``GANETI_ROOTDIR`` and
> +``GANETI_HOSTNAME`` and then runs the command. Example:

LGTM, thanks!

Bernardo

Reply via email to