Skip to content

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

Merged
martinconic merged 2 commits into
masterfrom
feat/bee-otlp-tracing-config
May 18, 2026
Merged

feat(config): support new bee OTLP tracing flags#585
martinconic merged 2 commits into
masterfrom
feat/bee-otlp-tracing-config

Conversation

@martinconic
Copy link
Copy Markdown
Contributor

@martinconic martinconic commented May 11, 2026

Paired change for the bee OpenTelemetry migration PR#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.

@martinconic martinconic merged commit 34f33e4 into master May 18, 2026
5 checks passed
@martinconic martinconic deleted the feat/bee-otlp-tracing-config branch May 18, 2026 12:25
@martinconic martinconic restored the feat/bee-otlp-tracing-config branch May 27, 2026 15:24
martinconic added a commit that referenced this pull request May 27, 2026
This reverts commit 34f33e4.

PR #585 introduced `tracing-otlp-endpoint`, `tracing-otlp-insecure`,
and `tracing-sampling-ratio` config keys in the bee config template,
replacing `tracing-endpoint`. Bee master does not yet recognise these
keys (the OpenTelemetry migration is on the
`feat/opentelemetry-migration` branch and hasn't merged), so bee fails
startup with "unknown parameters" and bee PR CI consistently times
out at "bootnode-0 is not ready yet".

Revert until the bee OpenTelemetry migration lands; the change will
re-land via #595.
gacevicljubisa pushed a commit that referenced this pull request May 27, 2026
This reverts commit 34f33e4.

PR #585 introduced `tracing-otlp-endpoint`, `tracing-otlp-insecure`,
and `tracing-sampling-ratio` config keys in the bee config template,
replacing `tracing-endpoint`. Bee master does not yet recognise these
keys (the OpenTelemetry migration is on the
`feat/opentelemetry-migration` branch and hasn't merged), so bee fails
startup with "unknown parameters" and bee PR CI consistently times
out at "bootnode-0 is not ready yet".

Revert until the bee OpenTelemetry migration lands; the change will
re-land via #595.
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