Repository navigation
Conversation
|
Thanks for contributing to Relax, @Echo-Nie! 感谢你为 Relax 做出贡献! Contribution guide / 贡献指南Describe the problem, your changes, and how you validated them. Keep each PR focused and run 请说明问题、改动和验证方式,保持 PR 聚焦,并在提交前运行 CI commands / CI 指令
Put one command on the first line of a new PR comment. Rerun/cancel require PR authorship or repository write access. 在新 PR 评论的首行写一条指令。PR 作者或有仓库写权限的贡献者可以重跑、取消 CI。 |
Nyanpasu 审查看板审查状态: 💬 已完成 · 有补充意见 审查版本: ba730d5 目标分支: main 审查已全部完成并应用户请求发布最终结论:F1 已修复确认(线程已标记 resolved),F2 按作者对 KEDA 官方文档的更正撤回,唯一剩余项为 F3(L411 cooldownPeriod 行描述建议,非阻塞,采用与否由作者决定)。翻译结构一致性、代码引用、链接与侧边栏注册均已核对通过,无阻塞问题。
审查发现待处理
已解决或已取代
提交范围 · 接收 5 · 建议移出 0 · 待确认 0接收 5 个文件 · 建议移出 0 个文件 · 待确认 0 个文件。移出与待确认部分暂停深审,不代表审查通过。
Powered by Nyanpasu with glm-5.3-flash max, please check the suggestions carefully.
|
rai-studio-bot
left a comment
There was a problem hiding this comment.
审查结论:两处低优先级补充意见
翻译结构与中文原版逐行对应(41 个标题、代码块与表格位置一致),文档中的事实性引用已与代码逐一核对:scale_out/scale_in/engines 端点、HTTP 409 互斥、LIFO 缩容、初始引擎下限保护、状态机枚举、relax/components/rollout.py:29 的幂等说明均属实;./elastic-rollout.md 与 ./fully-async-training.md 链接有效;config.mts 新增的 en/zh 侧边栏项均指向已存在页面;当前 head 上没有失败的 CI 检查。余下两处低优先级问题(默认值笔误与一处漏译)见上方行内评论。
Signed-off-by: Echo-Nie <nyxchaoji123@163.com>
rai-studio-bot
left a comment
There was a problem hiding this comment.
审查补充:一条低优先级修正建议
F2 讨论串中已确认 cooldownPeriod 仅在缩容到 0 时生效(KEDA 官方文档明确 “only applies when scaling to 0”,本示例 minReplicaCount: 2 下不生效),据此撤回 F2,英文版现有表述比中文原版更准确;同时把关键时间参数表中该参数行的描述修正为准确口径,见行内评论。F1 此前已确认修复,无其他遗留问题。
| | `--scale-in-drain-timeout` (Relax) | 30s (default) | Timeout for in-flight requests to finish | | ||
| | `--scale-in-shutdown-timeout` (Relax) | 30s (default) | Graceful engine shutdown timeout | | ||
| | preStop `MAX_WAIT` | 150s | Must be less than `terminationGracePeriodSeconds` | | ||
| | KEDA `cooldownPeriod` | 300s | Scale-down cooldown to avoid frequent scaling | |
There was a problem hiding this comment.
cooldownPeriod 的作用描述
接 F2 讨论串的结论:KEDA ScaledObject spec 明确 “the KEDA cooldownPeriod only applies when scaling to 0”,1 到 N 之间的扩缩由 Kubernetes HPA 负责。本示例 minReplicaCount: 2,永远不会缩到 0,该参数在这套配置里实际不生效——“Scale-down cooldown to avoid frequent scaling” 会误导读者以为它能抑制 2↔N 的缩容(实际由上一行的 stabilizationWindowSeconds / policies 控制)。
建议改为:
| | KEDA `cooldownPeriod` | 300s | Scale-down cooldown to avoid frequent scaling | | |
| | KEDA `cooldownPeriod` | 300s | Cooldown before scaling to 0; not used when `minReplicaCount` > 0 | |
同一说法还出现在 ScaledObject YAML 示例的 cooldownPeriod: 300 注释(“Scale-down cooldown in seconds”),以及中文原页同位置的表格行与“互斥保护”一句,可按同一口径一并修正。
rai-studio-bot
left a comment
There was a problem hiding this comment.
审查结论:无阻塞问题,仅剩一条可选修正
收到审查邀请,确认当前状态:F1(shutdown 超时默认值)已由 ba730d58 修复并核对无误;F2 已按你对 KEDA 官方文档的更正撤回,英文版现有表述更准确;唯一剩余项是 F3——关键时间参数表中 cooldownPeriod 行的描述建议(行内评论已附 suggestion),采用与否由你决定,不阻塞合并。我这边审查已全部完成,其余内容(翻译结构一致性、代码引用、链接、侧边栏注册)均已核对通过。

What
发现这个是中文文档...
https://redai-studio.github.io/Relax/en/guide/autoscaler-k8s-keda.html
Why
因为在en目录下
How
翻译,不过先用Astra翻了一下,明天人肉check一下,先提一个draft
二编:检查了一下,稍微改了一点点内容。然后还发现一个问题是 docs/.vitepress/config.mts 里面好像没注册?加了一下
Testing
pre-commit run --all-filespassespytest tests/)Type of Change
Screenshots / Logs