[receiver/{dockerstatsreceiver, kubeletstatsreceiver}] Collision with metric container.memory.rss

[mackjmr]

Component(s)

receiver/dockerstats, receiver/kubeletstats

Describe the issue you're reporting

Both dockerstatsreceiver and kubeletstatsreceiver emit metric with same name container.memory.rss.

• In dockerstatsreceiver the metric is pulled from rss in: https://pkg.go.dev/github.com/docker/docker@v24.0.2+incompatible/api/types#MemoryStats.Stats

• In kubeletstatsreceiver the metric is pulled from:
  https://pkg.go.dev/k8s.io/kubelet@v0.27.3/pkg/apis/stats/v1alpha1#MemoryStats.RSSBytes

I believe one should be renamed to avoid the collision.

[github-actions]

Pinging code owners:

• receiver/dockerstats: @rmfitzpatrick @jamesmoessis
• receiver/kubeletstats: @dmitryax

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[github-actions]

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

• receiver/dockerstats: @rmfitzpatrick @jamesmoessis
• receiver/kubeletstats: @dmitryax @TylerHelmuth
• needs: Github issue template generation code needs this to generate the corresponding labels.

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[crobert-1]

From the spec, metrics should never have the same name, so this is a valid issue.

Links to duplicate metric names:
dockerstats
kubeletstats

[crobert-1]

My apologies, from comments on #24229 it looks like this may be a more complicated situation than I assumed.

The outcome of this bug will be determined by the decision on #26499.

[github-actions]

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

• receiver/dockerstats: @rmfitzpatrick @jamesmoessis
• receiver/kubeletstats: @dmitryax @TylerHelmuth

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[github-actions]

This issue has been inactive for 60 days. It will be closed in 60 days if there is no activity. To ping code owners by adding a component label, see Adding Labels via Comments, or if you are unsure of which component this issue relates to, please ping @open-telemetry/collector-contrib-triagers. If this issue is still relevant, please ping the code owners or leave a comment explaining why it is still relevant. Otherwise, please close it.

Pinging code owners:

• receiver/dockerstats: @rmfitzpatrick @jamesmoessis
• receiver/kubeletstats: @dmitryax @TylerHelmuth

See Adding Labels via Comments if you do not have permissions to add labels yourself.

[github-actions]

This issue has been closed as inactive because it has been stale for 120 days with no activity.