rgw: revert PR #8765

As talk with cbodley in PR(#16716), we shouldn't not mix such dangerous
configuration reload with periodically SIGHUP for logs zip.

Signed-off-by: fang yuxiang fang.yuxiang@eisoo.com

diff --git a/src/rgw/rgw_main.cc b/src/rgw/rgw_main.cc
index 85ec5fec0c0ecf0174841a2255b1598e9f0bcac2..ecbf23a4161ade74ab9c5780346b72b5a769c528 100644 (file)
--- a/src/rgw/rgw_main.cc
+++ b/src/rgw/rgw_main.cc
@@ -189,18 +189,6 @@ static RGWRESTMgr *rest_filter(RGWRados *store, int dialect, RGWRESTMgr *orig)
   return sync_module->get_rest_filter(dialect, orig);
 }
 
-RGWRealmReloader *preloader = NULL;
-
-static void reloader_handler(int signum)
-{
-  if (preloader) {
-    bufferlist bl;
-    bufferlist::iterator p = bl.begin();
-    preloader->handle_notify(RGWRealmNotify::Reload, p);
-  }
-  sighup_handler(signum);
-}
-
 /*
  * start up the RADOS connection and then handle HTTP messages as they come in
  */
@@ -465,7 +453,7 @@ int main(int argc, const char **argv)
   }
 
   init_async_signal_handler();
-  register_async_signal_handler(SIGHUP, reloader_handler);
+  register_async_signal_handler(SIGHUP, sighup_handler);
   register_async_signal_handler(SIGTERM, handle_sigterm);
   register_async_signal_handler(SIGINT, handle_sigterm);
   register_async_signal_handler(SIGUSR1, handle_sigterm);
@@ -561,8 +549,6 @@ int main(int argc, const char **argv)
   RGWFrontendPauser pauser(fes, &pusher);
   RGWRealmReloader reloader(store, service_map_meta, &pauser);
 
-  preloader = &reloader;
-
   RGWRealmWatcher realm_watcher(g_ceph_context, store->realm);
   realm_watcher.add_watcher(RGWRealmNotify::Reload, reloader);
   realm_watcher.add_watcher(RGWRealmNotify::ZonesNeedPeriod, pusher);
@@ -596,7 +582,7 @@ int main(int argc, const char **argv)
     delete fec;
   }
 
-  unregister_async_signal_handler(SIGHUP, reloader_handler);
+  unregister_async_signal_handler(SIGHUP, sighup_handler);
   unregister_async_signal_handler(SIGTERM, handle_sigterm);
   unregister_async_signal_handler(SIGINT, handle_sigterm);
   unregister_async_signal_handler(SIGUSR1, handle_sigterm);