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

State the limitations and considerations #2

Closed
trstringer opened this issue Mar 28, 2022 · 2 comments · Fixed by #24
Closed

State the limitations and considerations #2

trstringer opened this issue Mar 28, 2022 · 2 comments · Fixed by #24
Labels
documentation Improvements or additions to documentation

Comments

@trstringer
Copy link
Owner

The readme should reflect the following limitations and considerations:

  • While the workflow is paused, it will take a concurrent job spot
  • A job is only allowed to run for a maximum of 6 hours and then it'll be failed
  • The paused workflow is still compute that is running that could incur costs like any other idling VM

More information here on GitHub docs.
@trstringer trstringer added the documentation Improvements or additions to documentation label Mar 28, 2022
@unfor19
Copy link

unfor19 commented May 7, 2022

@trstringer thanks for that, the third bullet point is a deal-breaker for me.
I thought that this action solves exactly this issue - "wait without using compute" :\

@trstringer
Copy link
Owner Author

Totally understand. That is the primary drawback, unfortunately. Thanks for the feedback!

trstringer added a commit that referenced this issue Jun 30, 2022
@trstringer
Add limitations to the README
8bee6dd 
Add the specific limitations and considerations to the project
documentation. Closes #2.
@trstringer trstringer mentioned this issue Jun 30, 2022
Merged
trstringer added a commit that referenced this issue Jun 30, 2022
@trstringer
Add limitations to the README (#24)
fee68ac 
Add the specific limitations and considerations to the project
documentation. Closes #2.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
documentation Improvements or additions to documentation
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Add limitations to the README trstringer/manual-approval
2 participants
@trstringer @unfor19