Skip to content

beholder: add telemetry LogLevel config option for log streaming - #1591

Merged
pavel-raykov merged 3 commits into
mainfrom
infoplat-2449-otelzap-logger-add-telemetry-log-level
Oct 9, 2025
Merged

pavel-raykov merged 3 commits into
mainfrom
infoplat-2449-otelzap-logger-add-telemetry-log-level

Conversation

@pkcll

@pkcll pkcll commented Oct 8, 2025 •

Copy link
Copy Markdown
Contributor

beholder: add Telemetry.LogLevel config option for log streaming

Related PR smartcontractkit/chainlink#19775

@github-actions

github-actions Bot commented Oct 8, 2025 •

Copy link
Copy Markdown
Contributor

⚠️ API Diff Results - Breaking changes detected

📦 Module: github-com-smartcontractkit-chainlink-common

🔴 Breaking Changes (1)

pkg/loop (1)
  • NewOtelLogger — Type changed:
- func(go.opentelemetry.io/otel/log.Logger) (github.com/smartcontractkit/chainlink-common/pkg/logger.Logger, error)
+ func(go.opentelemetry.io/otel/log.Logger, go.uber.org/zap/zapcore.Level) (github.com/smartcontractkit/chainlink-common/pkg/logger.Logger, error)

📄 View full apidiff report | 📚 Learn about apidiff

@pkcll
pkcll force-pushed the infoplat-2449-otelzap-logger-add-telemetry-log-level branch from 3d3cad8 to b811a20 Compare October 8, 2025 03:37
@pkcll
pkcll marked this pull request as ready for review October 8, 2025 03:41
@pkcll
pkcll requested review from a team as code owners October 8, 2025 03:41
@pkcll pkcll changed the title beholder: add Telemetry.LogLevel config option for log streaming beholder: add telemetry LogLevel config option for log streaming Oct 8, 2025
Comment thread pkg/beholder/config.go Resolved
@pkcll
pkcll requested review from bolekk and patrickhuie19 and removed request for albert597 October 9, 2025 16:02
@pavel-raykov
pavel-raykov enabled auto-merge (squash) October 9, 2025 16:17
@pavel-raykov
pavel-raykov merged commit 1bb9638 into main Oct 9, 2025
@pavel-raykov
pavel-raykov deleted the infoplat-2449-otelzap-logger-add-telemetry-log-level branch October 9, 2025 16:22

This branch had an error being deployed

1 failed deployment
integration — 3e82a7f4 Deployed Oct 9, 2025 by pavel-raykov
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.

5 participants