Skip to content

Commit

Permalink
deprecate jaegergrpc
Browse files Browse the repository at this point in the history
  • Loading branch information
atoulme committed Sep 30, 2024
1 parent dcf6ad7 commit 711b9fe
Show file tree
Hide file tree
Showing 3 changed files with 8 additions and 0 deletions.
4 changes: 4 additions & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,10 @@

## Unreleased

### 🚩Deprecations 🚩

- (Splunk) Deprecate the jaegergrpc monitor ([#](https://github.com/signalfx/splunk-otel-collector/pull/))

## 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.
Expand Down
3 changes: 3 additions & 0 deletions internal/signalfx-agent/pkg/monitors/jaegergrpc/metadata.yaml
Original file line number Diff line number Diff line change
@@ -1,6 +1,9 @@
monitors:
- dimensions:
doc: |
The jaegergrpc monitor is deprecated and will be removed in a future release.
Use the jaegerreceiver 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
Expand Down
1 change: 1 addition & 0 deletions internal/signalfx-agent/pkg/monitors/jaegergrpc/monitor.go
Original file line number Diff line number Diff line change
Expand Up @@ -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 jaegerreceiver instead.")
ctx, cancel := context.WithCancel(context.Background())
m.cancel = cancel

Expand Down

0 comments on commit 711b9fe

Please sign in to comment.