Skip to content

beholder: enable compression for OTel metrics, traces logs (INFOPLAT-3163)#1536

Merged
pavel-raykov merged 7 commits intomainfrom
beholder-enable-metric-compression
Nov 18, 2025
Merged

beholder: enable compression for OTel metrics, traces logs (INFOPLAT-3163)#1536
pavel-raykov merged 7 commits intomainfrom
beholder-enable-metric-compression

Conversation

@pkcll
Copy link
Copy Markdown
Contributor

@pkcll pkcll commented Sep 10, 2025

What?

Add flag to enable compression for metrics, traces, logs OTLP/gRPC telemetry data exported via beholder client

Companion PR smartcontractkit/chainlink#20283

Why?

Jira https://smartcontract-it.atlassian.net/browse/INFOPLAT-3163

Slack thread https://chainlink-core.slack.com/archives/C07GQNPVBB5/p1762514534836419

Mitigate error:

failed to upload metrics: rpc error: code = ResourceExhausted desc = grpc: received message larger than max (5289803 vs. 4194304)

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Sep 10, 2025

✅ API Diff Results - No breaking changes


📄 View full apidiff report

@pkcll pkcll marked this pull request as ready for review September 12, 2025 14:57
@pkcll pkcll requested review from a team as code owners September 12, 2025 14:57
pavel-raykov
pavel-raykov previously approved these changes Sep 12, 2025
kirqz23
kirqz23 previously approved these changes Sep 12, 2025
Copy link
Copy Markdown
Contributor

@kirqz23 kirqz23 left a comment

Choose a reason for hiding this comment

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

LGTM

engnke
engnke previously approved these changes Sep 12, 2025
@pkcll pkcll requested review from patrickhuie19 and removed request for mchain0 September 12, 2025 21:50
@github-actions
Copy link
Copy Markdown

This PR is stale because it has been open 30 days with no activity.
Remove the stale label or comment or this will be closed in 7 days.

@github-actions github-actions Bot added the Stale label Oct 13, 2025
@github-actions
Copy link
Copy Markdown

This PR has been automatically closed because it has been stale for > 30 days.
If you wish to continue working on this PR, please reopen it and make any necessary changes.

@github-actions github-actions Bot closed this Oct 20, 2025
@github-actions github-actions Bot deleted the beholder-enable-metric-compression branch October 20, 2025 00:49
@hendoxc hendoxc restored the beholder-enable-metric-compression branch November 7, 2025 16:42
@hendoxc hendoxc reopened this Nov 7, 2025
@pkcll pkcll dismissed stale reviews from pavel-raykov, engnke, and kirqz23 via ac14edb November 8, 2025 00:19
@pkcll pkcll force-pushed the beholder-enable-metric-compression branch from 9286dc6 to ac14edb Compare November 8, 2025 00:19
@pkcll pkcll force-pushed the beholder-enable-metric-compression branch from e0fc2bc to 1883d07 Compare November 14, 2025 18:57
Comment thread pkg/beholder/client.go Outdated
@pavel-raykov pavel-raykov added this pull request to the merge queue Nov 18, 2025
Merged via the queue into main with commit 3b1ede2 Nov 18, 2025
35 of 37 checks passed
@pavel-raykov pavel-raykov deleted the beholder-enable-metric-compression branch November 18, 2025 09:26
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.

6 participants