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

runner: Convert idle-timeout="never" to 0 with GitLab driver #1408

Merged
merged 3 commits into from Jul 28, 2023

Conversation

BradyJ27
Copy link
Contributor

GitLab runners do not support "never" as an argument for --wait-timeout, instead this should be 0, which by default is no timeout.

Fixes #1407

GitLab runners do not support "never" as an argument for --wait-timeout, instead this should be 0, which by default is no timeout.

Fixes iterative#1407
@BradyJ27 BradyJ27 temporarily deployed to external July 28, 2023 00:02 — with GitHub Actions Inactive
@github-actions
Copy link
Contributor

Test Comment

@github-actions
Copy link
Contributor

Test Comment

@dacbd
Copy link
Contributor

dacbd commented Jul 28, 2023

@BradyJ27 can you resolve the const issue, and I'll merge this for you.

@BradyJ27 BradyJ27 temporarily deployed to external July 28, 2023 01:31 — with GitHub Actions Inactive
@github-actions
Copy link
Contributor

Test Comment

@github-actions
Copy link
Contributor

Test Comment

dacbd
dacbd previously approved these changes Jul 28, 2023
src/drivers/gitlab.js Outdated Show resolved Hide resolved
@github-actions
Copy link
Contributor

Test Comment

@dacbd dacbd enabled auto-merge (squash) July 28, 2023 01:59
@github-actions
Copy link
Contributor

Test Comment

@dacbd dacbd merged commit b64044f into iterative:master Jul 28, 2023
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

--idle-time="never" causes an unhandled error with GitLab driver
2 participants