On Mon, Mar 12, 2012 at 14:46, Iustin Pop <ius...@google.com> wrote:
> On Mon, Mar 12, 2012 at 02:11:05PM +0100, René Nussbaumer wrote:
>>  Algorithm options:
>> diff --git a/man/htools.rst b/man/htools.rst
>> index cc9ef39..49fd08b 100644
>> --- a/man/htools.rst
>> +++ b/man/htools.rst
>> @@ -203,6 +203,11 @@ support all options. Some common options are:
>>    on which the master daemon listens; otherwise, the default path used
>>    by Ganeti (configured at build time) is used.
>>
>> +-I|\--ialloc-src *path*
>> +  Backend specification: collect data directly from an iallocator
>> +  specification (as produced by Ganeti when doing an iallocator call).
>> +  The iallocator specification is read from specified path.
>
> LGTM on the rest, but I would kindly ask to reword this a bit. "Collect"
> is not so good as it implies actively connecting to something - I would
> suggest "load data directly…". And specification doesn't sound right -
> can we just say request?

Sure! Here's the interdiff:

diff --git a/man/htools.rst b/man/htools.rst
index 49fd08b..a05039b 100644
--- a/man/htools.rst
+++ b/man/htools.rst
@@ -204,9 +204,9 @@ support all options. Some common options are:
   by Ganeti (configured at build time) is used.

 -I|\--ialloc-src *path*
-  Backend specification: collect data directly from an iallocator
-  specification (as produced by Ganeti when doing an iallocator call).
-  The iallocator specification is read from specified path.
+  Backend specification: load data directly from an iallocator request
+  (as produced by Ganeti when doing an iallocator call).  The iallocator
+  request is read from specified path.

 \--simulate *description*
   Backend specification: instead of using actual data, build an empty


René

Reply via email to