Hey,
You should take a look at the thread "Calling si_addclients without
'--ip-range'" in sisuite-users, where I proposed different solutions to the
problem.
Andrea preferred the escape character solution because its more backward
compatible.
While I personally don't care about that (I just started using SI), I can
imagine that simply changing the delimiter from one version to the other
could cause problems for some users.
Other than that I would agree with you that using some other delimiter would
be the cleanest solution (although I also don't like the --delimiter
solution, that Andrea suggested).
Maybe we should allow both delimiters ('-' and ':') for a few versions
(encouraging the user to use ':'), before disabling the '-' delimiter.
Thomas Krause
-----Ursprüngliche Nachricht-----
Von: Bernard Li [mailto:[EMAIL PROTECTED]
Gesendet: Freitag, 10. August 2007 21:50
An: Thomas Krause
Cc: [email protected]
Betreff: Re: [Sisuite-devel] PATCH: Allow hostname-ranges with hosts that
contain a '-' (HostRange.pm) [FIXED VERSION]
Hi Thomas:
Perhaps it makes sense for us to change the delimiter '-' and use
something else which is not allowed for hostnames (for instance ':')
-- so it will become something like:
si_addclients --script xyz --domainname abc --hosts client-01:client-99
Andrea, what do you think?
Cheers,
Bernard
On 8/10/07, Thomas Krause <[EMAIL PROTECTED]> wrote:
> Well, this would create the single host client-01-client-99, because you
> didn't escape the '-'. The correct call would be:
>
> si_addclients --script xyz --domainname abc --hosts
> 'client\\-01-client\\-99'
>
> which works with or without the quotes (I didn't change anything in this
> behavior).
>
> However by trying your example I found a small bug in my patch, that
causes
> an error if you are trying to add a single host with an '-' and without
> escaping the '-' (e.g. 'abc-xyz'). I attached a new patch as a replacement
> for the old one, which should fix the problem. Sorry for that!
>
> Thomas Krause
>
> -----Ursprüngliche Nachricht-----
> Von: Bernard Li [mailto:[EMAIL PROTECTED]
> Gesendet: Freitag, 10. August 2007 19:23
> An: Thomas Krause
> Cc: [email protected]
> Betreff: Re: [Sisuite-devel] PATCH: Allow hostname-ranges with hosts that
> contain a '-' (HostRange.pm)
>
> Hi Thomas:
>
> On 8/10/07, Thomas Krause <[EMAIL PROTECTED]> wrote:
>
> > Attached to this mail, is the patch to allow hostnames that contain an
'-'
> > to be used with host ranges in si_addclients. After applying the patch
you
> > can call si_addclients like:
> >
> > si_addclients --script xyz --domainname abc --hosts
> client\\-01-client\\-99
> >
> > which adds client-01 to client-99.
>
> Thanks for the patch -- but would it also work if you do:
>
> si_addclients --script xyz --domainname abc --hosts 'client-01-client-99'
>
> in quotes?
>
> Cheers,
>
> Bernard
>
> -------------------------------------------------------------------------
> This SF.net email is sponsored by: Splunk Inc.
> Still grepping through log files to find problems? Stop.
> Now Search log events and configuration files using AJAX and a browser.
> Download your FREE copy of Splunk now >> http://get.splunk.com/
> _______________________________________________
> sisuite-devel mailing list
> [email protected]
> https://lists.sourceforge.net/lists/listinfo/sisuite-devel
>
>
>
-------------------------------------------------------------------------
This SF.net email is sponsored by: Splunk Inc.
Still grepping through log files to find problems? Stop.
Now Search log events and configuration files using AJAX and a browser.
Download your FREE copy of Splunk now >> http://get.splunk.com/
_______________________________________________
sisuite-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/sisuite-devel
