From: Deepika Upadhyay <deepika@koor.tech>
Date: Sat, 8 Apr 2023 06:40:21 +0000 (+0530)
Subject: src/common: add debug logs to opentelementry tracing
X-Git-Tag: v19.0.0~551^2~3
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=2ad51e35cc8cd84c6bd8afffc274b9d525a2899a;p=ceph.git

src/common: add debug logs to opentelementry tracing

this is needed to investigate in case there is init, deployment or trace
formation problem.

Signed-off-by: Deepika Upadhyay <deepika@koor.tech>
---

diff --git a/src/common/tracer.cc b/src/common/tracer.cc
index ffabc0b20a11..1d6901b895bb 100644
--- a/src/common/tracer.cc
+++ b/src/common/tracer.cc
@@ -4,12 +4,18 @@
 #include "common/ceph_context.h"
 #include "global/global_context.h"
 #include "tracer.h"
+#include "common/debug.h"
 
 #ifdef HAVE_JAEGER
 #include "opentelemetry/sdk/trace/batch_span_processor.h"
 #include "opentelemetry/sdk/trace/tracer_provider.h"
 #include "opentelemetry/exporters/jaeger/jaeger_exporter.h"
 
+#define dout_context g_ceph_context
+#define dout_subsys ceph_subsys_osd
+#undef dout_prefix
+#define dout_prefix *_dout << "opentelemetry_jaeger_tracing "
+
 namespace tracing {
 
 const opentelemetry::nostd::shared_ptr<opentelemetry::trace::Tracer> Tracer::noop_tracer = opentelemetry::trace::Provider::GetTracerProvider()->GetTracer("no-op", OPENTELEMETRY_SDK_VERSION);
@@ -23,6 +29,7 @@ Tracer::Tracer(opentelemetry::nostd::string_view service_name) {
 
 void Tracer::init(opentelemetry::nostd::string_view service_name) {
   if (!tracer) {
+    dout(3) << "tracer was not loaded, initializing tracing" << dendl;
     opentelemetry::exporter::jaeger::JaegerExporterOptions exporter_options;
     if (g_ceph_context) {
       exporter_options.server_port = g_ceph_context->_conf.get_val<int64_t>("jaeger_agent_port");
@@ -39,13 +46,16 @@ void Tracer::init(opentelemetry::nostd::string_view service_name) {
 
 jspan Tracer::start_trace(opentelemetry::nostd::string_view trace_name) {
   if (is_enabled()) {
+    dout(20) << "start trace for " << trace_name << " " << dendl;
     return tracer->StartSpan(trace_name);
   }
   return noop_span;
 }
 
 jspan Tracer::start_trace(opentelemetry::nostd::string_view trace_name, bool trace_is_enabled) {
+  dout(20) << "start trace enabled " << trace_is_enabled << " " << dendl;
   if (trace_is_enabled) {
+    dout(20) << "start trace for " << trace_name << " " << dendl;
     return tracer->StartSpan(trace_name);
   }
   return noop_tracer->StartSpan(trace_name);
@@ -55,6 +65,7 @@ jspan Tracer::add_span(opentelemetry::nostd::string_view span_name, const jspan&
   if (is_enabled() && parent_span->IsRecording()) {
     opentelemetry::trace::StartSpanOptions span_opts;
     span_opts.parent = parent_span->GetContext();
+    dout(20) << "adding span " << span_name << " " << dendl;
     return tracer->StartSpan(span_name, span_opts);
   }
   return noop_span;
@@ -64,6 +75,7 @@ jspan Tracer::add_span(opentelemetry::nostd::string_view span_name, const jspan_
   if (is_enabled() && parent_ctx.IsValid()) {
     opentelemetry::trace::StartSpanOptions span_opts;
     span_opts.parent = parent_ctx;
+    dout(20) << "adding span " << span_name << " " << dendl;
     return tracer->StartSpan(span_name, span_opts);
   }
   return noop_span;