Skip to content

tke-extend-network-controller: bump to 2.4.3#233

Merged
pavlelee merged 1 commit into
tkestack:mainfrom
imroc:tke-extend-network-controller-2.4.3
Jul 9, 2026
Merged

tke-extend-network-controller: bump to 2.4.3#233
pavlelee merged 1 commit into
tkestack:mainfrom
imroc:tke-extend-network-controller-2.4.3

Conversation

@imroc

@imroc imroc commented Jul 9, 2026

Copy link
Copy Markdown
Member

变更内容

将 tke-extend-network-controller chart 版本从 2.4.2 升级到 2.4.3。

仅更新 Chart.yaml 中的 versionappVersion,templates 和 values.yaml 无变化(与组件仓库 v2.4.3 tag 对齐,chart diff 仅版本号)。

v2.4.3 Release Notes

  • 修复:IP 复用导致 OtherTargetBound 死锁。VPC-CNI 非固定 IP 场景下,Pod 创建后 IP 变更,旧 IP 被其它 Pod 复用时,controller 想把 CLB 后端从旧 IP 更新为新 IP,但发现旧 IP 已属于另一个 Pod,OtherTargetBound 保护逻辑直接放弃纠正,导致端口永久指向错误 Pod,流量不可达。修复方式:收敛冲突判定,只有当持有旧 IP 的对象通过自己的 CLBBinding 合法占用了同一监听器时才保护(真冲突),否则视为历史残留,允许安全清理并替换为当前 Pod 真实 IP。CLBPodBinding 和 CLBNodeBinding 均适用。
  • 新增:端到端测试框架(make e2e),基于 Ginkgo 在真实 TKE 集群中验证 CLB 端口映射全流程,覆盖 15 个测试用例,包括多协议映射、端口段映射、LB 分配策略、黑名单、自动扩容等场景。

@pavlelee pavlelee left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

本地评审没有发现代码层阻塞,但当前还不能 LGTM/合并:

  • PR 只修改 incubator/tke-extend-network-controller/Chart.yaml,将 version/appVersion2.4.2 升到 2.4.3
  • 本地验证通过:helm lint incubator/tke-extend-network-controllerhelm lint incubator/tke-extend-network-controller --strict 均通过;带必填 dummy values 的 helm template 能正常渲染 2.4.3 镜像与标签。
  • 已与上游 tkestack/tke-extend-network-controllerv2.4.3 tag 对比,templates 目录一致;Chart metadata 和 values 中镜像仓库差异为应用市场既有差异。
  • 但 GitHub Actions Lint and Test Charts 最新 run 原始结果是 cancelled/failure。我已触发 rerun,目前 attempt 2 的 lint-test 仍是 queued,尚未拿到 success 结论。

等 CI 跑到 success 后可以再确认并合并。

@pavlelee

pavlelee commented Jul 9, 2026

Copy link
Copy Markdown
Collaborator

lgtm

@pavlelee pavlelee merged commit 347a660 into tkestack:main Jul 9, 2026
1 of 2 checks passed
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.

2 participants