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.

Sign up for GitHub

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

Jump to bottom

max time in pending queue #11977

Open
tooptoop4 opened this issue Oct 10, 2023 · 4 comments
Open

max time in pending queue #11977

tooptoop4 opened this issue Oct 10, 2023 · 4 comments
Labels
area/controller Controller issues, panics area/spec Changes to the workflow specification. type/feature Feature request

Comments

@tooptoop4
Copy link
Contributor

would be good to have a setting like maxSecondsInPendingQueue: 300
so that workflows that are pending for more than this time are auto deleted

this can prevent a denial of service attack where someone submits too many and overwhelms etcd
@tooptoop4 tooptoop4 added the type/feature Feature request label Oct 10, 2023
@agilgur5 agilgur5 added area/controller Controller issues, panics area/spec Changes to the workflow specification. labels Oct 10, 2023
@agilgur5
Copy link

activeDeadlineSeconds may partly suffice for this, but you can't specify "pending" Workflows specifically with it

@tooptoop4
Copy link
Contributor Author

its not suitable as that time includes actual runtime

@shuangkun
Copy link
Member

Maybe can be combined with kueue in some way.

@tooptoop4
Copy link
Contributor Author

#12762 is similar but at node instead of workflow level

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/controller Controller issues, panics area/spec Changes to the workflow specification. type/feature Feature request
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
3 participants
@agilgur5 @tooptoop4 @shuangkun