dodjdnh commented on PR #3551: URL: https://github.com/apache/dubbo-go/pull/3551#issuecomment-5599002500
Hi,@AsperforMias @jiaming2li，我这边正在做 Dubbo Admin 的可观测性接入，当前比较关注 Trace -> Metric 的字段映射。 我看到这个 PR 里目前 rpc.service 的取值是： semconv.RPCService(url.ServiceKey()) 同时这个 PR 已经把 group 和 version 分别作为独立的 dubbo.group、dubbo.version Span 属性输出。 从 Dubbo Admin 侧来看，我们希望 Trace 的 rpc.service 可以直接和 Prometheus Metric 的 interface Label 对应，例如： Trace: rpc.service = org.apache.dubbo.samples.OrderService Metric: interface = org.apache.dubbo.samples.OrderService 如果 rpc.service 继续使用 ServiceKey()，在带 group/version 的情况下可能会变成： gray/org.apache.dubbo.samples.OrderService:1.0.0 这样 Admin 在做 Trace -> Metric 时就无法直接、稳定地和 Metric 的 interface 匹配。 所以想确认一下，是否可以让 rpc.service 只保留纯接口全限定名，而 group、version 继续使用这个 PR 已经增加的独立属性： rpc.service = org.apache.dubbo.samples.OrderService dubbo.group = gray dubbo.version = 1.0.0 如果这个方向可以的话，也希望补充一下 Provider / Consumer 在配置 group/version 时的测试，确保最终满足： rpc.service == Metric interface 这样 Dubbo Admin 就可以把它作为比较稳定的 Trace -> Metric 映射契约。 我把更完整的 Dubbo Admin ↔ dubbo-go 第一阶段可观测性对齐背景整理在这里： https://github.com/apache/dubbo-go/issues/3562#issuecomment-5598910584 -- 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] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
