Skip to content

bump otel deps past breaking changes - #2343

Merged
jmank88 merged 3 commits into
mainfrom
bump-otel
Aug 27, 2026
Merged

bump otel deps past breaking changes#2343
jmank88 merged 3 commits into
mainfrom
bump-otel

Conversation

@jmank88

@jmank88 jmank88 commented Aug 26, 2026

Copy link
Copy Markdown
Contributor

No description provided.

justinkaseman
justinkaseman previously approved these changes Aug 26, 2026
@jmank88
jmank88 force-pushed the bump-otel branch 4 times, most recently from d8e94f7 to 13475f4 Compare August 26, 2026 17:53
@jmank88
jmank88 requested a review from a team as a code owner August 26, 2026 17:53
@jmank88
jmank88 requested review from a team as code owners August 26, 2026 18:05
vreff
vreff previously approved these changes Aug 26, 2026
JohnChangUK
JohnChangUK previously approved these changes Aug 26, 2026
@pavel-raykov
pavel-raykov dismissed stale reviews from JohnChangUK and vreff via c48c852 August 27, 2026 09:18
@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

📊 API Diff Results

No changes detected for module github.com/smartcontractkit/chainlink-common/pkg/monitoring

View full report

@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

⚠️ API Diff Results - github.com/smartcontractkit/chainlink-common

⚠️ Breaking Changes (1)

pkg/beholder (1)
  • OtelAttr — Type changed:
func(
  string, 
  any
)
- go.opentelemetry.io/otel/log.KeyValue
+ go.opentelemetry.io/otel/attribute.KeyValue

📄 View full apidiff report

@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown
Contributor

📊 API Diff Results

No changes detected for module github.com/smartcontractkit/chainlink-common/keystore

View full report

@jmank88
jmank88 added this pull request to the merge queue Aug 27, 2026
Merged via the queue into main with commit 7374c48 Aug 27, 2026
34 of 36 checks passed
@jmank88
jmank88 deleted the bump-otel branch August 27, 2026 11:46
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.

10 participants