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

agent: enforce a soft limit on the number of bindings per task #1640

Open
psFried opened this issue Sep 19, 2024 · 1 comment
Open

agent: enforce a soft limit on the number of bindings per task #1640

psFried opened this issue Sep 19, 2024 · 1 comment
Labels
control-plane enhance New feature or enhancement with UX impact

Comments

@psFried
Copy link
Member

psFried commented Sep 19, 2024

Flow can only handle so many bindings for a given task. We don't precisely know what that number is, and it varies somewhat from task to task, but suffice to say that there's such a thing as too many. We'd like to prevent users from publishing tasks that have more than that number of bindings. Regardless of the specific maximum number we pick, there's existing tasks that already have more bindings than that. The logic for enforcing this maximum must permit existing tasks to maintain their current bindings, even if it's over the maximum. In that case, it should only prevent net new bindings from being added.

We also anticipate needing to adjust this limit on more of a case-by-case basis. So we should store the limit in the database (probably next to the quotas on the tenants table).

Bikeshedding: what number to pick as the default limit

@psFried psFried added enhance New feature or enhancement with UX impact control-plane labels Sep 19, 2024
@dyaffe
Copy link
Member

dyaffe commented Sep 19, 2024

It would be really nice if what we pick doesn't prevent the standard netsuite case from working...

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
control-plane enhance New feature or enhancement with UX impact
Projects
None yet
Development

No branches or pull requests

2 participants