From db365c30d07f5faae4197e37148f64e75b54a7ec Mon Sep 17 00:00:00 2001 From: Raghavendra G Date: Tue, 31 Mar 2009 10:56:10 -0700 Subject: Enable glusterfs client to exit after a configured number of failed connects \ while fetching volume specification file from server Signed-off-by: Anand V. Avati --- glusterfsd/src/fetch-spec.c | 6 ++++++ glusterfsd/src/glusterfsd.c | 16 ++++++++++++++++ glusterfsd/src/glusterfsd.h | 1 + 3 files changed, 23 insertions(+) (limited to 'glusterfsd') diff --git a/glusterfsd/src/fetch-spec.c b/glusterfsd/src/fetch-spec.c index f914a416dd4..3e9712d8db4 100644 --- a/glusterfsd/src/fetch-spec.c +++ b/glusterfsd/src/fetch-spec.c @@ -71,6 +71,7 @@ fetch_notify (xlator_t *this_xl, int event, void *data, ...) { int ret = 0; call_frame_t *frame = NULL; + static int failed_connects = 0; switch (event) { @@ -85,6 +86,11 @@ fetch_notify (xlator_t *this_xl, int event, void *data, ...) 0); break; case GF_EVENT_CHILD_DOWN: + failed_connects++; + if (failed_connects + >= this_xl->ctx->cmd_args.max_connect_attempts) { + exit (1); + } break; default: ret = default_notify (this_xl, event, data); diff --git a/glusterfsd/src/glusterfsd.c b/glusterfsd/src/glusterfsd.c index 37eefa01bec..d4ac16a3083 100644 --- a/glusterfsd/src/glusterfsd.c +++ b/glusterfsd/src/glusterfsd.c @@ -85,6 +85,10 @@ static struct argp_option gf_options[] = { {"volfile-server", ARGP_VOLFILE_SERVER_KEY, "SERVER", 0, "Server to get the volume file from. This option overrides " "--volfile option"}, + {"volfile-max-fetch-attempts", ARGP_VOLFILE_MAX_FETCH_ATTEMPTS, + "MAX-ATTEMPTS", 0, "Maximum number of connect attempts to server. " + "This option should be provided with --volfile-server option" + "[default: 1]"}, {"volfile", ARGP_VOLUME_FILE_KEY, "VOLFILE", 0, "File to use as VOLUME_FILE [default: "DEFAULT_CLIENT_VOLUME_FILE" or " DEFAULT_SERVER_VOLUME_FILE"]"}, @@ -659,7 +663,19 @@ parse_opts (int key, char *arg, struct argp_state *state) case ARGP_VOLFILE_SERVER_KEY: cmd_args->volfile_server = strdup (arg); break; + + case ARGP_VOLFILE_MAX_FETCH_ATTEMPTS: + n = 0; + if (gf_string2uint_base10 (arg, &n) == 0) { + cmd_args->max_connect_attempts = n; + break; + } + + argp_failure (state, -1, 0, + "Invalid limit on connect attempts %s", arg); + break; + case ARGP_VOLUME_FILE_KEY: cmd_args->volume_file = strdup (arg); break; diff --git a/glusterfsd/src/glusterfsd.h b/glusterfsd/src/glusterfsd.h index 0d6423ab487..e06526b0664 100644 --- a/glusterfsd/src/glusterfsd.h +++ b/glusterfsd/src/glusterfsd.h @@ -71,6 +71,7 @@ enum argp_option_keys { #endif /* DARWIN */ ARGP_VOLFILE_ID_KEY = 143, ARGP_VOLFILE_CHECK_KEY = 144, + ARGP_VOLFILE_MAX_FETCH_ATTEMPTS = 145, }; /* Moved here from fetch-spec.h */ -- cgit