[Pacemaker] timeout value with RA xen and drbd (s or no s?)

[Dejan Muhamedagic]

Hi,

On Mon, Jan 04, 2010 at 03:14:00PM +0100, Martin Gombač wrote:
> Dejan Muhamedagic wrote:
> >Hi,
> >
> >On Mon, Jan 04, 2010 at 02:07:37PM +0100, Martin Gombač wrote:
> >>Which is correct? Adding s to timeout values or not?
> >>
> >>Like this?
> >>
> >>primitive drbd_r0 ocf:linbit:drbd \
> >>       params drbd_resource="r0" \
> >>       op monitor interval="15s" role="Master" timeout="30s" \
> >>      ....
> >>primitive Hosting ocf:heartbeat:Xen \
> >>   params xmfile="/etc/xen/Hosting.cfg" shutdown_timeout="303s" \
> >>   meta target-role="Started" allow-migrate="true" is-managed="true" \
> >>   op monitor interval="120s" timeout="506s" start-delay="33s" \
> >>      ....
> >>
> >>
> >>or like this?
> >>primitive drbd_r0 ocf:linbit:drbd \
> >>       params drbd_resource="r0" \
> >>       op monitor interval="15" role="Master" timeout="30" \
> >>      ....
> >>primitive Hosting ocf:heartbeat:Xen \
> >>   params xmfile="/etc/xen/Hosting.cfg" shutdown_timeout="303" \
> >>   meta target-role="Started" allow-migrate="true" is-managed="true" \
> >>   op monitor interval="120" timeout="506" start-delay="33" \
> >>      ....
> >>
> >>crm info Xen and drbd tells me to set options without s, while let's
> >>say: http://www.clusterlabs.org/wiki/DRBD_HowTo_1.0 examples use s
> >>with timeout, delay and other values.
> >
> >The two versions are equivalent.
> >The default for time values are seconds (s), but I think it would
> >be good style to still append "s".
> >
> >Thanks,
> >
> >Dejan
> >
> 
> Jan  4 15:01:05 ibm1 lrmd: [25113]: info: RA output:
> (Hosting:stop:stderr) /usr/lib/ocf/resource.d//heartbeat/Xen: line
> 279: [: 303s: integer expression expected
> /usr/lib/ocf/resource.d//heartbeat/Xen: line 289: [: 303s: integer
> expression expected
> 
> According to above line I'm guessing that parameter shutdown_timeout
> from Xen resource agent (not an option) requires without s.

Sorry, missed that there was also an instance parameter. That one
has to be an integer. Even though it is about the same thing
(stop timeout), the handling is very different. What I wrote
above is only about the meta parameters.

Thanks,

Dejan

> Regards,
> M.
> 
> _______________________________________________
> Pacemaker mailing list
> Pacemaker at oss.clusterlabs.org
> http://oss.clusterlabs.org/mailman/listinfo/pacemaker