Skip to content

Conversation

@CH3CHO
Copy link
Collaborator

@CH3CHO CH3CHO commented Sep 8, 2025

Ⅰ. Describe what this PR did

Exclude built-in labels when filling Route.customLabels so they can be removed during update.

Ⅱ. Does this pull request fix one issue?

fixes alibaba/higress#2868

Ⅲ. Why don't you add test cases (unit test/integration test)?

Ⅳ. Describe how to verify it

Ⅴ. Special notes for reviews

@CH3CHO CH3CHO requested a review from lexburner as a code owner September 8, 2025 02:51
@lingma-agents
Copy link

lingma-agents bot commented Sep 8, 2025

修复 Route.customLabels 逻辑以排除内置标签

变更概述
  • 问题修复

    • 修复了在处理 Route.customLabels 时未排除内置标签的问题,这会导致在更新路由时无法正确移除自定义标签。
    • 新增了一个静态不可变集合 BUILT_IN_LABELS,用于存储所有内置标签键。
    • 实现了一个新的辅助方法 isCustomLabel,用于判断一个标签是否为自定义标签(即非内置标签)。
    • 修改了 fillCustomLabels 方法,使其在填充 Route.customLabels 时只包含自定义标签,从而确保内置标签不会被错误地当作自定义标签处理。
  • 重构

    • KubernetesModelConverter 类进行了部分重构,将内置标签的初始化逻辑集中到静态代码块中,提升了代码的可维护性和可读性。
变更文件
文件路径 变更说明
backend/​sdk/​src/​main/​java/​com/​alibaba/​higress/​sdk/​service/​kubernetes/​KubernetesModelConverter.​java 新增了对内置标签的识别和过滤逻辑,确保在设置 Route.customLabels 时仅包含用户自定义的标签。同时重构了相关初始化代码以提高可维护性。
时序图
sequenceDiagram
    participant KMC as KubernetesModelConverter
    participant Route as Route
    participant Meta as V1ObjectMeta
    Note over KMC: 初始化时加载所有内置标签到 BUILT_IN_LABELS
    KMC->>KMC: static block loads built-in labels
    Note over KMC: 填充自定义标签时过滤掉内置标签
    KMC->>KMC: fillCustomLabels filters out built-in labels
    KMC->>Route: route.setCustomLabels(only custom labels)
Loading

💡 小贴士

与 lingma-agents 交流的方式

📜 直接回复评论
直接回复本条评论,lingma-agents 将自动处理您的请求。例如:

  • 在当前代码中添加详细的注释说明。

  • 请详细介绍一下你说的 LRU 改造方案,并使用伪代码加以说明。

📜 在代码行处标记
在文件的特定位置创建评论并 @lingma-agents。例如:

  • @lingma-agents 分析这个方法的性能瓶颈并提供优化建议。

  • @lingma-agents 对这个方法生成优化代码。

📜 在讨论中提问
在任何讨论中 @lingma-agents 来获取帮助。例如:

  • @lingma-agents 请总结上述讨论并提出解决方案。

  • @lingma-agents 请根据讨论内容生成优化代码。

@CH3CHO CH3CHO force-pushed the fix/route-custom-label branch from b648c5f to 4a2f727 Compare September 8, 2025 05:23
Copy link
Contributor

@johnlanni johnlanni left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@johnlanni johnlanni merged commit a5f82b0 into higress-group:main Sep 21, 2025
4 checks passed
@CH3CHO CH3CHO deleted the fix/route-custom-label branch September 21, 2025 06:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

删除域名失败 || Delete the domain name failed

2 participants