[pve-devel] [PATCH pve-manager] metrics add OpenTelemetry support

[Dominik Csapak]

On 7/15/25 08:52, Thomas Lamprecht wrote:
> Am 15.07.25 um 05:55 schrieb nansen.su:
[snip]>> +
>> +sub type {
>> +    return 'opentelemetry';
>> +}
>> +
>> +sub properties {
>> +    return {
>> +        'otel-protocol' => {
> 
> I'm fine with the plugin specific otel prefix, but @Dominik: these here might be
> a good fit for the property separation? It's not many plugins and each of them has
> not that many properties. Or do you know anything that would speak against this?
> 
> Anyhow, nothing that needs to block this for real and nothing you @Nansen Su need
> to worry about.
> 
Yes, the property isolation should be working fine here, at least no blocker
comes to mind. If we do that afterwards though, the property-names are still
prefixed, so we'd have to reintroduce them without prefix + deprecate, etc.
(or we just leave them prefixed)