dodjdnh commented on PR #1591: URL: https://github.com/apache/dubbo-admin/pull/1591#issuecomment-6235524358
好的 > 整体 LGTM，感谢贡献。CI 已全部通过，几点小意见供参考： > > 1. **Instance dashboard 过滤不完整**：`grafana-dashboardDefinitions.yaml` 中 Instance metrics 面板里的 `dubbo_provider_requests_failed_network_total` 和 `dubbo_provider_requests_failed_codec_total`（约 5932、5945 行，带 `or vector(0)`）只按 `application_name` 过滤，没有应用 ip 选择模式，因此展示的是整个应用的数据而不是所选实例。这是历史遗留问题，但 PR 描述称已处理所有 Instance 查询，建议一并修复或在说明中注明。 > 2. **同一 IP 多个实例**：元数据路径按 `ip` 关联抓取目标 `instance`，同一主机上端口不同的多个 Java 实例会同时匹配，选中一个 IP 时可能混入其他实例的序列。可以接受，建议在文档中补充一句说明。 > 3. **文档**：`docs/observability-metric-compatibility.md` 中出现的 “D009” 是内部跟踪编号，上游读者无法理解，建议删除。 > 4. **验证方式**：PromQL 仅做了结构性检查，没有自动化测试，主要依赖手动运行验证，请确保 Java 与 Go 两条路径都有实际截图或结果记录。 > 5. **发布顺序**：`var-instance` 参数约定已从 `IP:port` 变为纯 IP，后端与 Grafana dashboard 需要一起部署，建议在 release notes 中说明。 好的 -- 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]
