jaeger: make tracing default turned off, until teuthology integration

Signed-off-by: Deepika Upadhyay <dupadhya@redhat.com>
ceph · Dec 10, 2020 · 440f7aa · 440f7aa
1 parent 2ad7dfc
commit 440f7aa
Show file tree

Hide file tree

Showing 4 changed files with 8 additions and 5 deletions.
diff --git a/CMakeLists.txt b/CMakeLists.txt
@@ -364,7 +364,7 @@ if(WITH_BLKIN)
   include_directories(SYSTEM src/blkin/blkin-lib)
 endif(WITH_BLKIN)
 
-option(WITH_JAEGER "Enable jaegertracing and it's dependent libraries" ON)
+option(WITH_JAEGER "Enable jaegertracing and it's dependent libraries" OFF)
 if(WITH_JAEGER)
   set(HAVE_JAEGER TRUE)
 endif()

diff --git a/ceph.spec.in b/ceph.spec.in
@@ -61,7 +61,7 @@
 %endif
 %endif
 %bcond_with seastar
-%bcond_without jaeger
+%bcond_with jaeger
 %if 0%{?fedora} || 0%{?suse_version} >= 1500
 # distros that ship cmd2 and/or colorama
 %bcond_without cephfs_shell

diff --git a/debian/libjaeger.install b/debian/libjaeger.install
@@ -1,3 +1,3 @@
-usr/lib/libopentracing.so.*
-usr/lib/libjaegertracing.so.*
-usr/lib/libthrift.so.*
+#usr/lib/libopentracing.so.*
+#usr/lib/libjaegertracing.so.*
+#usr/lib/libthrift.so.*
diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
@@ -396,6 +396,9 @@ if(WITH_JAEGER)
     set(jaeger_libs ${CMAKE_BINARY_DIR}/external/lib/libyaml-cpp.so
 		    ${CMAKE_BINARY_DIR}/external/lib/libyaml-cpp.so.0.6
 		    ${CMAKE_BINARY_DIR}/external/lib/libyaml-cpp.so.0.6.2)
+    #customize libjaeger.install
+    execute_process(COMMAND bash -c "sed -i 's/#//' debian/libjaeger.install"
+		    WORKING_DIRECTORY ${CMAKE_SOURCE_DIR})
     execute_process(COMMAND bash -c "grep -q 'yaml-cpp' debian/libjaeger.install || echo 'usr/lib/libyaml-cpp.so.*' >> debian/libjaeger.install"
 		    WORKING_DIRECTORY ${CMAKE_SOURCE_DIR})
   endif()