On Fri, Oct 10, 2008 at 01:39:11PM +1100, Simon Horman wrote:
> On Fri, Oct 10, 2008 at 09:40:11AM +1100, Simon Horman wrote:
> > Hi Stefan,
> > 
> > I think that there is a silly parsing bug. Can you please try:
> > 
> >     checkcommand = /usr/local/sbin/check_lustre_on_realserver
> > 
> > Instead of
> > 
> >     checkcommand = "/usr/local/sbin/check_lustre_on_realserver"
> 
> Hi Stefan,
> 
> could you try the following patch to see if it solves your
> problem without needing to update the configuration file?

Sorry,

the previous version of this patch was useless.
This one is better.

-- 
Simon Horman
  VA Linux Systems Japan K.K., Sydney, Australia Satellite Office
  H: www.vergenet.net/~horms/             W: www.valinux.co.jp/en

Index: lha-dev/ldirectord/ldirectord.in
===================================================================
--- lha-dev.orig/ldirectord/ldirectord.in       2008-10-15 18:02:16.000000000 
+1100
+++ lha-dev/ldirectord/ldirectord.in    2008-10-15 18:16:20.000000000 +1100
@@ -1179,7 +1179,7 @@ sub read_config
                                        } else {
                                                &config_error($line, "checktype 
must be \"connect\", \"negotiate\", \"on\", \"off\", \"ping\", \"external\" or 
a positive number");
                                        }
-                               } elsif ($rcmd =~ /^checkcommand\s*=\s*(.*)/){
+                               } elsif ($rcmd =~ 
/^checkcommand\s*=\s*\"(.*)\"/ or $rcmd =~ /^checkcommand\s*=\s*(.*)/){
                                         $1 =~ /(.+)/ or &config_error($line, 
"invalid external script");
                                         $vsrv{checkcommand} = $1;
                                } elsif ($rcmd =~ /^checktimeout\s*=\s*(.*)/){
@@ -1321,7 +1321,8 @@ sub read_config
                                        $vsrv{fallback} =
                                                parse_fallback($line, $1,
                                                               \%vsrv);
-                               } elsif ($rcmd =~ 
/^fallbackcommand\s*=\s*(.*)/) {
+                               } elsif ($rcmd =~
+                               /^fallbackcommand\s*=\s*\"(.*)\"/ or $rcmd =~ 
/^fallbackcommand\s*=\s*(.*)/) {
                                         $1 =~ /(.+)/ or &config_error($line, 
"invalid fallback command");
                                         $vsrv{fallbackcommand} = $1;
                                 } elsif ($rcmd =~ /^quiescent\s*=\s*(.*)/) {

_______________________________________________
Linux-HA mailing list
[email protected]
http://lists.linux-ha.org/mailman/listinfo/linux-ha
See also: http://linux-ha.org/ReportingProblems

Reply via email to