Skip to content

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

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

beholder: add telemetry LogLevel config option for log streaming#1591
pavel-raykov merged 3 commits intomainfrom
infoplat-2449-otelzap-logger-add-telemetry-log-level

Conversation

@pkcll
Copy link
Contributor

@pkcll pkcll commented Oct 8, 2025

beholder: add Telemetry.LogLevel config option for log streaming

Related PR smartcontractkit/chainlink#19775

@github-actions
Copy link

github-actions bot commented Oct 8, 2025

⚠️ 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
@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
20 of 21 checks passed
@pavel-raykov pavel-raykov deleted the infoplat-2449-otelzap-logger-add-telemetry-log-level branch October 9, 2025 16:22
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