Attachment 314331 Details for Bug 459127 – proposed patch

proposed patch

fenced-skip.patch (text/plain), 2.88 KB, created by David Teigland on 2008-08-14 16:48:14 UTC

(hide)

Description:

Filename:

MIME Type:

Creator: David Teigland

Created: 2008-08-14 16:48:14 UTC

Size: 2.88 KB

patch

obsolete

>commit 8128da2dc6d87f48f7bbcf9d8b16b8bcc5a69158
>Author: David Teigland <teigland@redhat.com>
>Date:   Thu Aug 14 11:35:51 2008 -0500
>
>    fenced: add skip_undefined option
>    
>    bz 459127
>    
>    New fenced config option <fence_daemon skip_undefined="1"/>
>    would cause fenced to not attempt any fencing on nodes with zero
>    defined fence methods.  Currently, fenced tries, fails, and retries
>    (repeatedly) to fence a node with undefined fencing, which requires
>    an override.
>    
>    The primary use for this option would be asymmetric cluster configs
>    (http://sources.redhat.com/cluster/wiki/asymmetric_cluster_config)
>    where client/small/spectator nodes do not join the fence domain
>    and have no fencing configured.  The problem we have is that even with
>    no fencing configured, and not joining the fence domain, other
>    nodes may attempt (and fail) to fence these client nodes during
>    startup fencing.
>    
>    Signed-off-by: David Teigland <teigland@redhat.com>
>
>diff --git a/fence/fenced/agent.c b/fence/fenced/agent.c
>index 66ae77a..e3aab9a 100644
>--- a/fence/fenced/agent.c
>+++ b/fence/fenced/agent.c
>@@ -219,7 +219,7 @@ static int get_device(int cd, char *victim, char *method, int d, char **device)
> 	return error;
> }
> 
>-static int count_methods(int cd, char *victim)
>+int count_methods(int cd, char *victim)
> {
> 	char path[256], *name;
> 	int error, i;
>diff --git a/fence/fenced/fd.h b/fence/fenced/fd.h
>index 0b78d19..0283b51 100644
>--- a/fence/fenced/fd.h
>+++ b/fence/fenced/fd.h
>@@ -134,9 +134,11 @@ struct commandline
> 	int post_fail_delay;
> 	char *override_path;
> 	int8_t clean_start;
>+	int8_t skip_undefined;
> 	int8_t post_join_delay_opt;
> 	int8_t post_fail_delay_opt;
> 	int8_t clean_start_opt;
>+	int8_t skip_undefined_opt;
> 	int8_t override_path_opt;
> };
> 
>@@ -177,6 +179,7 @@ void do_recovery_done(fd_t *fd);
> 
> /* agent.c */
> int dispatch_fence_agent(char *victim, int force);
>+int count_methods(int cd, char *victim);
> 
> /* group.c */
> int setup_groupd(void);
>diff --git a/fence/fenced/main.c b/fence/fenced/main.c
>index c3d1720..a9a1b78 100644
>--- a/fence/fenced/main.c
>+++ b/fence/fenced/main.c
>@@ -117,6 +117,20 @@ static int setup_ccs(fd_t *fd)
> 			free(str);
> 	}
> 
>+	if (comline.skip_undefined_opt == FALSE) {
>+		str = NULL;
>+		memset(path, 0, 256);
>+		sprintf(path, "/cluster/fence_daemon/@skip_undefined");
>+
>+		error = ccs_get(cd, path, &str);
>+		if (!error)
>+			comline.skip_undefined = atoi(str);
>+		else
>+			comline.skip_undefined = DEFAULT_SKIP_UNDEFINED;
>+		if (str)
>+			free(str);
>+	}
>+
> 	if (comline.post_join_delay_opt == FALSE) {
> 		str = NULL;
> 		memset(path, 0, 256);
>@@ -179,6 +193,13 @@ static int setup_ccs(fd_t *fd)
> 		if (error || !name)
> 			break;
> 
>+		num_methods = count_methods(cd, name);
>+
>+		if (comline.skip_undefined && !num_methods) {
>+			log_debug("skip %s with zero methods", name);
>+			continue;
>+		}
>+
> 		add_complete_node(fd, 0, name);
> 		free(name);
> 		count++;

Actions: View

Attachments on bug 459127: 314331 | 314338