Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Proposal: Loop Detection #8230

Open
Cali0707 opened this issue Oct 5, 2024 · 1 comment
Open

Proposal: Loop Detection #8230

Cali0707 opened this issue Oct 5, 2024 · 1 comment

Comments

@Cali0707
Copy link
Member

Cali0707 commented Oct 5, 2024

Problem
It is easy to accidentally create loops/cycles in an eventing system which can lead to events going in a cycle for a long time and consuming lots of resources. However, there are also many valid use cases for creating a cycle so we should not always disallow them.

This proposal is to:

  1. Add a feature flag to automatically reject resources that would complete a cycle
  2. Add a label which can be added to resources to allow for the cycle to be made anyways
  3. Emit an event from every resource involved in a cycle (to notify users that they have a cycle there).

A proposal doc is here: https://docs.google.com/document/d/1V_DiONSL9iV4iyRtzLFvAQfrtDEyi0ebkM5cvrY5gzY/edit?usp=sharing

Persona:
Administrators, developers

Exit Criteria
TODO

Time Estimate (optional):
5-10 days

Additional context (optional)
AWS Lambda added something similar here: https://aws.amazon.com/blogs/compute/aws-lambda-introduces-recursive-loop-detection-apis/

@Cali0707
Copy link
Member Author

Cali0707 commented Oct 5, 2024

/triage needs-wg-discussion

@Cali0707 Cali0707 added the kind/proposal Issues or PRs related to proposals. label Oct 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant