]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
Revert "ceph_context: re-expand admin_socket metavariables in child process" 18545/head
authorPatrick Donnelly <pdonnell@redhat.com>
Wed, 25 Oct 2017 20:29:27 +0000 (13:29 -0700)
committerPatrick Donnelly <pdonnell@redhat.com>
Wed, 25 Oct 2017 20:29:42 +0000 (13:29 -0700)
This commit introduces a regression where the admin socket specified in
ceph.conf overrides command line arguments or environment variables.

This reverts commit 1630f4bbef3a3ff6efb56acbaa9b8786b945b1e6.

Signed-off-by: Patrick Donnelly <pdonnell@redhat.com>
src/common/ceph_context.cc

index 96b78e2fa2e76097d7660c39da1ea090aed75097..2f5256baf5479c0f58ef05de7338d29d3b0955bf 100644 (file)
@@ -741,19 +741,6 @@ void CephContext::start_service_thread()
   if (_conf->log_flush_on_exit)
     _log->set_flush_on_exit();
 
-  if (_conf->admin_socket.length()) {
-    // Reset admin_socket raw value if it is defined in conffile.
-    // Just in case it used metavarirables (e.g, $pid) which could be expanded 
-    // again to the correct value here in child process.
-    std::vector <std::string> my_sections;
-    std::string admin_socket;
-    _conf->get_my_sections(my_sections);
-    if (_conf->get_val_from_conf_file(my_sections, "admin_socket",
-        admin_socket, false) == 0) {
-      _conf->set_val("admin_socket", admin_socket);
-    }
-  }
-
   // Trigger callbacks on any config observers that were waiting for
   // it to become safe to start threads.
   _conf->set_val("internal_safe_to_start_threads", "true");