asafm commented on issue #19554: URL: https://github.com/apache/pulsar/issues/19554#issuecomment-1589666563
I also considered adding a Tenant attribute in OpenTelemetry metrics (once voted approved and started working on it) and changing the namespace to be only the Namespace name. I'm ok with `pulsar_namespace` to avoid confusion with k8s. In OTel, all attributes have their domain prefix. I still need to figure it out - researching as there are Messaging Semantics standards that are being revised as we speak. Of course, we need to retain the namespace attribute to avoid breaking anyone. Also, we need to change the existing Grafana dashboards accordingly. Please note that there is yet to be a single place in Pulsar where you can ask: given a topic, give me its Attributes (labels). I planned to write one for OTel. Once you have that, you also need to make sure it's used everywhere. In OTel, I plan that this will be part of the metrics infrastructure so that Plugins will also use this. Otherwise, you would also get inconsistency in plugins reporting this. @codelipenghui @hangc0276, any opinion -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
