From: Sage Weil <sage@redhat.com>
Date: Tue, 3 Dec 2019 23:54:44 +0000 (-0600)
Subject: common/config: less noise about configs from mon we can't apply
X-Git-Tag: v14.2.12~128^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=e6b1e524def1c7532e4b151398bbae0858cd8e67;p=ceph.git

common/config: less noise about configs from mon we can't apply

It is normal to set things in teh mon config database that can't be absorbed
by clients/daemons at runtime.  Things like ms_type can/should be set there
(so that they, for instance, help produce the 'config minimal-conf') but
we don't need daemons to spit out error messages every time they get a config
update.

Signed-off-by: Sage Weil <sage@redhat.com>
(cherry picked from commit c7e3dd0af4172633de6cd93126818abd8fde91ab)
---

diff --git a/src/common/config.cc b/src/common/config.cc
index 8816512cd6ce..14646cd71c3f 100644
--- a/src/common/config.cc
+++ b/src/common/config.cc
@@ -295,8 +295,8 @@ int md_config_t::set_mon_vals(CephContext *cct,
     std::string err;
     int r = _set_val(values, tracker, i.second, *o, CONF_MON, &err);
     if (r < 0) {
-      lderr(cct) << __func__ << " failed to set " << i.first << " = "
-		 << i.second << ": " << err << dendl;
+      ldout(cct, 4) << __func__ << " failed to set " << i.first << " = "
+		    << i.second << ": " << err << dendl;
       ignored_mon_values.emplace(i);
     } else if (r == ConfigValues::SET_NO_CHANGE ||
 	       r == ConfigValues::SET_NO_EFFECT) {