Skip to content
New issue

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

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

Already on GitHub? # to your account

Add push work pools max run time to docs #16293

Merged
merged 3 commits into from
Dec 11, 2024
Merged

Conversation

jakekaplan
Copy link
Contributor

No description provided.

@jakekaplan jakekaplan added the docs label Dec 9, 2024
@jakekaplan jakekaplan changed the title Note push work pools max run time Add push work pools max run time to docs Dec 9, 2024
@jakekaplan jakekaplan enabled auto-merge (squash) December 9, 2024 21:53
Comment on lines +683 to +684
Push work pool usage is unlimited. However push work pools only support flow runs with
a max run time of 24 hours.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

nit: just bc the second sentence read strangely to me, but feel free to ignore

Suggested change
Push work pool usage is unlimited. However push work pools only support flow runs with
a max run time of 24 hours.
Push work pool usage is unlimited. However push work pools limit flow runs to 24 hours.

@jakekaplan jakekaplan merged commit 33339e1 into main Dec 11, 2024
7 checks passed
@jakekaplan jakekaplan deleted the push-work-pools-max-run branch December 11, 2024 02:57
@zzstoatzz
Copy link
Collaborator

oh haha auto-merge caught me off guard

@jakekaplan
Copy link
Contributor Author

@zzstoatzz Ah I am sorry, updated here cause I like your version better.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants