crimson: remove unused include and forward decl

and add those used

Signed-off-by: Kefu Chai <kchai@redhat.com>

diff --git a/src/crimson/admin/admin_socket.cc b/src/crimson/admin/admin_socket.cc
index 2a85ac9435a8cee24ad04b40a66071015342b7e2..1f7fba8c75820d5eda6b882c88e88b8319576df5 100644 (file)
--- a/src/crimson/admin/admin_socket.cc
+++ b/src/crimson/admin/admin_socket.cc
@@ -3,7 +3,6 @@
 
 #include "crimson/admin/admin_socket.h"
 
-#include <boost/algorithm/string.hpp>
 #include <fmt/format.h>
 #include <seastar/net/api.hh>
 #include <seastar/net/inet_address.hh>

diff --git a/src/crimson/admin/admin_socket.h b/src/crimson/admin/admin_socket.h
index e7d7eb0f23fce0cf03a95bdcc46e2bfa10fe63e9..400b36aada3db0b936a05426513dfa9be484e7c7 100644 (file)
--- a/src/crimson/admin/admin_socket.h
+++ b/src/crimson/admin/admin_socket.h
@@ -20,12 +20,8 @@
 
 #include "common/cmdparse.h"
 
-class CephContext;
-
 using namespace std::literals;
 
-inline constexpr auto CEPH_ADMIN_SOCK_VERSION = "2"sv;
-
 namespace crimson::admin {
 
 class AdminSocket;

diff --git a/src/crimson/admin/osd_admin.cc b/src/crimson/admin/osd_admin.cc
index 760004180f76b3a04780b4e4433072a4c5284668..e271535f2d8e3b3e1d639ac393ad1adfae9eb019 100644 (file)
--- a/src/crimson/admin/osd_admin.cc
+++ b/src/crimson/admin/osd_admin.cc
@@ -3,9 +3,6 @@
 
 #include "crimson/admin/osd_admin.h"
 
-#include <iostream>
-
-#include <boost/algorithm/string.hpp>
 #include <fmt/format.h>
 #include <seastar/core/do_with.hh>
 #include <seastar/core/future.hh>

diff --git a/src/crimson/admin/osd_admin.h b/src/crimson/admin/osd_admin.h
index c40f38f893e65c5e41b6dd5c5a3424131e7b7b31..7c79b796457aa1771ecd99e75aee7467d1d80473 100644 (file)
--- a/src/crimson/admin/osd_admin.h
+++ b/src/crimson/admin/osd_admin.h
@@ -3,8 +3,7 @@
 #pragma once
 
 #include <memory>
-
-#include "crimson/common/config_proxy.h"
+#include <seastar/core/future.hh>
 
 namespace crimson::osd {
 class OSD;

diff --git a/src/crimson/osd/object_context.cc b/src/crimson/osd/object_context.cc
index 0c70df5f7fbca1c7fd19032c5de993208c0788a4..6f20b62a1c4c48fc844c37ac3147738ca7ab0278 100644 (file)
--- a/src/crimson/osd/object_context.cc
+++ b/src/crimson/osd/object_context.cc
@@ -3,6 +3,9 @@
 
 #include "crimson/osd/object_context.h"
 
+#include "common/Formatter.h"
+#include "crimson/common/config_proxy.h"
+
 namespace crimson::osd {
 
 void ObjectContext::dump_detail(Formatter *f) const
@@ -28,7 +31,7 @@ const char** ObjectContextRegistry::get_tracked_conf_keys() const
 }
 
 void ObjectContextRegistry::handle_conf_change(
-  const ConfigProxy& conf,
+  const crimson::common::ConfigProxy& conf,
   const std::set <std::string> &changed)
 {
   obc_lru.set_target_size(conf.get_val<uint64_t>("crimson_osd_obc_lru_size"));

diff --git a/src/crimson/osd/object_context.h b/src/crimson/osd/object_context.h
index 2d61963e645319997f70f98e859b77d24d21448b..6ff49dedb3f6f9f0cc6e03f7b11492de5dc98103 100644 (file)
--- a/src/crimson/osd/object_context.h
+++ b/src/crimson/osd/object_context.h
@@ -3,19 +3,24 @@
 
 #pragma once
 
+#include <map>
+#include <optional>
+#include <utility>
 #include <seastar/core/shared_future.hh>
 #include <seastar/core/shared_ptr.hh>
 
-#include <boost/intrusive_ptr.hpp>
-#include <boost/intrusive/list.hpp>
-#include <boost/intrusive/set.hpp>
-#include <boost/smart_ptr/intrusive_ref_counter.hpp>
-
 #include "common/intrusive_lru.h"
 #include "osd/object_state.h"
-#include "crimson/common/config_proxy.h"
 #include "crimson/osd/osd_operation.h"
 
+namespace ceph {
+  class Formatter;
+}
+
+namespace crimson::common {
+  class ConfigProxy;
+}
+
 namespace crimson::osd {
 
 class Watch;
@@ -227,7 +232,7 @@ public:
   }
 
   const char** get_tracked_conf_keys() const final;
-  void handle_conf_change(const ConfigProxy& conf,
+  void handle_conf_change(const crimson::common::ConfigProxy& conf,
                           const std::set <std::string> &changed) final;
 };
 

diff --git a/src/crimson/osd/ops_executer.h b/src/crimson/osd/ops_executer.h
index a66df94a502986168c3bd8f1bd7ca894e3c2d584..dd7da88ac5039d35d580dc401ea221b3663b4a1e 100644 (file)
--- a/src/crimson/osd/ops_executer.h
+++ b/src/crimson/osd/ops_executer.h
@@ -4,7 +4,6 @@
 #pragma once
 
 #include <memory>
-#include <optional>
 #include <type_traits>
 #include <boost/intrusive_ptr.hpp>
 #include <boost/smart_ptr/intrusive_ref_counter.hpp>

diff --git a/src/crimson/osd/osd.h b/src/crimson/osd/osd.h
index 94521ce664e8145a54c853317fbbf76a52f40a0d..0630793f44f42406ab6f1a0fa882d8564770943d 100644 (file)
--- a/src/crimson/osd/osd.h
+++ b/src/crimson/osd/osd.h
@@ -3,9 +3,6 @@
 
 #pragma once
 
-#include <map>
-#include <tuple>
-#include <optional>
 #include <seastar/core/future.hh>
 #include <seastar/core/shared_future.hh>
 #include <seastar/core/gate.hh>