On Wed, Dec 29, 2010 at 02:28:35PM +0100, Alexander Krauth wrote: > # HG changeset patch > # User Alexander Krauth <[email protected]> > # Date 1293629310 -3600 > # Node ID 1f196eca3d29efe1ed0cef547ff62cba22e0941e > # Parent 7f6dae9eb6ec668abb9dcc8c80de58fd98902758 > High: SAPInstance: Eliminate hostname command call for default parameter of > sapcontrol > > sapcontrol executes via default on the host where it is called. So we can > omit the call of the hostname command.
Applied with summary changed to "Low: ..." > > diff -r 7f6dae9eb6ec -r 1f196eca3d29 heartbeat/SAPInstance > --- a/heartbeat/SAPInstance Wed Dec 29 14:23:11 2010 +0100 > +++ b/heartbeat/SAPInstance Wed Dec 29 14:28:30 2010 +0100 > @@ -546,8 +546,7 @@ > if [ $rc -eq $OCF_SUCCESS ] > then > count=0 > - LOCALHOST=`hostname` > - output=`$SAPCONTROL -nr $InstanceNr -host $LOCALHOST -function > GetProcessList -format script` > + output=`$SAPCONTROL -nr $InstanceNr -function GetProcessList -format > script` > > # we have to parse the output, because the returncode doesn't tell > anything about the instance status > for SERVNO in `echo "$output" | grep '^[0-9] ' | cut -d' ' -f1 | sort -u` > _______________________________________________________ > Linux-HA-Dev: [email protected] > http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev > Home Page: http://linux-ha.org/ _______________________________________________________ Linux-HA-Dev: [email protected] http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev Home Page: http://linux-ha.org/
