librados,libcephfs,...: use parse_env()

Signed-off-by: Sage Weil <sage@redhat.com>

diff --git a/src/libcephfs.cc b/src/libcephfs.cc
index a903318cbd98879f4d8ac3a5f83901ffc153e9ae..d5600d74cd4bca2adbaf03169bcebe676c1a8e06 100644 (file)
--- a/src/libcephfs.cc
+++ b/src/libcephfs.cc
@@ -209,11 +209,7 @@ public:
   int conf_parse_env(const char *name)
   {
     md_config_t *conf = cct->_conf;
-    vector<const char*> args;
-    env_to_vec(args, name);
-    int ret = conf->parse_argv(args, CONF_ENV);
-    if (ret)
-      return ret;
+    conf->parse_env(name);
     conf->apply_changes(nullptr);
     return 0;
   }

diff --git a/src/librados/librados.cc b/src/librados/librados.cc
index cac8c2360802ce485a5c590c720af75caccce5df..02eee4fa006704b72db141c4be7b0c371573a362 100644 (file)
--- a/src/librados/librados.cc
+++ b/src/librados/librados.cc
@@ -3003,13 +3003,7 @@ extern "C" int rados_conf_parse_env(rados_t cluster, const char *env)
   tracepoint(librados, rados_conf_parse_env_enter, cluster, env);
   librados::RadosClient *client = (librados::RadosClient *)cluster;
   md_config_t *conf = client->cct->_conf;
-  vector<const char*> args;
-  env_to_vec(args, env);
-  int ret = conf->parse_argv(args, CONF_ENV);
-  if (ret) {
-    tracepoint(librados, rados_conf_parse_env_exit, ret);
-    return ret;
-  }
+  conf->parse_env(env);
   conf->apply_changes(NULL);
   tracepoint(librados, rados_conf_parse_env_exit, 0);
   return 0;

diff --git a/src/test/librados_test_stub/LibradosTestStub.cc b/src/test/librados_test_stub/LibradosTestStub.cc
index 2d88f884432b5128e6636ff3e4f8ebe783f24d5a..e3fc1f200eab82f566e17f3b10e72a4f73c75420 100644 (file)
--- a/src/test/librados_test_stub/LibradosTestStub.cc
+++ b/src/test/librados_test_stub/LibradosTestStub.cc
@@ -152,13 +152,9 @@ extern "C" int rados_conf_parse_env(rados_t cluster, const char *var) {
   librados::TestRadosClient *client =
     reinterpret_cast<librados::TestRadosClient*>(cluster);
   md_config_t *conf = client->cct()->_conf;
-  std::vector<const char*> args;
-  env_to_vec(args, var);
-  int ret = conf->parse_argv(args);
-  if (ret == 0) {
-    conf->apply_changes(NULL);
-  }
-  return ret;
+  conf->parse_env(var);
+  conf->apply_changes(NULL);
+  return 0;
 }
 
 extern "C" int rados_conf_read_file(rados_t cluster, const char *path) {

diff --git a/src/tools/rbd_mirror/PoolReplayer.cc b/src/tools/rbd_mirror/PoolReplayer.cc
index b90f1642e3184385add640a245fb68292883a34a..9b932b2cfd543865fd407a9897d20afa8fd9ade3 100644 (file)
--- a/src/tools/rbd_mirror/PoolReplayer.cc
+++ b/src/tools/rbd_mirror/PoolReplayer.cc
@@ -422,7 +422,6 @@ int PoolReplayer::init_rados(const std::string &cluster_name,
 
   // librados::Rados::conf_parse_env
   std::vector<const char*> args;
-  env_to_vec(args, nullptr);
   r = cct->_conf->parse_argv(args);
   if (r < 0) {
     derr << "could not parse environment for " << description << ":"
@@ -430,6 +429,7 @@ int PoolReplayer::init_rados(const std::string &cluster_name,
     cct->put();
     return r;
   }
+  cct->_conf->parse_env();
 
   if (!m_args.empty()) {
     // librados::Rados::conf_parse_argv