redis / redis/node-redis

OpenTelemetry: disabled and inaccurate metrics

Open
#3,229 1 comment 0 reactions 1 assignee View on GitHub

@PavelPashov is already working on this.

Since Apr 14, 2026.

Bug
Dominant language
TypeScript
Stars
17.6k
Forks
2k
Avg merge
2d 3h
Merged PRs (30d)
40

Description

Description

Some OpenTelemetry metrics are still present in code even though they should not be exposed in their current form.

redis.client.csc.network_saved

Issue
This metric is still defined, but a correct implementation should measure payload avoided at the socket layer, which may require refactoring the client-side cache.

Possible fix
Refactor the client-side cache to track payload size so the metric can be implemented correctly.

db.client.connection.count

Issue
This metric is misleading because db.client.connection.state is always reported as used.

Possible fix
Rename the metric and convert it to an ObservableGauge.

db.client.connection.pending_requests

Issue
This metric is still defined, but should stay disabled until we settle on the right name and behavior.

Possible fix
Rename the metric, but keep it as an ObservableGauge, since changing it to an UpDownCounter would likely add unnecessary overhead.

Node.js Version

No response

Redis Server Version

No response

Node Redis Version

No response

Platform

No response

Logs

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.