Skip to content

[ci-maintainer] fix(docker): per-attempt timeouts on bobshell download in src/Dockerfile + src/Dockerfile.contributor - #4942

Merged
clubanderson merged 1 commit into
v4from
ci/bobshell-curl-timeouts
Aug 28, 2026
Merged

[ci-maintainer] fix(docker): per-attempt timeouts on bobshell download in src/Dockerfile + src/Dockerfile.contributor#4942
clubanderson merged 1 commit into
v4from
ci/bobshell-curl-timeouts

Conversation

@kubestellar-hive

Copy link
Copy Markdown
Contributor

CI Fix

Files claimed: src/Dockerfile (Layer 9, bobshell curl) and src/Dockerfile.contributor (bobshell curl) — no overlap with open PRs #4939/#4931/#4923/#4905 (test files) or #4032 (proxy/auth Go code).

Three v2 CI runs in ~36h failed in the docker / overlayfs-exec-guard jobs with curl exit 28 downloading bobshell-1.0.6.tgz from IBM COS (runs 32990236986, 33072306789, 33132187354). The curl had --retry-max-time 300 but no per-attempt bound, so one stalled connection burned the whole retry window and curl exited 28 without a healthy retry.

Changes:

  • src/Dockerfile: add --connect-timeout 15 --speed-limit 1024 --speed-time 30 --max-time 180, raise --retry-max-time to 600; comment documents why.
  • src/Dockerfile.contributor: same bobshell download had no retry/timeout flags at all; hardened to the same pattern.

Fixes #4941


Filed by ci-maintainer agent (ACMM L4/L5 — hold-gated mode). Hold-gated: human review required.

— hive: agent=ci-maintainer backend=copilot model=claude-fable-5

…led IBM COS connections retry instead of exhausting the retry budget

Three v2 CI runs in ~36h (32990236986, 33072306789, 33132187354) failed
in the docker / overlayfs-exec-guard jobs with curl exit 28 while
fetching bobshell-1.0.6.tgz from IBM Cloud Object Storage. The curl
invocation had --retry-max-time 300 but no per-attempt bound, so one
stalled connection consumed the entire retry window and no healthy
retry ever happened.

- src/Dockerfile (Layer 9): add --connect-timeout 15 --speed-limit 1024
  --speed-time 30 --max-time 180 and raise --retry-max-time to 600 so
  bounded attempts still get a full retry ladder.
- src/Dockerfile.contributor: the same bobshell download had no retry
  or timeout flags at all; bring it up to the same hardened pattern.

Refs #4941

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Signed-off-by: sec-check <sec-check@hive.kubestellar.io>
@kubestellar-prow kubestellar-prow Bot added the dco-signoff: yes Indicates the PR's author has signed the DCO. label Aug 28, 2026
@kubestellar-prow

Copy link
Copy Markdown
Contributor

[APPROVALNOTIFIER] This PR is NOT APPROVED

This pull-request has been approved by:
Once this PR has been reviewed and has the lgtm label, please assign clubanderson for approval. For more information see the Code Review Process.

The full list of commands accepted by this bot can be found here.

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@github-actions

Copy link
Copy Markdown
Contributor

Changelog: this PR changes code but does not touch CHANGELOG.md.

If it is user-visible — a feature, a fix an operator would notice, a
security change, a migration, a deprecation, or anything breaking —
please add a line under ## Unreleased. If it is a refactor, a
test-only change, or dependency churn, no entry is needed and you can
ignore this.

This is a reminder, not a gate; it never blocks a merge.

@kubestellar-prow kubestellar-prow Bot added the size/S Denotes a PR that changes 10-29 lines, ignoring generated files. label Aug 28, 2026
@clubanderson
clubanderson merged commit f9ab6ad into v4 Aug 28, 2026
46 of 47 checks passed
@clubanderson
clubanderson deleted the ci/bobshell-curl-timeouts branch August 28, 2026 13:01
@github-actions

Copy link
Copy Markdown
Contributor

Thank you for your contribution! Your PR has been merged.

Check out what's new:

Stay connected: Slack #kubestellar-dev | Multi-Cluster Survey

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

copilot dco-signoff: yes Indicates the PR's author has signed the DCO. hold size/S Denotes a PR that changes 10-29 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[ci-maintainer] bobshell tarball download from IBM COS flakes with curl exit 28, failing v2 CI docker/overlayfs-exec-guard jobs

1 participant