fix(kubernetes): missing `kubernetes_kind` or `attribute_path` errors are dropped
维护者通常 1 天内回复
评估
调研方向
从 src/tirith/providers/kubernetes/handler.py 开始。其中 get_value 直接返回 "kubernetes_kind must be provided" 和 "attribute_path must be provided" 的结果,而不是将它们追加到 outputs 中,并且 provide() 丢弃了返回值。使用 issue 中展示的调用进行复现,然后在 tests/providers/kubernetes/ 下添加一个测试,期望每个缺失的参数都对应一个带有匹配 err 的结果。当两种情况都返回该错误且 pytest tests/ 通过时,即完成。
由索引模型根据 Issue 内容生成。
描述
Problem
In src/tirith/providers/kubernetes/handler.py, get_value returns the "kubernetes_kind must be provided" and "attribute_path must be provided" results instead of appending them to outputs. provide() ignores the return value, so it returns [] and the policy author never sees the error.
To reproduce:
from tirith.providers.kubernetes.handler import provide
provide({"operation_type": "attribute", "attribute_path": "spec"}, [{"kind": "Pod", "spec": {}}])
# -> []
What to change
Append both error results to outputs (as the "kind is not found" branch already does) and return.
Done when
A new test in tests/providers/kubernetes/ calls provide() without each argument and gets one result carrying the matching err.
Comment to claim this issue and wait to be assigned. Unclaimed or inactive for 3 days, it returns to the pool. Run pytest tests/ and black --line-length 120 before opening the PR.
- 主要语言
- Python
- 星标
- 170
- 派生
- 46
- 平均合并
- 1 天 20 小时
- 30 天内合并 PR
- 9
环境准备
在浏览器里用你自己的 GitHub 账号启动这个项目的开发容器。
- 没有 Dockerfile 或 Docker Compose 文件
- 有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue,再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库,在一个分支上完成修改。
- 提交 Pull Request,并在描述里引用这个 Issue 编号。
StackGuardian/tirith 的其他 Issue
-
chore: remove stale TODOs and commented-out code可能已有人在做 @krishna2707 于 2 天前认领。 未关闭chore good first issue hacktoberfest
难度 1/5 1 小时以内 新手友好度 82/100
StackGuardian/tirith#408 · 1 条评论 ·
维护者通常 1 天内回复
-
test(evaluators): edge cases for `Equals` and `NotEquals`可能已有人在做 @Ankit-cs 于 1 天前认领。 未关闭good first issue hacktoberfest tests
难度 2/5 1-3 小时 新手友好度 85/100
StackGuardian/tirith#406 ·
维护者通常 1 天内回复
-
test(kubernetes): cover the provider's main paths可能已有人在做 @afsuyadi 于 1 天前认领。 未关闭good first issue hacktoberfest tests
难度 2/5 1-3 小时 新手友好度 72/100
StackGuardian/tirith#405 · 3 条评论 · 已指派 1 人 ·
维护者通常 1 天内回复
-
good first issue hacktoberfest tests
难度 1/5 1 小时以内 新手友好度 92/100
StackGuardian/tirith#403 ·
维护者通常 1 天内回复
-
research
难度 1/5 1-3 小时 新手友好度 78/100
StackGuardian/tirith#356 ·
维护者通常 1 天内回复
查看 StackGuardian/tirith 的全部 Issue
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 76/100
omegaup/omegaup#10353 · 1 条评论 ·
维护者通常 5 天内回复
-
bot-triaged module: LrScheduler module: optimizer triaged
难度 2/5 1-3 小时 新手友好度 70/100
维护者通常 1 天内回复
-
documentation glm5 llm-model
难度 2/5 1-3 小时 新手友好度 78/100
vllm-project/vllm-ascend#19062 ·
维护者通常 1 天内回复
-
author-page correction
难度 2/5 1-3 小时 新手友好度 68/100
acl-org/acl-anthology#10496 · 1 条评论 ·
维护者通常 1 天内回复