Set node platform build info beat to 3 minutes#21777
Conversation
|
I see you updated files related to
|
|
✅ No conflicts with other open PRs targeting |
|
There was a problem hiding this comment.
Pull request overview
Risk Rating: MEDIUM (changes runtime telemetry emission frequency; low code complexity but potentially impacts observability/consumers)
Reduces the node-platform build info emission cadence by decoupling it from the global telemetry heartbeat and using a fixed 3-minute interval instead, with a test to prevent regressions.
Changes:
- Introduce a dedicated
nodePlatformBeat = 3 * time.Minuteconstant for node-platform build info emissions. - Update
NewNodePlatformBuildInfoConfigto use the fixed beat rather thanTelemetry().HeartbeatInterval(). - Add a unit test ensuring the beat remains
3meven if the global telemetry heartbeat is configured differently.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| core/services/chainlink/node_platform.go | Switches build info beat from global heartbeat-derived to a fixed 3-minute interval via a new constant. |
| core/services/chainlink/node_platform_test.go | Adds regression test asserting build info beat is 3m even when telemetry heartbeat is overridden. |
Areas for scrupulous human review:
- Confirm operational expectations around time-to-first emission (service currently only emits on ticker ticks) and whether a 3-minute interval is acceptable for downstream consumers of node build info.
Reviewer recommendations (per .github/CODEOWNERS):
@smartcontractkit/foundations,@smartcontractkit/core





This PR reduces the node platform build info emission cadence from the global telemetry heartbeat to a fixed
3minterval.Previously,
NodePlatformBuildInfoServiceusedTelemetry().HeartbeatInterval(), which defaults to1s, so build info could be emitted once per second. That is more frequent than needed for static build metadata. This change makes the service use its own explicit3 * time.Minutebeat instead.It also adds a test to verify the node platform build info config stays at
3meven when the global telemetry heartbeat is set to a different value.