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

[Serge Dubrouski]

Thanks, Dejan -

There was one more little patch for pgsql to replace:

 ocf_run $quietrun su $OCF_RESKEY_pgdba -c "$*"

with

 ocf_run $quietrun su $OCF_RESKEY_pgdba -c "cd $OCF_RESKEY_pgdata; $@"

earlier.

Can you please apply that one too or you want me to resubmit it?

Thanks.

On Wed, Sep 22, 2010 at 8:21 AM, Dejan Muhamedagic <deja...@fastmail.fm> wrote:
> 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/
>



-- 
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/