On Mon, Nov 11, 2013 at 4:37 PM, Petr Pudlak <[email protected]> wrote:

> A new option `ssh-port` is added. If set to a non-standard port, the QA
> script
> sets up the default node group with this port, and before running test it
> adds
> `iptable` rules to all nodes so that the nodes see each other's SSH
> servers as
> running on this port. Their SSH configuration is _not_ changed and other
> machines see the nodes on 22 as before.
>

The first sentence sounds a bit unnatural, maybe change it to:
"This patch adds the `ssh-port` option."

Rest LGTM, thanks!

Hrvoje Ribicic
Ganeti Engineering
Google Germany GmbH
Dienerstr. 12, 80331, München

Registergericht und -nummer: Hamburg, HRB 86891
Sitz der Gesellschaft: Hamburg
Geschäftsführer: Graham Law, Christine Elizabeth Flores
Steuernummer: 48/725/00206
Umsatzsteueridentifikationsnummer: DE813741370

Reply via email to