search

open-telemetry / opentelemetry-collector-contrib

Contrib repository for the OpenTelemetry Collector
https://opentelemetry.io
Apache License 2.0
2.97k stars 2.3k forks source link

vcenter.cluster.vm.count reports datacenter VMs as amount for all member clusters #33445

Closed nicolai-hornung-bl closed 2 months ago

nicolai-hornung-bl commented 3 months ago

Component(s)

receiver/vcenter

What happened?

Description

The vcenter.cluster.vm.count that should provide the number of virtual machines in a cluster only works correctly if the managing vCenter only has a single cluster. If there are multiple clusters registered the metric will be the same for all member clusters and reports the total number of VMs in the datacenter for each of them.

Steps to Reproduce

Expected Result

Collector version

v0.87.0

Environment information

Environment

Running the embedded receiver in Grafana Alloy however as the rest of the data seems to be formatted correctly I suspect this is an issue in the upstream component.

OpenTelemetry Collector configuration

No response

Log output

No response

Additional context

No response

github-actions[bot] commented 3 months ago

Pinging code owners:

StefanKurek commented 3 months ago

@djaglowski

@nicolai-hornung-bl Are you able to try a newer version of the collector? This should have been fixed starting at version 0.97.0. Thanks!

nicolai-hornung-bl commented 3 months ago

Hi @StefanKurek in that case I will ask whether grafana/alloy can update their dependency. I couldn't find exactly what version they were pulling anyway but their docs linked to the 0.87.0 version so I took this. I will come back to you once I have more info.

StefanKurek commented 3 months ago

@nicolai-hornung-bl Do you know exactly what version of Alloy you were using? I think at one point they were using an older version of the collector but the newest version should not have this issue.

nicolai-hornung-bl commented 3 months ago

@StefanKurek, currently I'm running Alloy version 1.0.0 as 1.1 is bugged.

StefanKurek commented 3 months ago

@nicolai-hornung-bl ah. Well whenever you are able to switch to a newer version of Alloy, that should take care of it. Starting with version v1.1.0 of Alloy, this issue should no longer be there.

nicolai-hornung-bl commented 2 months ago

Hi @StefanKurek I upgraded to v1.2.0 and can confirm it's fixed. Sorry for the confusion and thanks for checking!