Skip to content

apply timeout to whole push process for gitlab not just per call - #412

Merged
thelamer merged 1 commit into
monthlyfrom
gitlab-timeoutfix
Oct 14, 2026
Merged

thelamer merged 1 commit into
monthlyfrom
gitlab-timeoutfix

Conversation

@thelamer

Copy link
Copy Markdown
Member

Fix for previous logic that could take up to 80 minutes with all calls having individual 10 minute timeouts.

@thelamer
thelamer requested a review from aptalca October 14, 2026 14:29
@LinuxServer-CI LinuxServer-CI moved this from PRs to PRs Ready For Team Review in Issue & PR Tracker Oct 14, 2026
@github-project-automation github-project-automation Bot moved this from PRs Ready For Team Review to PRs Approved in Issue & PR Tracker Oct 14, 2026
@LinuxServer-CI

Copy link
Copy Markdown
Contributor

I am a bot, here are the test results for this PR for commit dafee49 :
https://ci-tests.linuxserver.io/lspipepr/jenkins-builder/efea2443-pkg-efea2443-dev-dafee4927946680821ebd38e45a91a754cd1043c-pr-412/index.html
https://ci-tests.linuxserver.io/lspipepr/jenkins-builder/efea2443-pkg-efea2443-dev-dafee4927946680821ebd38e45a91a754cd1043c-pr-412/shellcheck-result.xml

Tag Passed
amd64-efea2443-pkg-efea2443-dev-dafee4927946680821ebd38e45a91a754cd1043c-pr-412 ✅
arm64v8-efea2443-pkg-efea2443-dev-dafee4927946680821ebd38e45a91a754cd1043c-pr-412 ✅

@thelamer

Copy link
Copy Markdown
Member Author

Didn't hit timeout just 200 seconds on gitlab copy step, so gotta test it live

@thelamer
thelamer merged commit 8e37cc2 into monthly Oct 14, 2026
6 checks passed
@LinuxServer-CI LinuxServer-CI moved this from PRs Approved to Done in Issue & PR Tracker Oct 14, 2026
@thelamer thelamer mentioned this pull request Oct 14, 2026
@aptalca
aptalca deleted the gitlab-timeoutfix branch October 14, 2026 16:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants