Skip to content

feat(config): support new bee OTLP tracing flags#598

Open
martinconic wants to merge 3 commits into
masterfrom
feat/bee-otlp-tracing-config
Open

feat(config): support new bee OTLP tracing flags#598
martinconic wants to merge 3 commits into
masterfrom
feat/bee-otlp-tracing-config

Conversation

@martinconic
Copy link
Copy Markdown
Contributor

@martinconic martinconic commented May 27, 2026

Paired change for the bee OpenTelemetry migration ethersphere/bee#5456 Bee renamed its tracing CLI flags and dropped the legacy Jaeger thrift UDP endpoint; without this PR, beekeeper-managed deployments would fail to start once the bee PR ships.

Copy link
Copy Markdown
Member

@gacevicljubisa gacevicljubisa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just to note, that beekeeper's own tracer still target Jaeger UDP, but it is out of scope for this PR.

Comment thread pkg/config/bee.go
Comment thread pkg/config/bee.go
Add TracingOTLPProtocol to BeeConfig and the orchestration Config, emit
it in the k8s config template, and default TracingOTLPInsecure to true
and TracingOTLPProtocol to http in Export() to mirror bee's OTLP
defaults so exporters can opt into grpc.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants