Skip to content

Wire Telemetry metric views config to Beholder and LOOP plugins#23085

Open
pkcll wants to merge 8 commits into
developfrom
beholder-otel-metric-filter-views
Open

Wire Telemetry metric views config to Beholder and LOOP plugins#23085
pkcll wants to merge 8 commits into
developfrom
beholder-otel-metric-filter-views

Conversation

@pkcll

@pkcll pkcll commented Jul 11, 2026

Copy link
Copy Markdown
Collaborator

Summary

Companion to chainlink-common #2238. Bases on develop (independent of chainlink #23084).

  • Add MetricViewsDisabled and MetricViewsAttributeBlacklist under [Telemetry] in core.toml (defaults: false, ["event_id"])
  • Wire through telemetry config accessors, shell.go Beholder client, and LOOP plugin env forwarding
  • Regenerate docs/CONFIG.md and update config golden fixtures (web resolver + txtar validate scripts)
  • Bump chainlink-common to v0.11.2-0.20260713174720-ff6c540906a9 (ff6c540906a9, common Bump go.dedis.ch/kyber/v3 from 3.0.11 to 3.0.12 #2238 head)

Test plan

Merge order

  1. chainlink-common #2225main
  2. chainlink-common #2238main
  3. This PR → develop

@github-actions

github-actions Bot commented Jul 11, 2026

Copy link
Copy Markdown
Contributor

✅ No conflicts with other open PRs targeting develop

@trunk-io

trunk-io Bot commented Jul 11, 2026

Copy link
Copy Markdown

Static BadgeStatic BadgeStatic BadgeStatic Badge

View Full Report ↗︎Docs

@pkcll
pkcll force-pushed the beholder-otel-limit-metric-cardinality branch from af4fd5e to a43c1b3 Compare July 13, 2026 17:46
@pkcll
pkcll force-pushed the beholder-otel-metric-filter-views branch 2 times, most recently from 6098e68 to b5f8fd9 Compare July 13, 2026 17:59
@pkcll
pkcll force-pushed the beholder-otel-limit-metric-cardinality branch from 331b2f2 to 724abfc Compare July 13, 2026 18:00
@pkcll
pkcll force-pushed the beholder-otel-metric-filter-views branch 2 times, most recently from 5f369f9 to 1a9e28e Compare July 13, 2026 18:37
@pkcll
pkcll force-pushed the beholder-otel-limit-metric-cardinality branch from 107f0e8 to 3bbc6a1 Compare July 13, 2026 18:39
@pkcll
pkcll force-pushed the beholder-otel-metric-filter-views branch from 1a9e28e to ddcec5d Compare July 13, 2026 18:39
@pkcll
pkcll changed the base branch from beholder-otel-limit-metric-cardinality to develop July 13, 2026 18:40
@pkcll
pkcll force-pushed the beholder-otel-metric-filter-views branch from ddcec5d to fae259e Compare July 13, 2026 18:44
Add MetricViewsDisabled and MetricViewsAttributeBlacklist to core.toml,
forward through shell and loop registry, and bump chainlink-common to
ff6c540906a9 for companion PR chainlink-common #2238.

Co-authored-by: Cursor <cursoragent@cursor.com>
@pkcll
pkcll force-pushed the beholder-otel-metric-filter-views branch from fae259e to 8768af3 Compare July 13, 2026 18:44
Co-authored-by: Cursor <cursoragent@cursor.com>
@pkcll
pkcll force-pushed the beholder-otel-metric-filter-views branch from 0e14f8c to bbb94f0 Compare July 13, 2026 19:14
pkcll and others added 2 commits July 13, 2026 16:00
Drop MetricViewsDisabled; empty MetricViewsAttributeDenylist disables default
views. Rename blacklist to denylist and bump chainlink-common to c2bb0f6d4.

Co-authored-by: Cursor <cursoragent@cursor.com>
@pkcll
pkcll force-pushed the beholder-otel-metric-filter-views branch from a0ec95c to de3c85c Compare July 17, 2026 00:19
pkcll added 4 commits July 16, 2026 20:34
…gin/develop

- Merge origin/develop into beholder-otel-metric-filter-views
- Resolve go.mod/go.sum conflicts keeping views branch chainlink-common
- Bump chainlink-common to v0.11.2-0.20260720200433-dee51f29efab
- Bump chainlink-common/pkg/chipingress to v0.0.11-0.20260716165322-7f2edff6e954
- Rename MetricViewsAttributeDenylist -> MetricViewsDenyAttributes to match
  the updated chainlink-common beholder.Config field name
- Regenerate docs/CONFIG.md and test fixtures
- Run make gomodtidy and make config-docs
@cl-sonarqube-production

Copy link
Copy Markdown

@pkcll
pkcll marked this pull request as ready for review July 20, 2026 23:56
@pkcll
pkcll requested review from a team as code owners July 20, 2026 23:56
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.

2 participants