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

UI: Implement timeouts, retries for jobs #169

Open
JasonWeill opened this issue Oct 19, 2022 · 1 comment · May be fixed by #371
Open

UI: Implement timeouts, retries for jobs #169

JasonWeill opened this issue Oct 19, 2022 · 1 comment · May be fixed by #371
Labels
enhancement New feature or request

Comments

@JasonWeill
Copy link
Collaborator

Problem

This should be fixed after #90 (API) is done.

Users should be able to set timeouts and retry options for jobs.

Proposed Solution

Implement these features in the create job UI, mapped to the fields listed below in the API.

  • timeout_seconds timeout in seconds for the job to complete (must be a number)
  • retry_on_timeout whether to retry if a timeout has occurred (default no)
  • max_retries no of times to attempt retry (number)
  • min_retry_interval_millis delay in milliseconds between retry attempts (number)

All of these inputs should be optional and should only be sent in the API request if they are present (nonblank/defined).

Additional context

#45 (comment)

@JasonWeill JasonWeill added the enhancement New feature or request label Oct 19, 2022
@JasonWeill JasonWeill linked a pull request May 19, 2023 that will close this issue
@aiqc
Copy link

aiqc commented Oct 23, 2023

Agreed, especially on retries. Sometimes internet dependent processes get hung or a Docker container crashes for unknown reasons.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants