Discussion:
[Linux-ha-dev] Some minor patches for cluster-glue
Kristoffer Grönlund
2016-08-10 10:32:48 UTC
Permalink
Hi everyone (Lars and Dejan in particular),

Here are some minor patches for cluster-glue. The first one is an
attempt to get the stonith man page somewhat up to date, and the other
two are minor issues discovered when compiling cluster-glue using GCC
6.

Cheers,
Kristoffer
--
// Kristoffer Grönlund
// ***@suse.com
Dejan Muhamedagic
2016-08-11 19:22:50 UTC
Permalink
Hi Kristoffer,
Post by Kristoffer Grönlund
Hi everyone (Lars and Dejan in particular),
Here are some minor patches for cluster-glue. The first one is an
attempt to get the stonith man page somewhat up to date, and the other
two are minor issues discovered when compiling cluster-glue using GCC
6.
Pushed just now the man page patch which was pending in my queue.
Will apply the other two too.

Thanks for the contribution!

Cheers,

Dejan
Post by Kristoffer Grönlund
Cheers,
Kristoffer
--
// Kristoffer Grönlund
changeset: 2820:13875518ed6b
parent: 2815:643ac28499bd
date: Wed Aug 10 12:13:13 2016 +0200
files: doc/stonith.xml.in
Low: stonith: Update man page with -E, -m parameters (bsc#970307)
diff --git a/doc/stonith.xml.in b/doc/stonith.xml.in
--- a/doc/stonith.xml.in
+++ b/doc/stonith.xml.in
@@ -7,22 +7,28 @@
<authorgroup>
<author>
- <firstname>Alan</firstname>
- <surname>Robertson</surname>
- <contrib>stonith</contrib>
+ <firstname>Alan</firstname>
+ <surname>Robertson</surname>
+ <contrib>stonith</contrib>
</author>
<author>
- <firstname>Simon</firstname>
- <surname>Horman</surname>
- <contrib>man page</contrib>
+ <firstname>Simon</firstname>
+ <surname>Horman</surname>
+ <contrib>man page</contrib>
</author>
<author>
- <firstname>Florian</firstname>
- <surname>Haas</surname>
- <contrib>man page</contrib>
+ <firstname>Florian</firstname>
+ <surname>Haas</surname>
+ <contrib>man page</contrib>
+ </author>
+ <author>
+ <firstname>Kristoffer</firstname>
+ <surname>Gronlund</surname>
+ <contrib>man page</contrib>
</author>
</authorgroup>
</refentryinfo>
@@ -44,12 +50,14 @@
<cmdsynopsis>
<command>stonith</command>
<arg choice="opt"><option>-s</option></arg>
+ <arg choice="opt"><option>-v</option></arg>
<arg choice="opt"><option>-h</option></arg>
<arg choice="plain"><option>-L</option></arg>
</cmdsynopsis>
<cmdsynopsis>
<command>stonith</command>
<arg choice="opt"><option>-s</option></arg>
+ <arg choice="opt"><option>-v</option></arg>
<arg choice="opt"><option>-h</option></arg>
<arg choice="plain"><option>-t</option> <replaceable>stonith-device-type</replaceable></arg>
<arg choice="plain"><option>-n</option></arg>
@@ -57,14 +65,24 @@
<cmdsynopsis>
<command>stonith</command>
<arg choice="opt"><option>-s</option></arg>
+ <arg choice="opt"><option>-v</option></arg>
+ <arg choice="opt"><option>-h</option></arg>
+ <arg choice="plain"><option>-t</option> <replaceable>stonith-device-type</replaceable></arg>
+ <arg choice="plain"><option>-m</option></arg>
+ </cmdsynopsis>
+ <cmdsynopsis>
+ <command>stonith</command>
+ <arg choice="opt"><option>-s</option></arg>
+ <arg choice="opt"><option>-v</option></arg>
<arg choice="opt"><option>-h</option></arg>
<arg choice="plain"><option>-t</option> <replaceable>stonith-device-type</replaceable></arg>
<group choice="req" rep="norepeat">
- <group choice="plain" rep="repeat">
- <arg choice="plain"><replaceable>name</replaceable>=<replaceable>value</replaceable></arg>
- </group>
- <arg choice="plain"><option>-p</option> <replaceable>stonith-device-parameters</replaceable></arg>
- <arg choice="plain"><option>-F</option> <replaceable>stonith-device-parameters-file</replaceable></arg>
+ <group choice="plain" rep="repeat">
+ <arg choice="plain"><replaceable>name</replaceable>=<replaceable>value</replaceable></arg>
+ </group>
+ <arg choice="plain"><option>-p</option> <replaceable>stonith-device-parameters</replaceable></arg>
+ <arg choice="plain"><option>-E</option></arg>
+ <arg choice="plain"><option>-F</option> <replaceable>stonith-device-parameters-file</replaceable></arg>
</group>
<arg choice="opt"><option>-c</option> <replaceable>count</replaceable></arg>
<arg choice="opt"><option>-l</option></arg>
@@ -73,22 +91,24 @@
<cmdsynopsis>
<command>stonith</command>
<arg choice="opt"><option>-s</option></arg>
+ <arg choice="opt"><option>-v</option></arg>
<arg choice="opt"><option>-h</option></arg>
<arg choice="plain"><option>-t</option> <replaceable>stonith-device-type</replaceable></arg>
<group choice="req" rep="norepeat">
- <group choice="plain" rep="repeat">
- <arg choice="plain"><replaceable>name</replaceable>=<replaceable>value</replaceable></arg>
- </group>
- <arg choice="plain"><option>-p</option> <replaceable>stonith-device-parameters</replaceable></arg>
- <arg choice="plain"><option>-F</option> <replaceable>stonith-device-parameters-file</replaceable></arg>
+ <group choice="plain" rep="repeat">
+ <arg choice="plain"><replaceable>name</replaceable>=<replaceable>value</replaceable></arg>
+ </group>
+ <arg choice="plain"><option>-p</option> <replaceable>stonith-device-parameters</replaceable></arg>
+ <arg choice="plain"><option>-E</option></arg>
+ <arg choice="plain"><option>-F</option> <replaceable>stonith-device-parameters-file</replaceable></arg>
</group>
<arg choice="opt"><option>-c</option> <replaceable>count</replaceable></arg>
<arg choice="opt"><option>-T</option>
<group choice="req">
- <arg choice="plain">reset</arg>
- <arg choice="plain">on</arg>
- <arg choice="plain">off</arg>
- </group>
+ <arg choice="plain">reset</arg>
+ <arg choice="plain">on</arg>
+ <arg choice="plain">off</arg>
+ </group>
</arg>
<arg><replaceable>nodename</replaceable></arg>
</cmdsynopsis>
@@ -108,145 +128,161 @@
<para>The following options are supported:</para>
<variablelist>
<varlistentry>
- <term>
- <option>-c</option> <replaceable>count</replaceable>
- </term>
- <listitem>
- <para>Perform any actions identified by the
- <option>-l</option>, <option>-S</option> and
- <option>-T</option> options <replaceable>count</replaceable>
- times.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-F</option> <replaceable>stonith-device-parameters-file</replaceable>
- </term>
- <listitem>
- <para>Path of file specifying parameters for a stonith
- device. To determine the syntax of the parameters file for a
- given device type run:</para>
- <screen><computeroutput># </computeroutput><userinput>stonith -t stonith-device-type -n</userinput></screen>
- <para>All of the listed parameters need to appear in order
- on a single line in the parameters file and be delimited by
- whitespace.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-h</option>
- </term>
- <listitem>
- <para>Display detailed information about a stonith device
- including description, configuration information, parameters
- and any other related information. When specified without a
- stonith-device-type, detailed information on all stonith
- devices is displayed.</para>
- <para>If you don't yet own a stonith device and want to know
- more about the ones we support, this information is likely
- to be helpful.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-L</option>
- </term>
- <listitem>
- <para>List the valid stonith device types, suitable for
- passing as an argument to the <option>-t</option>
- option.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-l</option>
- </term>
- <listitem>
- <para>List the hosts controlled by the stonith device.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-n</option>
- </term>
- <listitem>
- <para>Output the parameter names of the stonith device.</para>
- </listitem>
+ <term>
+ <option>-c</option> <replaceable>count</replaceable>
+ </term>
+ <listitem>
+ <para>Perform any actions identified by the
+ <option>-l</option>, <option>-S</option> and
+ <option>-T</option> options <replaceable>count</replaceable>
+ times.</para>
+ </listitem>
</varlistentry>
<varlistentry>
- <term>
- <replaceable>name</replaceable>=<replaceable>value</replaceable>
- </term>
- <listitem>
- <para>Parameter, in the form of a name/value pair, to pass
- directly to the stonith device. To determine the syntax of
- the parameters for a given device type run:</para>
- <screen><computeroutput># </computeroutput><userinput>stonith -t stonith-device-type -n</userinput></screen>
- <para>All of the listed parameter names need to be passed
- with their corresponding values.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-p</option> <replaceable>stonith-device-parameters</replaceable>
- </term>
- <listitem>
- <para>Parameters to pass directly to the stonith device. To
- determine the syntax of the parameters for a given device
- type run:</para>
- <screen><computeroutput># </computeroutput><userinput>stonith -t stonith-device-type -n</userinput></screen>
- <para>All of the listed parameter names need to appear in
- order and be delimited by whitespace.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-S</option>
- </term>
- <listitem>
- <para>Show the status of the stonith device.</para>
- </listitem>
+ <term>
+ <option>-E</option>
+ </term>
+ <listitem>
+ <para>Get parameters from the environment.</para>
+ </listitem>
</varlistentry>
<varlistentry>
- <term>
- <option>-s</option>
- </term>
- <listitem>
- <para>Silent operation. Suppress logging of error messages to standard error.</para>
- </listitem>
+ <term>
+ <option>-F</option> <replaceable>stonith-device-parameters-file</replaceable>
+ </term>
+ <listitem>
+ <para>Path of file specifying parameters for a stonith
+ device. To determine the syntax of the parameters file for a
+ given device type run:</para>
+ <screen><computeroutput># </computeroutput><userinput>stonith -t stonith-device-type -n</userinput></screen>
+ <para>All of the listed parameters need to appear in order
+ on a single line in the parameters file and be delimited by
+ whitespace.</para>
+ </listitem>
</varlistentry>
<varlistentry>
- <term>
- <option>-T</option> <replaceable>action</replaceable>
- </term>
- <listitem>
- <para>The stonith action to perform on the node identified
- by nodename. Chosen from <token>reset</token>,
- <token>on</token>, and <token>off</token>.</para>
- <note>
- <para>If a nodename is specified without the
- <option>-T</option> option, the stonith action defaults to
- <token>reset</token>.</para>
- </note>
- </listitem>
+ <term>
+ <option>-h</option>
+ </term>
+ <listitem>
+ <para>Display detailed information about a stonith device
+ including description, configuration information, parameters
+ and any other related information. When specified without a
+ stonith-device-type, detailed information on all stonith
+ devices is displayed.</para>
+ <para>If you don't yet own a stonith device and want to know
+ more about the ones we support, this information is likely
+ to be helpful.</para>
+ </listitem>
</varlistentry>
<varlistentry>
- <term>
- <option>-t</option> <replaceable>stonith-device-type</replaceable>
- </term>
- <listitem>
- <para>The type of the stonith device to be used to effect
- stonith. A list of supported devices for an installation may
- be obtained using the <option>-L</option> option.</para>
- </listitem>
+ <term>
+ <option>-L</option>
+ </term>
+ <listitem>
+ <para>List the valid stonith device types, suitable for
+ passing as an argument to the <option>-t</option>
+ option.</para>
+ </listitem>
</varlistentry>
<varlistentry>
- <term>
- <option>-v</option>
- </term>
- <listitem>
- <para>Ignored.</para>
- </listitem>
+ <term>
+ <option>-l</option>
+ </term>
+ <listitem>
+ <para>List the hosts controlled by the stonith device.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-n</option>
+ </term>
+ <listitem>
+ <para>Output the parameter names of the stonith device.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-m</option>
+ </term>
+ <listitem>
+ <para>Display meta-data of the stonith device type.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <replaceable>name</replaceable>=<replaceable>value</replaceable>
+ </term>
+ <listitem>
+ <para>Parameter, in the form of a name/value pair, to pass
+ directly to the stonith device. To determine the syntax of
+ the parameters for a given device type run:</para>
+ <screen><computeroutput># </computeroutput><userinput>stonith -t stonith-device-type -n</userinput></screen>
+ <para>All of the listed parameter names need to be passed
+ with their corresponding values.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-p</option> <replaceable>stonith-device-parameters</replaceable>
+ </term>
+ <listitem>
+ <para>Parameters to pass directly to the stonith device. To
+ determine the syntax of the parameters for a given device
+ type run:</para>
+ <screen><computeroutput># </computeroutput><userinput>stonith -t stonith-device-type -n</userinput></screen>
+ <para>All of the listed parameter names need to appear in
+ order and be delimited by whitespace.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-S</option>
+ </term>
+ <listitem>
+ <para>Show the status of the stonith device.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-s</option>
+ </term>
+ <listitem>
+ <para>Silent operation. Suppress logging of error messages to standard error.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-T</option> <replaceable>action</replaceable>
+ </term>
+ <listitem>
+ <para>The stonith action to perform on the node identified
+ by nodename. Chosen from <token>reset</token>,
+ <token>on</token>, and <token>off</token>.</para>
+ <note>
+ <para>If a nodename is specified without the
+ <option>-T</option> option, the stonith action defaults to
+ <token>reset</token>.</para>
+ </note>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-t</option> <replaceable>stonith-device-type</replaceable>
+ </term>
+ <listitem>
+ <para>The type of the stonith device to be used to effect
+ stonith. A list of supported devices for an installation may
+ be obtained using the <option>-L</option> option.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-v</option>
+ </term>
+ <listitem>
+ <para>Ignored.</para>
+ </listitem>
</varlistentry>
</variablelist>
</refsection>
changeset: 2821:f44088ebf874
date: Wed Aug 10 12:15:08 2016 +0200
files: lib/clplumbing/cl_msg.c
Low: clplumbing: Remove incorrect negation of n
diff --git a/lib/clplumbing/cl_msg.c b/lib/clplumbing/cl_msg.c
--- a/lib/clplumbing/cl_msg.c
+++ b/lib/clplumbing/cl_msg.c
@@ -1344,7 +1344,7 @@ cl_msg_add_list_str(struct ha_msg* msg,
if (n <= 0 || buf == NULL|| name ==NULL ||msg == NULL){
cl_log(LOG_ERR, "%s:"
"invalid parameter(%s)",
"msg is NULL",__FUNCTION__);
@@ -1397,7 +1397,7 @@ cl_msg_add_list_int(struct ha_msg* msg,
if (n <= 0 || buf == NULL|| name ==NULL ||msg == NULL){
cl_log(LOG_ERR, "cl_msg_add_list_int:"
"invalid parameter(%s)",
"msg is NULL");
changeset: 2822:aeadfcd8425a
tag: tip
date: Wed Aug 10 12:17:17 2016 +0200
files: logd/ha_logd.c
Dev: ha_logd: Remove unused variables
diff --git a/logd/ha_logd.c b/logd/ha_logd.c
--- a/logd/ha_logd.c
+++ b/logd/ha_logd.c
@@ -75,9 +75,6 @@
#define EOS '\0'
#define nullchk(a) ((a) ? (a) : "<null>")
-static const int logd_keepalive_ms = 1000;
-static const int logd_warntime_ms = 5000;
-static const int logd_deadtime_ms = 10000;
static gboolean verbose = FALSE;
static pid_t write_process_pid;
static IPC_Channel *chanspair[2];
_______________________________________________________
Linux-HA-Dev: Linux-HA-***@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/
Kristoffer Grönlund
2016-08-12 05:13:32 UTC
Permalink
Post by Dejan Muhamedagic
Hi Kristoffer,
Post by Kristoffer Grönlund
Hi everyone (Lars and Dejan in particular),
Here are some minor patches for cluster-glue. The first one is an
attempt to get the stonith man page somewhat up to date, and the other
two are minor issues discovered when compiling cluster-glue using GCC
6.
Pushed just now the man page patch which was pending in my queue.
Will apply the other two too.
Thanks for the contribution!
Excellent, thank you!

Cheers,
Kristoffer
Post by Dejan Muhamedagic
Cheers,
Dejan
Post by Kristoffer Grönlund
Cheers,
Kristoffer
--
// Kristoffer Grönlund
changeset: 2820:13875518ed6b
parent: 2815:643ac28499bd
date: Wed Aug 10 12:13:13 2016 +0200
files: doc/stonith.xml.in
Low: stonith: Update man page with -E, -m parameters (bsc#970307)
diff --git a/doc/stonith.xml.in b/doc/stonith.xml.in
--- a/doc/stonith.xml.in
+++ b/doc/stonith.xml.in
@@ -7,22 +7,28 @@
<authorgroup>
<author>
- <firstname>Alan</firstname>
- <surname>Robertson</surname>
- <contrib>stonith</contrib>
+ <firstname>Alan</firstname>
+ <surname>Robertson</surname>
+ <contrib>stonith</contrib>
</author>
<author>
- <firstname>Simon</firstname>
- <surname>Horman</surname>
- <contrib>man page</contrib>
+ <firstname>Simon</firstname>
+ <surname>Horman</surname>
+ <contrib>man page</contrib>
</author>
<author>
- <firstname>Florian</firstname>
- <surname>Haas</surname>
- <contrib>man page</contrib>
+ <firstname>Florian</firstname>
+ <surname>Haas</surname>
+ <contrib>man page</contrib>
+ </author>
+ <author>
+ <firstname>Kristoffer</firstname>
+ <surname>Gronlund</surname>
+ <contrib>man page</contrib>
</author>
</authorgroup>
</refentryinfo>
@@ -44,12 +50,14 @@
<cmdsynopsis>
<command>stonith</command>
<arg choice="opt"><option>-s</option></arg>
+ <arg choice="opt"><option>-v</option></arg>
<arg choice="opt"><option>-h</option></arg>
<arg choice="plain"><option>-L</option></arg>
</cmdsynopsis>
<cmdsynopsis>
<command>stonith</command>
<arg choice="opt"><option>-s</option></arg>
+ <arg choice="opt"><option>-v</option></arg>
<arg choice="opt"><option>-h</option></arg>
<arg choice="plain"><option>-t</option> <replaceable>stonith-device-type</replaceable></arg>
<arg choice="plain"><option>-n</option></arg>
@@ -57,14 +65,24 @@
<cmdsynopsis>
<command>stonith</command>
<arg choice="opt"><option>-s</option></arg>
+ <arg choice="opt"><option>-v</option></arg>
+ <arg choice="opt"><option>-h</option></arg>
+ <arg choice="plain"><option>-t</option> <replaceable>stonith-device-type</replaceable></arg>
+ <arg choice="plain"><option>-m</option></arg>
+ </cmdsynopsis>
+ <cmdsynopsis>
+ <command>stonith</command>
+ <arg choice="opt"><option>-s</option></arg>
+ <arg choice="opt"><option>-v</option></arg>
<arg choice="opt"><option>-h</option></arg>
<arg choice="plain"><option>-t</option> <replaceable>stonith-device-type</replaceable></arg>
<group choice="req" rep="norepeat">
- <group choice="plain" rep="repeat">
- <arg choice="plain"><replaceable>name</replaceable>=<replaceable>value</replaceable></arg>
- </group>
- <arg choice="plain"><option>-p</option> <replaceable>stonith-device-parameters</replaceable></arg>
- <arg choice="plain"><option>-F</option> <replaceable>stonith-device-parameters-file</replaceable></arg>
+ <group choice="plain" rep="repeat">
+ <arg choice="plain"><replaceable>name</replaceable>=<replaceable>value</replaceable></arg>
+ </group>
+ <arg choice="plain"><option>-p</option> <replaceable>stonith-device-parameters</replaceable></arg>
+ <arg choice="plain"><option>-E</option></arg>
+ <arg choice="plain"><option>-F</option> <replaceable>stonith-device-parameters-file</replaceable></arg>
</group>
<arg choice="opt"><option>-c</option> <replaceable>count</replaceable></arg>
<arg choice="opt"><option>-l</option></arg>
@@ -73,22 +91,24 @@
<cmdsynopsis>
<command>stonith</command>
<arg choice="opt"><option>-s</option></arg>
+ <arg choice="opt"><option>-v</option></arg>
<arg choice="opt"><option>-h</option></arg>
<arg choice="plain"><option>-t</option> <replaceable>stonith-device-type</replaceable></arg>
<group choice="req" rep="norepeat">
- <group choice="plain" rep="repeat">
- <arg choice="plain"><replaceable>name</replaceable>=<replaceable>value</replaceable></arg>
- </group>
- <arg choice="plain"><option>-p</option> <replaceable>stonith-device-parameters</replaceable></arg>
- <arg choice="plain"><option>-F</option> <replaceable>stonith-device-parameters-file</replaceable></arg>
+ <group choice="plain" rep="repeat">
+ <arg choice="plain"><replaceable>name</replaceable>=<replaceable>value</replaceable></arg>
+ </group>
+ <arg choice="plain"><option>-p</option> <replaceable>stonith-device-parameters</replaceable></arg>
+ <arg choice="plain"><option>-E</option></arg>
+ <arg choice="plain"><option>-F</option> <replaceable>stonith-device-parameters-file</replaceable></arg>
</group>
<arg choice="opt"><option>-c</option> <replaceable>count</replaceable></arg>
<arg choice="opt"><option>-T</option>
<group choice="req">
- <arg choice="plain">reset</arg>
- <arg choice="plain">on</arg>
- <arg choice="plain">off</arg>
- </group>
+ <arg choice="plain">reset</arg>
+ <arg choice="plain">on</arg>
+ <arg choice="plain">off</arg>
+ </group>
</arg>
<arg><replaceable>nodename</replaceable></arg>
</cmdsynopsis>
@@ -108,145 +128,161 @@
<para>The following options are supported:</para>
<variablelist>
<varlistentry>
- <term>
- <option>-c</option> <replaceable>count</replaceable>
- </term>
- <listitem>
- <para>Perform any actions identified by the
- <option>-l</option>, <option>-S</option> and
- <option>-T</option> options <replaceable>count</replaceable>
- times.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-F</option> <replaceable>stonith-device-parameters-file</replaceable>
- </term>
- <listitem>
- <para>Path of file specifying parameters for a stonith
- device. To determine the syntax of the parameters file for a
- given device type run:</para>
- <screen><computeroutput># </computeroutput><userinput>stonith -t stonith-device-type -n</userinput></screen>
- <para>All of the listed parameters need to appear in order
- on a single line in the parameters file and be delimited by
- whitespace.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-h</option>
- </term>
- <listitem>
- <para>Display detailed information about a stonith device
- including description, configuration information, parameters
- and any other related information. When specified without a
- stonith-device-type, detailed information on all stonith
- devices is displayed.</para>
- <para>If you don't yet own a stonith device and want to know
- more about the ones we support, this information is likely
- to be helpful.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-L</option>
- </term>
- <listitem>
- <para>List the valid stonith device types, suitable for
- passing as an argument to the <option>-t</option>
- option.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-l</option>
- </term>
- <listitem>
- <para>List the hosts controlled by the stonith device.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-n</option>
- </term>
- <listitem>
- <para>Output the parameter names of the stonith device.</para>
- </listitem>
+ <term>
+ <option>-c</option> <replaceable>count</replaceable>
+ </term>
+ <listitem>
+ <para>Perform any actions identified by the
+ <option>-l</option>, <option>-S</option> and
+ <option>-T</option> options <replaceable>count</replaceable>
+ times.</para>
+ </listitem>
</varlistentry>
<varlistentry>
- <term>
- <replaceable>name</replaceable>=<replaceable>value</replaceable>
- </term>
- <listitem>
- <para>Parameter, in the form of a name/value pair, to pass
- directly to the stonith device. To determine the syntax of
- the parameters for a given device type run:</para>
- <screen><computeroutput># </computeroutput><userinput>stonith -t stonith-device-type -n</userinput></screen>
- <para>All of the listed parameter names need to be passed
- with their corresponding values.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-p</option> <replaceable>stonith-device-parameters</replaceable>
- </term>
- <listitem>
- <para>Parameters to pass directly to the stonith device. To
- determine the syntax of the parameters for a given device
- type run:</para>
- <screen><computeroutput># </computeroutput><userinput>stonith -t stonith-device-type -n</userinput></screen>
- <para>All of the listed parameter names need to appear in
- order and be delimited by whitespace.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-S</option>
- </term>
- <listitem>
- <para>Show the status of the stonith device.</para>
- </listitem>
+ <term>
+ <option>-E</option>
+ </term>
+ <listitem>
+ <para>Get parameters from the environment.</para>
+ </listitem>
</varlistentry>
<varlistentry>
- <term>
- <option>-s</option>
- </term>
- <listitem>
- <para>Silent operation. Suppress logging of error messages to standard error.</para>
- </listitem>
+ <term>
+ <option>-F</option> <replaceable>stonith-device-parameters-file</replaceable>
+ </term>
+ <listitem>
+ <para>Path of file specifying parameters for a stonith
+ device. To determine the syntax of the parameters file for a
+ given device type run:</para>
+ <screen><computeroutput># </computeroutput><userinput>stonith -t stonith-device-type -n</userinput></screen>
+ <para>All of the listed parameters need to appear in order
+ on a single line in the parameters file and be delimited by
+ whitespace.</para>
+ </listitem>
</varlistentry>
<varlistentry>
- <term>
- <option>-T</option> <replaceable>action</replaceable>
- </term>
- <listitem>
- <para>The stonith action to perform on the node identified
- by nodename. Chosen from <token>reset</token>,
- <token>on</token>, and <token>off</token>.</para>
- <note>
- <para>If a nodename is specified without the
- <option>-T</option> option, the stonith action defaults to
- <token>reset</token>.</para>
- </note>
- </listitem>
+ <term>
+ <option>-h</option>
+ </term>
+ <listitem>
+ <para>Display detailed information about a stonith device
+ including description, configuration information, parameters
+ and any other related information. When specified without a
+ stonith-device-type, detailed information on all stonith
+ devices is displayed.</para>
+ <para>If you don't yet own a stonith device and want to know
+ more about the ones we support, this information is likely
+ to be helpful.</para>
+ </listitem>
</varlistentry>
<varlistentry>
- <term>
- <option>-t</option> <replaceable>stonith-device-type</replaceable>
- </term>
- <listitem>
- <para>The type of the stonith device to be used to effect
- stonith. A list of supported devices for an installation may
- be obtained using the <option>-L</option> option.</para>
- </listitem>
+ <term>
+ <option>-L</option>
+ </term>
+ <listitem>
+ <para>List the valid stonith device types, suitable for
+ passing as an argument to the <option>-t</option>
+ option.</para>
+ </listitem>
</varlistentry>
<varlistentry>
- <term>
- <option>-v</option>
- </term>
- <listitem>
- <para>Ignored.</para>
- </listitem>
+ <term>
+ <option>-l</option>
+ </term>
+ <listitem>
+ <para>List the hosts controlled by the stonith device.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-n</option>
+ </term>
+ <listitem>
+ <para>Output the parameter names of the stonith device.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-m</option>
+ </term>
+ <listitem>
+ <para>Display meta-data of the stonith device type.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <replaceable>name</replaceable>=<replaceable>value</replaceable>
+ </term>
+ <listitem>
+ <para>Parameter, in the form of a name/value pair, to pass
+ directly to the stonith device. To determine the syntax of
+ the parameters for a given device type run:</para>
+ <screen><computeroutput># </computeroutput><userinput>stonith -t stonith-device-type -n</userinput></screen>
+ <para>All of the listed parameter names need to be passed
+ with their corresponding values.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-p</option> <replaceable>stonith-device-parameters</replaceable>
+ </term>
+ <listitem>
+ <para>Parameters to pass directly to the stonith device. To
+ determine the syntax of the parameters for a given device
+ type run:</para>
+ <screen><computeroutput># </computeroutput><userinput>stonith -t stonith-device-type -n</userinput></screen>
+ <para>All of the listed parameter names need to appear in
+ order and be delimited by whitespace.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-S</option>
+ </term>
+ <listitem>
+ <para>Show the status of the stonith device.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-s</option>
+ </term>
+ <listitem>
+ <para>Silent operation. Suppress logging of error messages to standard error.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-T</option> <replaceable>action</replaceable>
+ </term>
+ <listitem>
+ <para>The stonith action to perform on the node identified
+ by nodename. Chosen from <token>reset</token>,
+ <token>on</token>, and <token>off</token>.</para>
+ <note>
+ <para>If a nodename is specified without the
+ <option>-T</option> option, the stonith action defaults to
+ <token>reset</token>.</para>
+ </note>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-t</option> <replaceable>stonith-device-type</replaceable>
+ </term>
+ <listitem>
+ <para>The type of the stonith device to be used to effect
+ stonith. A list of supported devices for an installation may
+ be obtained using the <option>-L</option> option.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-v</option>
+ </term>
+ <listitem>
+ <para>Ignored.</para>
+ </listitem>
</varlistentry>
</variablelist>
</refsection>
changeset: 2821:f44088ebf874
date: Wed Aug 10 12:15:08 2016 +0200
files: lib/clplumbing/cl_msg.c
Low: clplumbing: Remove incorrect negation of n
diff --git a/lib/clplumbing/cl_msg.c b/lib/clplumbing/cl_msg.c
--- a/lib/clplumbing/cl_msg.c
+++ b/lib/clplumbing/cl_msg.c
@@ -1344,7 +1344,7 @@ cl_msg_add_list_str(struct ha_msg* msg,
if (n <= 0 || buf == NULL|| name ==NULL ||msg == NULL){
cl_log(LOG_ERR, "%s:"
"invalid parameter(%s)",
"msg is NULL",__FUNCTION__);
@@ -1397,7 +1397,7 @@ cl_msg_add_list_int(struct ha_msg* msg,
if (n <= 0 || buf == NULL|| name ==NULL ||msg == NULL){
cl_log(LOG_ERR, "cl_msg_add_list_int:"
"invalid parameter(%s)",
"msg is NULL");
changeset: 2822:aeadfcd8425a
tag: tip
date: Wed Aug 10 12:17:17 2016 +0200
files: logd/ha_logd.c
Dev: ha_logd: Remove unused variables
diff --git a/logd/ha_logd.c b/logd/ha_logd.c
--- a/logd/ha_logd.c
+++ b/logd/ha_logd.c
@@ -75,9 +75,6 @@
#define EOS '\0'
#define nullchk(a) ((a) ? (a) : "<null>")
-static const int logd_keepalive_ms = 1000;
-static const int logd_warntime_ms = 5000;
-static const int logd_deadtime_ms = 10000;
static gboolean verbose = FALSE;
static pid_t write_process_pid;
static IPC_Channel *chanspair[2];
--
// Kristoffer Grönlund
// ***@suse.com
_______________________________________________________
Linux-HA-Dev: Linux-HA-***@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-
Dejan Muhamedagic
2016-08-30 14:08:49 UTC
Permalink
Post by Kristoffer Grönlund
Post by Dejan Muhamedagic
Hi Kristoffer,
Post by Kristoffer Grönlund
Hi everyone (Lars and Dejan in particular),
Here are some minor patches for cluster-glue. The first one is an
attempt to get the stonith man page somewhat up to date, and the other
two are minor issues discovered when compiling cluster-glue using GCC
6.
Pushed just now the man page patch which was pending in my queue.
Will apply the other two too.
Thanks for the contribution!
Excellent, thank you!
Apparently, the patches as they are cannot be imported with "hg
import", i.e. the metadata gets lost. Did you do "hg export"? Can
you supply them with "hg export"?

Cheers,

Dejan
Post by Kristoffer Grönlund
Cheers,
Kristoffer
Post by Dejan Muhamedagic
Cheers,
Dejan
Post by Kristoffer Grönlund
Cheers,
Kristoffer
--
// Kristoffer Grönlund
changeset: 2820:13875518ed6b
parent: 2815:643ac28499bd
date: Wed Aug 10 12:13:13 2016 +0200
files: doc/stonith.xml.in
Low: stonith: Update man page with -E, -m parameters (bsc#970307)
diff --git a/doc/stonith.xml.in b/doc/stonith.xml.in
--- a/doc/stonith.xml.in
+++ b/doc/stonith.xml.in
@@ -7,22 +7,28 @@
<authorgroup>
<author>
- <firstname>Alan</firstname>
- <surname>Robertson</surname>
- <contrib>stonith</contrib>
+ <firstname>Alan</firstname>
+ <surname>Robertson</surname>
+ <contrib>stonith</contrib>
</author>
<author>
- <firstname>Simon</firstname>
- <surname>Horman</surname>
- <contrib>man page</contrib>
+ <firstname>Simon</firstname>
+ <surname>Horman</surname>
+ <contrib>man page</contrib>
</author>
<author>
- <firstname>Florian</firstname>
- <surname>Haas</surname>
- <contrib>man page</contrib>
+ <firstname>Florian</firstname>
+ <surname>Haas</surname>
+ <contrib>man page</contrib>
+ </author>
+ <author>
+ <firstname>Kristoffer</firstname>
+ <surname>Gronlund</surname>
+ <contrib>man page</contrib>
</author>
</authorgroup>
</refentryinfo>
@@ -44,12 +50,14 @@
<cmdsynopsis>
<command>stonith</command>
<arg choice="opt"><option>-s</option></arg>
+ <arg choice="opt"><option>-v</option></arg>
<arg choice="opt"><option>-h</option></arg>
<arg choice="plain"><option>-L</option></arg>
</cmdsynopsis>
<cmdsynopsis>
<command>stonith</command>
<arg choice="opt"><option>-s</option></arg>
+ <arg choice="opt"><option>-v</option></arg>
<arg choice="opt"><option>-h</option></arg>
<arg choice="plain"><option>-t</option> <replaceable>stonith-device-type</replaceable></arg>
<arg choice="plain"><option>-n</option></arg>
@@ -57,14 +65,24 @@
<cmdsynopsis>
<command>stonith</command>
<arg choice="opt"><option>-s</option></arg>
+ <arg choice="opt"><option>-v</option></arg>
+ <arg choice="opt"><option>-h</option></arg>
+ <arg choice="plain"><option>-t</option> <replaceable>stonith-device-type</replaceable></arg>
+ <arg choice="plain"><option>-m</option></arg>
+ </cmdsynopsis>
+ <cmdsynopsis>
+ <command>stonith</command>
+ <arg choice="opt"><option>-s</option></arg>
+ <arg choice="opt"><option>-v</option></arg>
<arg choice="opt"><option>-h</option></arg>
<arg choice="plain"><option>-t</option> <replaceable>stonith-device-type</replaceable></arg>
<group choice="req" rep="norepeat">
- <group choice="plain" rep="repeat">
- <arg choice="plain"><replaceable>name</replaceable>=<replaceable>value</replaceable></arg>
- </group>
- <arg choice="plain"><option>-p</option> <replaceable>stonith-device-parameters</replaceable></arg>
- <arg choice="plain"><option>-F</option> <replaceable>stonith-device-parameters-file</replaceable></arg>
+ <group choice="plain" rep="repeat">
+ <arg choice="plain"><replaceable>name</replaceable>=<replaceable>value</replaceable></arg>
+ </group>
+ <arg choice="plain"><option>-p</option> <replaceable>stonith-device-parameters</replaceable></arg>
+ <arg choice="plain"><option>-E</option></arg>
+ <arg choice="plain"><option>-F</option> <replaceable>stonith-device-parameters-file</replaceable></arg>
</group>
<arg choice="opt"><option>-c</option> <replaceable>count</replaceable></arg>
<arg choice="opt"><option>-l</option></arg>
@@ -73,22 +91,24 @@
<cmdsynopsis>
<command>stonith</command>
<arg choice="opt"><option>-s</option></arg>
+ <arg choice="opt"><option>-v</option></arg>
<arg choice="opt"><option>-h</option></arg>
<arg choice="plain"><option>-t</option> <replaceable>stonith-device-type</replaceable></arg>
<group choice="req" rep="norepeat">
- <group choice="plain" rep="repeat">
- <arg choice="plain"><replaceable>name</replaceable>=<replaceable>value</replaceable></arg>
- </group>
- <arg choice="plain"><option>-p</option> <replaceable>stonith-device-parameters</replaceable></arg>
- <arg choice="plain"><option>-F</option> <replaceable>stonith-device-parameters-file</replaceable></arg>
+ <group choice="plain" rep="repeat">
+ <arg choice="plain"><replaceable>name</replaceable>=<replaceable>value</replaceable></arg>
+ </group>
+ <arg choice="plain"><option>-p</option> <replaceable>stonith-device-parameters</replaceable></arg>
+ <arg choice="plain"><option>-E</option></arg>
+ <arg choice="plain"><option>-F</option> <replaceable>stonith-device-parameters-file</replaceable></arg>
</group>
<arg choice="opt"><option>-c</option> <replaceable>count</replaceable></arg>
<arg choice="opt"><option>-T</option>
<group choice="req">
- <arg choice="plain">reset</arg>
- <arg choice="plain">on</arg>
- <arg choice="plain">off</arg>
- </group>
+ <arg choice="plain">reset</arg>
+ <arg choice="plain">on</arg>
+ <arg choice="plain">off</arg>
+ </group>
</arg>
<arg><replaceable>nodename</replaceable></arg>
</cmdsynopsis>
@@ -108,145 +128,161 @@
<para>The following options are supported:</para>
<variablelist>
<varlistentry>
- <term>
- <option>-c</option> <replaceable>count</replaceable>
- </term>
- <listitem>
- <para>Perform any actions identified by the
- <option>-l</option>, <option>-S</option> and
- <option>-T</option> options <replaceable>count</replaceable>
- times.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-F</option> <replaceable>stonith-device-parameters-file</replaceable>
- </term>
- <listitem>
- <para>Path of file specifying parameters for a stonith
- device. To determine the syntax of the parameters file for a
- given device type run:</para>
- <screen><computeroutput># </computeroutput><userinput>stonith -t stonith-device-type -n</userinput></screen>
- <para>All of the listed parameters need to appear in order
- on a single line in the parameters file and be delimited by
- whitespace.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-h</option>
- </term>
- <listitem>
- <para>Display detailed information about a stonith device
- including description, configuration information, parameters
- and any other related information. When specified without a
- stonith-device-type, detailed information on all stonith
- devices is displayed.</para>
- <para>If you don't yet own a stonith device and want to know
- more about the ones we support, this information is likely
- to be helpful.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-L</option>
- </term>
- <listitem>
- <para>List the valid stonith device types, suitable for
- passing as an argument to the <option>-t</option>
- option.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-l</option>
- </term>
- <listitem>
- <para>List the hosts controlled by the stonith device.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-n</option>
- </term>
- <listitem>
- <para>Output the parameter names of the stonith device.</para>
- </listitem>
+ <term>
+ <option>-c</option> <replaceable>count</replaceable>
+ </term>
+ <listitem>
+ <para>Perform any actions identified by the
+ <option>-l</option>, <option>-S</option> and
+ <option>-T</option> options <replaceable>count</replaceable>
+ times.</para>
+ </listitem>
</varlistentry>
<varlistentry>
- <term>
- <replaceable>name</replaceable>=<replaceable>value</replaceable>
- </term>
- <listitem>
- <para>Parameter, in the form of a name/value pair, to pass
- directly to the stonith device. To determine the syntax of
- the parameters for a given device type run:</para>
- <screen><computeroutput># </computeroutput><userinput>stonith -t stonith-device-type -n</userinput></screen>
- <para>All of the listed parameter names need to be passed
- with their corresponding values.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-p</option> <replaceable>stonith-device-parameters</replaceable>
- </term>
- <listitem>
- <para>Parameters to pass directly to the stonith device. To
- determine the syntax of the parameters for a given device
- type run:</para>
- <screen><computeroutput># </computeroutput><userinput>stonith -t stonith-device-type -n</userinput></screen>
- <para>All of the listed parameter names need to appear in
- order and be delimited by whitespace.</para>
- </listitem>
- </varlistentry>
- <varlistentry>
- <term>
- <option>-S</option>
- </term>
- <listitem>
- <para>Show the status of the stonith device.</para>
- </listitem>
+ <term>
+ <option>-E</option>
+ </term>
+ <listitem>
+ <para>Get parameters from the environment.</para>
+ </listitem>
</varlistentry>
<varlistentry>
- <term>
- <option>-s</option>
- </term>
- <listitem>
- <para>Silent operation. Suppress logging of error messages to standard error.</para>
- </listitem>
+ <term>
+ <option>-F</option> <replaceable>stonith-device-parameters-file</replaceable>
+ </term>
+ <listitem>
+ <para>Path of file specifying parameters for a stonith
+ device. To determine the syntax of the parameters file for a
+ given device type run:</para>
+ <screen><computeroutput># </computeroutput><userinput>stonith -t stonith-device-type -n</userinput></screen>
+ <para>All of the listed parameters need to appear in order
+ on a single line in the parameters file and be delimited by
+ whitespace.</para>
+ </listitem>
</varlistentry>
<varlistentry>
- <term>
- <option>-T</option> <replaceable>action</replaceable>
- </term>
- <listitem>
- <para>The stonith action to perform on the node identified
- by nodename. Chosen from <token>reset</token>,
- <token>on</token>, and <token>off</token>.</para>
- <note>
- <para>If a nodename is specified without the
- <option>-T</option> option, the stonith action defaults to
- <token>reset</token>.</para>
- </note>
- </listitem>
+ <term>
+ <option>-h</option>
+ </term>
+ <listitem>
+ <para>Display detailed information about a stonith device
+ including description, configuration information, parameters
+ and any other related information. When specified without a
+ stonith-device-type, detailed information on all stonith
+ devices is displayed.</para>
+ <para>If you don't yet own a stonith device and want to know
+ more about the ones we support, this information is likely
+ to be helpful.</para>
+ </listitem>
</varlistentry>
<varlistentry>
- <term>
- <option>-t</option> <replaceable>stonith-device-type</replaceable>
- </term>
- <listitem>
- <para>The type of the stonith device to be used to effect
- stonith. A list of supported devices for an installation may
- be obtained using the <option>-L</option> option.</para>
- </listitem>
+ <term>
+ <option>-L</option>
+ </term>
+ <listitem>
+ <para>List the valid stonith device types, suitable for
+ passing as an argument to the <option>-t</option>
+ option.</para>
+ </listitem>
</varlistentry>
<varlistentry>
- <term>
- <option>-v</option>
- </term>
- <listitem>
- <para>Ignored.</para>
- </listitem>
+ <term>
+ <option>-l</option>
+ </term>
+ <listitem>
+ <para>List the hosts controlled by the stonith device.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-n</option>
+ </term>
+ <listitem>
+ <para>Output the parameter names of the stonith device.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-m</option>
+ </term>
+ <listitem>
+ <para>Display meta-data of the stonith device type.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <replaceable>name</replaceable>=<replaceable>value</replaceable>
+ </term>
+ <listitem>
+ <para>Parameter, in the form of a name/value pair, to pass
+ directly to the stonith device. To determine the syntax of
+ the parameters for a given device type run:</para>
+ <screen><computeroutput># </computeroutput><userinput>stonith -t stonith-device-type -n</userinput></screen>
+ <para>All of the listed parameter names need to be passed
+ with their corresponding values.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-p</option> <replaceable>stonith-device-parameters</replaceable>
+ </term>
+ <listitem>
+ <para>Parameters to pass directly to the stonith device. To
+ determine the syntax of the parameters for a given device
+ type run:</para>
+ <screen><computeroutput># </computeroutput><userinput>stonith -t stonith-device-type -n</userinput></screen>
+ <para>All of the listed parameter names need to appear in
+ order and be delimited by whitespace.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-S</option>
+ </term>
+ <listitem>
+ <para>Show the status of the stonith device.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-s</option>
+ </term>
+ <listitem>
+ <para>Silent operation. Suppress logging of error messages to standard error.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-T</option> <replaceable>action</replaceable>
+ </term>
+ <listitem>
+ <para>The stonith action to perform on the node identified
+ by nodename. Chosen from <token>reset</token>,
+ <token>on</token>, and <token>off</token>.</para>
+ <note>
+ <para>If a nodename is specified without the
+ <option>-T</option> option, the stonith action defaults to
+ <token>reset</token>.</para>
+ </note>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-t</option> <replaceable>stonith-device-type</replaceable>
+ </term>
+ <listitem>
+ <para>The type of the stonith device to be used to effect
+ stonith. A list of supported devices for an installation may
+ be obtained using the <option>-L</option> option.</para>
+ </listitem>
+ </varlistentry>
+ <varlistentry>
+ <term>
+ <option>-v</option>
+ </term>
+ <listitem>
+ <para>Ignored.</para>
+ </listitem>
</varlistentry>
</variablelist>
</refsection>
changeset: 2821:f44088ebf874
date: Wed Aug 10 12:15:08 2016 +0200
files: lib/clplumbing/cl_msg.c
Low: clplumbing: Remove incorrect negation of n
diff --git a/lib/clplumbing/cl_msg.c b/lib/clplumbing/cl_msg.c
--- a/lib/clplumbing/cl_msg.c
+++ b/lib/clplumbing/cl_msg.c
@@ -1344,7 +1344,7 @@ cl_msg_add_list_str(struct ha_msg* msg,
if (n <= 0 || buf == NULL|| name ==NULL ||msg == NULL){
cl_log(LOG_ERR, "%s:"
"invalid parameter(%s)",
"msg is NULL",__FUNCTION__);
@@ -1397,7 +1397,7 @@ cl_msg_add_list_int(struct ha_msg* msg,
if (n <= 0 || buf == NULL|| name ==NULL ||msg == NULL){
cl_log(LOG_ERR, "cl_msg_add_list_int:"
"invalid parameter(%s)",
"msg is NULL");
changeset: 2822:aeadfcd8425a
tag: tip
date: Wed Aug 10 12:17:17 2016 +0200
files: logd/ha_logd.c
Dev: ha_logd: Remove unused variables
diff --git a/logd/ha_logd.c b/logd/ha_logd.c
--- a/logd/ha_logd.c
+++ b/logd/ha_logd.c
@@ -75,9 +75,6 @@
#define EOS '\0'
#define nullchk(a) ((a) ? (a) : "<null>")
-static const int logd_keepalive_ms = 1000;
-static const int logd_warntime_ms = 5000;
-static const int logd_deadtime_ms = 10000;
static gboolean verbose = FALSE;
static pid_t write_process_pid;
static IPC_Channel *chanspair[2];
--
// Kristoffer Grönlund
_______________________________________________________
Linux-HA-Dev: Linux-HA-***@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/
Kristoffer Grönlund
2016-08-30 16:23:15 UTC
Permalink
Post by Dejan Muhamedagic
Apparently, the patches as they are cannot be imported with "hg
import", i.e. the metadata gets lost. Did you do "hg export"? Can
you supply them with "hg export"?
I thought I did.. let me try again.
Lars Ellenberg
2016-08-30 17:00:59 UTC
Permalink
Post by Kristoffer Grönlund
Post by Dejan Muhamedagic
Apparently, the patches as they are cannot be imported with "hg
import", i.e. the metadata gets lost. Did you do "hg export"? Can
you supply them with "hg export"?
I thought I did.. let me try again.
I think what Dejan was expecting is the result of
"hg export", which should look more like

# HG changeset patch
# User Lars Ellenberg <***@linbit.com>
# Date 1413480257 -7200
# Thu Oct 16 19:24:17 2014 +0200
# Node ID 0a7add1d9996b6d869d441da6c82fb7b8abcef4f
# Parent f2227d4971baed13958306b2c7cabec0eda93e82
fix syslogmsgfmt logging inconsistency for stderr/stdout
...

not the output of "hg log -v -p",
which looks like what you sent.

Though the formats are very similar,
and possibly could be massaged by hand, even,
hg import is best used with the output created by hg export.
Or sent dejan a hg bundle which he then can unbundle.

Cheers,

Lars
Post by Kristoffer Grönlund
changeset: 2820:13875518ed6b
parent: 2815:643ac28499bd
date: Wed Aug 10 12:13:13 2016 +0200
files: doc/stonith.xml.in
Low: stonith: Update man page with -E, -m parameters (bsc#970307)
...

_______________________________________________________
Linux-HA-Dev: Linux-HA-***@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/
Kristoffer Grönlund
2016-08-30 18:02:26 UTC
Permalink
Post by Lars Ellenberg
I think what Dejan was expecting is the result of
"hg export", which should look more like
# HG changeset patch
# Date 1413480257 -7200
# Thu Oct 16 19:24:17 2014 +0200
# Node ID 0a7add1d9996b6d869d441da6c82fb7b8abcef4f
# Parent f2227d4971baed13958306b2c7cabec0eda93e82
fix syslogmsgfmt logging inconsistency for stderr/stdout
...
not the output of "hg log -v -p",
which looks like what you sent.
Though the formats are very similar,
and possibly could be massaged by hand, even,
hg import is best used with the output created by hg export.
Or sent dejan a hg bundle which he then can unbundle.
Hmm, the patches I sent this time were produced by "hg export".

Maybe it's a matter of mercurial configuration? git has pushed all
memories of mercurial off the top of my mental stack. :/

Cheers,
Kristoffer
--
// Kristoffer Grönlund
// ***@suse.com
_______________________________________________________
Linux-HA-Dev: Linux-HA-***@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/lin
Dejan Muhamedagic
2016-08-31 08:37:27 UTC
Permalink
Post by Kristoffer Grönlund
Post by Lars Ellenberg
I think what Dejan was expecting is the result of
"hg export", which should look more like
# HG changeset patch
# Date 1413480257 -7200
# Thu Oct 16 19:24:17 2014 +0200
# Node ID 0a7add1d9996b6d869d441da6c82fb7b8abcef4f
# Parent f2227d4971baed13958306b2c7cabec0eda93e82
fix syslogmsgfmt logging inconsistency for stderr/stdout
...
not the output of "hg log -v -p",
which looks like what you sent.
Though the formats are very similar,
and possibly could be massaged by hand, even,
hg import is best used with the output created by hg export.
Or sent dejan a hg bundle which he then can unbundle.
Hmm, the patches I sent this time were produced by "hg export".
Maybe it's a matter of mercurial configuration? git has pushed all
memories of mercurial off the top of my mental stack. :/
Similar here. I was surprised that hg import would always put my
name etc and looked hard for some option to accept another
format but found nothing.

Maybe we should move the glue and heartbeat to github/clusterlabs
too?

Cheers,

Dejan
Post by Kristoffer Grönlund
Cheers,
Kristoffer
--
// Kristoffer Grönlund
_______________________________________________________
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/
_______________________________________________________
Linux-HA-Dev: Linux-HA-***@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-ha-dev
Home Page: http://linux-ha.org/
Kristoffer Grönlund
2016-08-31 13:11:28 UTC
Permalink
Post by Dejan Muhamedagic
Maybe we should move the glue and heartbeat to github/clusterlabs
too?
That would make things easier for me at least :)

Cheers,
Kristoffer
--
// Kristoffer Grönlund
// ***@suse.com
_______________________________________________________
Linux-HA-Dev: Linux-HA-***@lists.linux-ha.org
http://lists.linux-ha.org/mailman/listinfo/linux-h
Loading...