Kafka sidecar: add a test for the record → CloudEvent → ack delivery path
维护者通常 1 天内回复
还没有人认领这个 Issue。
评估
调研方向
Start with pkg/keda and TestInt_KafkaScaling to see how the existing integration test sets up Kafka and verifies partition assignment. Then inspect the injected Kafka runtime sidecar’s delivery path and determine whether the test belongs in the slim Kafka job or knative-extensions/func-adapter-kafka. Done means testing CloudEvent delivery and offset commits after a 2xx, plus retry without commit after a non-2xx.
由索引模型根据 Issue 内容生成。
描述
What
There is no test asserting the actual record → CloudEvent → ack delivery path
through the injected Kafka runtime sidecar.
Gap
The KEDA integration test (pkg/keda, TestInt_KafkaScaling) only verifies
readiness = partition assignment; it produces no records. So the real path —
consume a record → build a CloudEvent → POST to the function's HTTP server →
2xx → commit offset — is unverified from the func side. The negative case
(a non-2xx response must not commit the offset) is also untested.
Proposal
Add a test that produces a record and asserts:
- the function receives the expected CloudEvent,
- the offset is committed after a 2xx, and
- the offset is not committed on a non-2xx (the record is retried).
This could live as an integration test on the slim Kafka job, or as a focused
test in knative-extensions/func-adapter-kafka.
/kind cleanup
Follow-up to #4060.
- 主要语言
- Go
- 星标
- 366
- 派生
- 226
- 平均合并
- 2 天 17 小时
- 30 天内合并 PR
- 21
环境准备
- 没有 Dockerfile 或 Docker Compose 文件
- 没有 Pull Request 模板
- 阅读贡献指南
从这里开始
- 先读完整个 Issue，再读项目的贡献指南。
- 在 Issue 下留言说明你要接手 —— 这能避免两个人做同样的事。
- Fork 仓库，在一个分支上完成修改。
- 提交 Pull Request，并在描述里引用这个 Issue 编号。
knative/func 的其他 Issue
-
Kafka sidecar: add a startupProbe so slow-starting functions aren't restart-looped可能已有人在做 关联的 PR 仍在进行中或已合并。 未关闭kind/bug
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复
-
KEDA deployer: default minReplicaCount prevents scale-to-zero可能已有人在做 @Tushar-Khandelwal-2004 于 113 天前认领。 未关闭lifecycle/stale
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复
-
git/gitlab: default project webhooks to SSL verification when creating hooks可能已有人在做 @Vi-shub 于 160 天前认领。 未关闭
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
mcp: config add/remove handlers bypass readonly guard可能已有人在做 @Elvand-Lie 于 162 天前认领。 未关闭kind/bug
难度 2/5 1-3 小时 新手友好度 72/100
维护者通常 1 天内回复
-
kind/bug
难度 3/5 1-2 天 新手友好度 72/100
维护者通常 1 天内回复
相似的 Issue
-
难度 2/5 1-3 小时 新手友好度 75/100
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 76/100
维护者通常 1 天内回复
-
area/frameworks area/steve kind/bug team/frameworks
难度 2/5 1-3 小时 新手友好度 78/100
维护者通常 1 天内回复
-
Exclude merge queue branches from push trigger in ci-workflow.yaml to eliminate duplicate CI runs未关闭bug infra ready-for-triage triaged
难度 2/5 1-3 小时 新手友好度 72/100
redhat-et/ProtoBot#312 · 2 条评论 · 1 个 reaction ·
维护者通常 1 天内回复
-
难度 2/5 1-3 小时 新手友好度 82/100
维护者通常 1 天内回复