Re: [Linux-ha-dev] [PATCH]The evasion of the probe error for pgsql.

[Dejan Muhamedagic]

Hi Hideo-san,

On Mon, Sep 20, 2010 at 04:43:35PM +0900, renayama19661...@ybb.ne.jp wrote:
> Hi Serge,
> 
> > This is good. Thanks, Hideo.
> 
> Thanks!!
> 
> Let a development version please reflect this patch.

Patch applied.

Cheers,

Dejan

> Best Regards,
> Hideo Yamauchi.
> 
> 
> --- Serge Dubrouski <serge...@gmail.com> wrote:
> 
> > This is good. Thanks, Hideo.
> > 
> > 2010/9/17  <renayama19661...@ybb.ne.jp>:
> > > Hi All,
> > >
> > > When we made pgdata a mount(example : /dbfp1 on shared disk), the next 
> > > error log is output in
> > probe by
> > > all means.
> > >
> > > ERROR: Cannot find configuration file /dbfp1/pgdata/data/postgresql.conf
> > >
> > > Do not make this error log warning in probe?
> > >
> > > Because the reason is because an operator watching error-log is confused.
> > >
> > > diff -r d2fdb87bded9 heartbeat/pgsql
> > > --- a/heartbeat/pgsql &#160; Fri Sep 17 15:30:20 2010 +0900
> > > +++ b/heartbeat/pgsql &#160; Fri Sep 17 15:33:04 2010 +0900
> > > @@ -22,6 +22,11 @@
> > > &#160;get_pgsql_param() {
> > > &#160; &#160; local config_file
> > > &#160; &#160; local param_name
> > > + &#160; &#160;local loglevel=err
> > > +
> > > + &#160; &#160;if ocf_is_probe; then
> > > + &#160; &#160; &#160; &#160;loglevel=warn
> > > + &#160; &#160;fi
> > >
> > > &#160; &#160; param_name=$1
> > >
> > > @@ -33,7 +38,7 @@
> > > &#160; &#160; fi
> > >
> > > &#160; &#160; if [ ! -f "$config" ]; then
> > > - &#160; &#160; &#160; &#160;ocf_log err "Cannot find configuration file 
> > > $config"
> > > + &#160; &#160; &#160; &#160;ocf_log $loglevel "Cannot find configuration 
> > > file $config"
> > > &#160; &#160; &#160; &#160; return
> > > &#160; &#160; fi
> > >
> > >
> > > Best Regards,
> > > Hideo Yamauchi.
> > >
> > > _______________________________________________________
> > > 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/
> > 
> 
> _______________________________________________________
> 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/