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.

Regards,
M.

_______________________________________________
Pacemaker mailing list
Pacemaker@oss.clusterlabs.org
http://oss.clusterlabs.org/mailman/listinfo/pacemaker

Reply via email to