dodjdnh commented on PR #1591:
URL: https://github.com/apache/dubbo-admin/pull/1591#issuecomment-6380423583
> ## 结论：可以合入，但这是有意的 Breaking 变更，需要发布说明
> ### 变更实质
> `GetInstanceDashboardVariables` 与 `GetInstanceTraceDashboardVariables` 的
`var-instance` 从 `IP:QoSPort`（QoS 端口缺省时用 22222）改为**裸 IP**，以匹配 Prometheus 的 `ip`
标签。同时修正了多处指标名拼写（`dubbo_*_tota` → `dubbo_*_total`、`requests_processing` →
`requests_processing_total`），这部分是净收益。
>
> ### 已核对
> * Instance Metric 仪表盘的 `instance` 变量查询已改为
`label_values(dubbo_application_info_total{application_name="$application"},
ip)`，与新后端返回值一致。
> * 文档中提到的另一条路径（RPC 序列缺 `ip` 时，用 `dubbo_application_info_total` 按
`(application_name, instance)` 关联）在查询里以 `A or (B and
on(application_name,instance) C)` 的形式实现，语义正确：`or` 保证两条路径命中同一序列时不重复。
> * `Instance traces` 仪表盘的 `instance` 是 `custom` 类型、`query` 为空，且整块 dashboard
的查询没有引用 `$instance`，因此改成裸 IP 不会影响 Jaeger 查询。
> * 其余 12 处 `"name": "instance"` 属于导入的 Kubernetes / node-exporter 仪表盘，仍然是
`label_values(up{...}, instance)`，本次未被误改，正确。
>
> ### P3 · 混合版本部署没有运行时保护
> `docs/observability-metric-compatibility.md` 已明确写出「旧后端 + 新仪表盘」或「新后端 +
旧仪表盘」会得到空结果。但代码里没有任何版本标记或降级提示，用户只能看到空面板。考虑到这是 Grafana
侧配置，后端能做的有限；建议至少在发布说明里把它列为升级步骤（先更新 Admin，再 reload provisioned dashboards），并说明
`var-instance` 是用户可见的 URL 契约变化。
>
> ### P3 · `var-instance` 的语义变化影响自定义看板
> 任何保存了 `var-instance=10.42.1.10:22222` 的外部链接都会失效。文档已提示，建议同时在 `CHANGELOG` /
release note 中体现，而不仅放在新增的文档里。
明白
--
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]
