deprecate jaegergrpc (#5428)

* deprecate jaegergrpc * Update CHANGELOG.md * Update internal/signalfx-agent/pkg/monitors/jaegergrpc/metadata.yaml Co-authored-by: Curtis Robert <crobert@splunk.com> * Update internal/signalfx-agent/pkg/monitors/jaegergrpc/monitor.go Co-authored-by: Curtis Robert <crobert@splunk.com> --------- Co-authored-by: Curtis Robert <crobert@splunk.com>
signalfx · Sep 30, 2024 · bf8871e · bf8871e
1 parent 8386718
commit bf8871e
Show file tree

Hide file tree

Showing 3 changed files with 8 additions and 0 deletions.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -2,6 +2,10 @@
 
 ## Unreleased
 
+### 🚩Deprecations 🚩
+
+- (Splunk) Deprecate the jaegergrpc monitor ([#5428](https://github.com/signalfx/splunk-otel-collector/pull/5428))
+
 ## v0.110.0
 
 This Splunk OpenTelemetry Collector release includes changes from the [opentelemetry-collector v0.110.0](https://github.com/open-telemetry/opentelemetry-collector/releases/tag/v0.110.0) and the [opentelemetry-collector-contrib v0.110.0](https://github.com/open-telemetry/opentelemetry-collector-contrib/releases/tag/v0.110.0) releases where appropriate.

diff --git a/internal/signalfx-agent/pkg/monitors/jaegergrpc/metadata.yaml b/internal/signalfx-agent/pkg/monitors/jaegergrpc/metadata.yaml
@@ -1,6 +1,9 @@
 monitors:
 - dimensions:
   doc: |
+    The jaegergrpc monitor is deprecated and will be removed in a future release.
+    Use the Jaeger receiver instead.
+    
     Runs a GRPC server that listens for Jaeger trace batches
     and forwards them to SignalFx (or the configured ingest host in the
     `writer` section of the agent config).  By default, the server listens on

diff --git a/internal/signalfx-agent/pkg/monitors/jaegergrpc/monitor.go b/internal/signalfx-agent/pkg/monitors/jaegergrpc/monitor.go
@@ -144,6 +144,7 @@ func (m *Monitor) setupListener(ctx context.Context, conf *Config) (net.Listener
 // Configure the monitor and kick off volume metric syncing
 func (m *Monitor) Configure(conf *Config) error {
 	m.logger = utils.NewThrottledLogger(log.WithFields(log.Fields{"monitorType": monitorType, "monitorID": conf.MonitorID}), 30*time.Second)
+	m.logger.Warn("[NOTICE] The jaegergrpc monitor is deprecated and will be removed in a future release. Use the Jaeger receiver instead.")
 	ctx, cancel := context.WithCancel(context.Background())
 	m.cancel = cancel