已合并
fix(config):测试issue修复 #540
王明琦创建于 29 天前
fix(config):测试issue修复 #540
已合并
Pull Request已成功合入, 合并人@张雅晴
(感谢 王明琦 的贡献)atomgit-bot
29 天前 评论:
29 天前 评论:
变更摘要
本 PR 主要解决两个问题:一是 AgentRuntimeConfig.default_namespace 的解析链缺少 in-cluster ServiceAccount 文件兜底,导致镜像新代码(删除了 AGENT_RUNTIME_DEFAULT_NAMESPACE)搭配旧部署模板(无 POD_NAMESPACE 注入)时落入字面 "default",SA 跨 namespace 建 Pod 返回 403;二是 follower 接管 leader 热备 Pod 时未摘出 idle 暖池,导致忙 Pod 永占 warm 底数、min_idle 热备永不重建。PR 通过新增 config.own_namespace() 兜底逻辑(POD_NAMESPACE env > SA 文件 > "default",存量部署零改动自愈)和新增幂等 Lua 脚本 LUA_POP_IDLE(忙占用摘出 idle 池并清 idle_since 计时)解耦镜像/模板升级顺序,并补齐了对应回归测试。
主要改动
own_namespace()SA 文件兜底: 在agent_runtime/config.py新增own_namespace(),将default_namespace解析链改为POD_NAMESPACEenv(显式覆盖)>/var/run/secrets/kubernetes.io/serviceaccount/namespace文件(Pod 必挂、零部署配置)>"default",解决镜像/模板升级顺序不一致导致的 SA 越 namespace 建 Pod 403。- 新增
LUA_POP_IDLE脚本与pop_idle接口: 在resource_manager/lua_scripts.py增加第 7 个 Lua 脚本LUA_POP_IDLE(SREM摘出 idle 池 +DEL清idle_since,幂等、非 idle 成员为 no-op),并在resource_manager/state.py暴露ResourceState.pop_idle()方法。 - follower 接管时对齐忙记账: 在
resource_manager/orchestrator.py的 acquire follower 复用 leader Pod 分支中调用pop_idle,使idle_flag=True注册的热备 Pod 被接管即转忙,避免 autoscale 的 warm 底数把忙 Pod 数成热备导致min_idle永不重建。 - 回归测试覆盖:
tests/test_config.py重构为覆盖 env 注入、SA 文件兜底、env 覆盖 SA 文件、空串下探、非 K8s 字面"default"五条路径;tests/resource_manager/test_rm_business.py新增test_follower_reuse_pops_idle_warm_pod回归用例,验证接管即摘 idle、清计时且 autoscale 能补回新热备。 - 验证与模板注释同步:
scripts/verify_redis_cluster.py补充pop_idle的验收检查;agent_runtime.template.yaml更新注释说明 SA 文件兜底逻辑。


不准确?
atomgit-bot
29 天前 评论:
29 天前 评论:
openJiuwen-bot
29 天前 评论:
29 天前 评论:
欢迎来到 openJiuwen 社区
Hey @wangmingqi , 感谢你对社区的贡献.
机器人使用手册
有关指令的使用,可以点击 此处 查看详情。开发人员可以在每个PR或Issue下方评论特定指令来触发机器人任务。


atomgit-bot
29 天前 评论:
29 天前 评论:
29 天前 添加了label:stat/needs-squash
29 天前 添加了label:openJiuwen-cla/yes
29 天前 修改了pull request 的描述
openJiuwen-bot
29 天前 评论:
29 天前 评论:
| 任务名称 | 结果 | 日志操作 |
|---|---|---|
| 静态检查 | ❌FAILED | 点此跳转 |
| 禁用词扫描 | ✅SUCCESS | 点此跳转 |
| 防投毒检查 | ✅SUCCESS | 点此跳转 |
| 开源合规检查 | ✅SUCCESS | 点此跳转 |
| UT测试 | N/A | N/A |
| ST测试 | N/A | N/A |
| build 编译包 | N/A | N/A |
| ruff codecheck | {{ruff_codecheck_result}} | {{ruff_codecheck_link_text}} |
| issue关联检查 | {{issue_linked_result}} | {{issue_linked_link_text}} |
| pr size检查 | {{pr_size_result}} | {{pr_size_link_text}} |


29 天前 添加了label:ci-failed
28 天前 推送了提交
28 天前 删除了label:openJiuwen-cla/yes
28 天前 添加了label:openJiuwen-cla/yes
openJiuwen-bot
28 天前 评论:
28 天前 评论:
28 天前 删除了label:ci-failed
openJiuwen-bot
28 天前 评论:
28 天前 评论:
| 任务名称 | 结果 | 日志操作 |
|---|---|---|
| 静态检查 | ❌FAILED | 点此跳转 |
| 禁用词扫描 | ✅SUCCESS | 点此跳转 |
| 防投毒检查 | ✅SUCCESS | 点此跳转 |
| 开源合规检查 | ✅SUCCESS | 点此跳转 |
| UT测试 | N/A | N/A |
| ST测试 | N/A | N/A |
| build 编译包 | N/A | N/A |
| ruff codecheck | {{ruff_codecheck_result}} | {{ruff_codecheck_link_text}} |
| issue关联检查 | {{issue_linked_result}} | {{issue_linked_link_text}} |
| pr size检查 | {{pr_size_result}} | {{pr_size_link_text}} |


28 天前 添加了label:ci-failed
28 天前 修改标题为 “fix(config):测试issue修复”,原标题为“fix(config): default_namespace 补 SA 文件兜底——镜像/模板升级顺序解耦”
王明琦
28 天前 评论:
28 天前 评论:
ci-pipeline


28 天前 删除了label:ci-failed
openJiuwen-bot
28 天前 评论:
28 天前 评论:
| 任务名称 | 结果 | 日志操作 |
|---|---|---|
| 静态检查 | ✅SUCCESS | 点此跳转 |
| 禁用词扫描 | ✅SUCCESS | 点此跳转 |
| 防投毒检查 | ✅SUCCESS | 点此跳转 |
| 开源合规检查 | ✅SUCCESS | 点此跳转 |
| UT测试 | N/A | N/A |
| ST测试 | N/A | N/A |
| build 编译包 | N/A | N/A |
| ruff codecheck | {{ruff_codecheck_result}} | {{ruff_codecheck_link_text}} |
| issue关联检查 | {{issue_linked_result}} | {{issue_linked_link_text}} |
| pr size检查 | {{pr_size_result}} | {{pr_size_link_text}} |


28 天前 添加了label:ci-successful
28 天前 关联了issue:[Bug]【接口测试】【0.0.27s】修改Agent模板模型引用后,有效旧session绑定查询返回404
28 天前 合入了pull request
28 天前 修改了pull request 的描述
28 天前 删除了label:ci-successful
Paired: GitHub #199 ↔ GitCode !540
What type of PR is this?
/kind
Self-checklist:(请自检,在[ ]内打上x,我们将检视你的完成情况,否则会导致pr无法合入)
Linked Closing Issues: