summaryrefslogtreecommitdiffstats
path: root/libglusterfs
diff options
context:
space:
mode:
authorJeff Darcy <jdarcy@redhat.com>2012-04-19 16:53:19 -0400
committerAnand Avati <avati@redhat.com>2012-04-23 14:46:46 -0700
commit6fbf369d59cf87b903403699666d99232a372fef (patch)
tree26f1b27f8c41bbdce944afcf3f0a0be16af69964 /libglusterfs
parent7d0397c2144810c8a396e00187a6617873c94002 (diff)
Make option code quieter to make logs more readable.
A lot of this stuff is extremely uninformative, such as messages about default values that can be read from the code. Having them in the logs makes more important messages harder to find, so they're down to GF_DEBUG_TRACE now. Change-Id: I18921b3deac7a01ad94c50670b90342d1130837b Signed-off-by: Jeff Darcy <jdarcy@redhat.com> Reviewed-on: http://review.gluster.com/3198 Tested-by: Gluster Build System <jenkins@build.gluster.com> Reviewed-by: Kaleb KEITHLEY <kkeithle@redhat.com> Reviewed-by: Anand Avati <avati@redhat.com>
Diffstat (limited to 'libglusterfs')
-rw-r--r--libglusterfs/src/options.c10
-rw-r--r--libglusterfs/src/options.h2
-rw-r--r--libglusterfs/src/xlator.c10
3 files changed, 11 insertions, 11 deletions
diff --git a/libglusterfs/src/options.c b/libglusterfs/src/options.c
index 6da68fd999a..762f15f99b1 100644
--- a/libglusterfs/src/options.c
+++ b/libglusterfs/src/options.c
@@ -81,7 +81,7 @@ xlator_option_validate_int (xlator_t *xl, const char *key, const char *value,
}
if ((opt->min == 0) && (opt->max == 0)) {
- gf_log (xl->name, GF_LOG_DEBUG,
+ gf_log (xl->name, GF_LOG_TRACE,
"no range check required for 'option %s %s'",
key, value);
ret = 0;
@@ -123,7 +123,7 @@ xlator_option_validate_sizet (xlator_t *xl, const char *key, const char *value,
}
if ((opt->min == 0) && (opt->max == 0)) {
- gf_log (xl->name, GF_LOG_DEBUG,
+ gf_log (xl->name, GF_LOG_TRACE,
"no range check required for 'option %s %s'",
key, value);
ret = 0;
@@ -336,7 +336,7 @@ xlator_option_validate_percent_or_sizet (xlator_t *xl, const char *key,
}
/* Check the range */
if ((opt->min == 0) && (opt->max == 0)) {
- gf_log (xl->name, GF_LOG_DEBUG,
+ gf_log (xl->name, GF_LOG_TRACE,
"no range check required for "
"'option %s %s'",
key, value);
@@ -390,7 +390,7 @@ xlator_option_validate_time (xlator_t *xl, const char *key, const char *value,
}
if ((opt->min == 0) && (opt->max == 0)) {
- gf_log (xl->name, GF_LOG_DEBUG,
+ gf_log (xl->name, GF_LOG_TRACE,
"no range check required for "
"'option %s %s'",
key, value);
@@ -442,7 +442,7 @@ xlator_option_validate_double (xlator_t *xl, const char *key, const char *value,
}
if ((opt->min == 0) && (opt->max == 0)) {
- gf_log (xl->name, GF_LOG_DEBUG,
+ gf_log (xl->name, GF_LOG_TRACE,
"no range check required for 'option %s %s'",
key, value);
ret = 0;
diff --git a/libglusterfs/src/options.h b/libglusterfs/src/options.h
index 722cc3b7a5d..943c303657a 100644
--- a/libglusterfs/src/options.h
+++ b/libglusterfs/src/options.h
@@ -143,7 +143,7 @@ xlator_option_init_##type (xlator_t *this, dict_t *options, char *key, \
return 0; \
} \
if (value == def_value) { \
- gf_log (this->name, GF_LOG_DEBUG, \
+ gf_log (this->name, GF_LOG_TRACE, \
"option %s using default value %s", \
key, value); \
} else { \
diff --git a/libglusterfs/src/xlator.c b/libglusterfs/src/xlator.c
index 1bff11fecde..1641a1909f6 100644
--- a/libglusterfs/src/xlator.c
+++ b/libglusterfs/src/xlator.c
@@ -238,23 +238,23 @@ xlator_dynload (xlator_t *xl)
}
if (!(xl->notify = dlsym (handle, "notify"))) {
- gf_log ("xlator", GF_LOG_DEBUG,
+ gf_log ("xlator", GF_LOG_TRACE,
"dlsym(notify) on %s -- neglecting", dlerror ());
}
if (!(xl->dumpops = dlsym (handle, "dumpops"))) {
- gf_log ("xlator", GF_LOG_DEBUG,
+ gf_log ("xlator", GF_LOG_TRACE,
"dlsym(dumpops) on %s -- neglecting", dlerror ());
}
if (!(xl->mem_acct_init = dlsym (handle, "mem_acct_init"))) {
- gf_log (xl->name, GF_LOG_DEBUG,
+ gf_log (xl->name, GF_LOG_TRACE,
"dlsym(mem_acct_init) on %s -- neglecting",
dlerror ());
}
if (!(xl->reconfigure = dlsym (handle, "reconfigure"))) {
- gf_log ("xlator", GF_LOG_DEBUG,
+ gf_log ("xlator", GF_LOG_TRACE,
"dlsym(reconfigure) on %s -- neglecting",
dlerror());
}
@@ -268,7 +268,7 @@ xlator_dynload (xlator_t *xl)
if (!(vol_opt->given_opt = dlsym (handle, "options"))) {
dlerror ();
- gf_log (xl->name, GF_LOG_DEBUG,
+ gf_log (xl->name, GF_LOG_TRACE,
"Strict option validation not enforced -- neglecting");
}
list_add_tail (&vol_opt->list, &xl->volume_options);