VM metrics should contains the vm and vmi's UUID

[matthewei]

Is your feature request related to a problem? Please describe:
we can create the same name vm in different namespace. we collect vm's metrics from different kubevirt cluster. So I want to add vm and vmi's UUID tags to distinguish them.

Describe the solution you'd like:
A clear and concise description of what you want to happen.

Describe alternatives you've considered:
A clear and concise description of any alternative solutions or features you've considered.

Additional context:
Add any other context or screenshots about the feature request here.

[matthewei]

Please assign this PR to me if you think that is necessay for kubevirt project.

[xpivarc]

@machadovilaca thoughts?
I am wondering if this is best practice or the exporter per cluster should enrich the metrics

[matthewei]

Even though, I think we should also need this UID just as kubernetes' pod. What do you think?

[machadovilaca]

You could use the pair {namespace, name} to avoid conflicts of VMs names between different namespaces
But I see that, for example, kube-state-metrics also the uid in it's metrics.

It's fine for me to add it.

I would suggest waiting for #11601 to get merged. After it goes in, should be simple to add a new label from the VMI UID in

kubevirt/pkg/monitoring/metrics/virt-handler/domainstats/domainstats.go

Line 60 in cf5aabb

func (vmiReport *VirtualMachineInstanceReport) buildRuntimeLabels() {

[matthewei]

yes, I want to work on it.
/assign

[matthewei]

/assign