On Sat, Jan 14, 2012 at 4:32 AM, Lars Ellenberg
<lars.ellenb...@linbit.com>wrote:

> On Mon, Jan 09, 2012 at 05:50:14PM +0100, Dejan Muhamedagic wrote:
> > Hi Serge,
> >
> > On Mon, Jan 09, 2012 at 09:11:43AM -0700, Serge Dubrouski wrote:
> > > I did a couple of weeks ago :-)
> >
> > Hmm, me completely missed it. Sorry about that. Will apply the
> > patch. Many thanks to Junko for the contribution.
>
> Hm. I apparently missed this, too.
>
> -    if [ $? -ne 0 ] || ! echo $output | grep -q '.* has address
> '"$OCF_RESKEY_monitor_response"
> +    if [ $? -ne 0 ] || ! echo $output | egrep -q '.* has |IPv6 address
> '"$OCF_RESKEY_monitor_response"
>
> Not good.
>
> Should be
> +    if [ $? -ne 0 ] || ! echo $output | grep -q '.* \(has\|IPv6\) address
> '"$OCF_RESKEY_monitor_response"
>
> Why?
> Because otherwise, as long as the resonse contains " has ", it
> would match, and $OCF_RESKEY_monitor_response would be ignored.
>
> And, using egrep (or grep -E) would also change how
> $OCF_RESKEY_monitor_response would be interpreted,
> so could in theory break existing configurations,
> if they use grep special chars.
> If you consider this as unlikely, do
>
> +    if [ $? -ne 0 ] || ! echo $output | grep -q -E '.* (has|IPv6) address
> '"$OCF_RESKEY_monitor_response"
>

Thanks, Lars. Of course you are right.

Dejan, could you please apply Lar's version?

>
> >
> > Thanks,
> >
> > Dejan
> >
> > >  On Jan 9, 2012 8:00 AM, "Dejan Muhamedagic" <de...@suse.de> wrote:
> > >
> > > > Hi Junko-san,
> > > >
> > > > On Tue, Dec 13, 2011 at 04:32:07PM +0900, Junko IKEDA wrote:
> > > > > Hi Serge,
> > > > >
> > > > > We are now investigating the support status of ocf RAs,
> > > > > and this is the issue for named.
> > > > >
> > > > > Here is the example output of host command;
> > > > >
> > > > > # host www.kame.net
> > > > > www.kame.net is an alias for orange.kame.net.
> > > > > orange.kame.net has address 203.178.141.194
> > > > > orange.kame.net has IPv6 address
> 2001:200:dff:fff1:216:3eff:feb1:44d7
> > > > >
> > > > > named_monitor() searches its named server with
> > > > $OCF_RESKEY_monitor_response.
> > > > > I'm not familiar with named's behavior,
> > > > > is it possible to set IPv6 to $OCF_RESKEY_monitor_response?
> > > > > If $OCF_RESKEY_monitor_response has IPv6 address,
> > > > > the following syntax can not hit the result, right?
> > > >
> > > > The patch looks OK to me. Serge, can you also ack please?
> > > >
> > > > Cheers,
> > > >
> > > > Dejan
> > > >
> > > > > named_monitor()
> > > > >
> > > > > output=`$OCF_RESKEY_host $OCF_RESKEY_monitor_request
> > > > $OCF_RESKEY_monitor_ip`
> > > > > if [ $? -ne 0 ] || ! echo $output | grep -q '.* has address
> > > > > '"$OCF_RESKEY_monitor_response"
> > > > >
> > > > > Would you please give me some advice?
> > > > >
> > > > > Regards,
> > > > > Junko IKEDA
> > > > >
> > > > > NTT DATA INTELLILINK CORPORATION
> > > >
> > > >
> > > > > _______________________________________________________
> > > > > Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> > > > > http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> > > > > Home Page: http://linux-ha.org/
> > > >
> > > > _______________________________________________________
> > > > Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> > > > http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> > > > Home Page: http://linux-ha.org/
> > > >
> >
> > > _______________________________________________________
> > > Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> > > http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> > > Home Page: http://linux-ha.org/
> >
> > _______________________________________________________
> > Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> > http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> > Home Page: http://linux-ha.org/
>
> --
> : Lars Ellenberg
> : LINBIT | Your Way to High Availability
> : DRBD/HA support and consulting http://www.linbit.com
>
> DRBD® and LINBIT® are registered trademarks of LINBIT, Austria.
> _______________________________________________________
> Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
> http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
> Home Page: http://linux-ha.org/
>



-- 
Serge Dubrouski.
_______________________________________________________
Linux-HA-Dev: Linux-HA-Dev@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/

Reply via email to