-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Link to Eventing Data Plane Contract/SPEC #4943
Copy link
Copy link
Open
Labels
kind/eventingkind/functionspriority/awaiting-more-evidenceLowest priority. Possibly useful, but not yet enough support to actually get it done.Lowest priority. Possibly useful, but not yet enough support to actually get it done.triage/acceptedIssues which should be fixed (post-triage)Issues which should be fixed (post-triage)
Description
Activity
Metadata
Metadata
Assignees
Labels
kind/eventingkind/functionspriority/awaiting-more-evidenceLowest priority. Possibly useful, but not yet enough support to actually get it done.Lowest priority. Possibly useful, but not yet enough support to actually get it done.triage/acceptedIssues which should be fixed (post-triage)Issues which should be fixed (post-triage)
Describe the change you'd like to see
Speaking to many function authors (i.e. Knative Service with Eventing), especially newcomers, most of them are not aware of the underlying data plane contract, e.g. used by
brokers. This can lead to issues where retry and error behavior by a function (Kservice) is incorrectly assumed/handled.Examples
SPECsays to retry on404which is typically not what users expect (404s are usually not retriable and the SPEC bends the rules here IMHO a bit for a particular use case).Proposal: add a section to the docs describing the impact of the data plane contract to Knative function authors so at least they are aware of the behavior. The only issue is that not all implementations might strictly follow the SPEC, so this should be considered best effort/best practice for now until we are sure that all implementations strictly follow the SPEC.
Additional context
Add any other context or screenshots about the feature request here.