Skip to content

ci: stabilize SeaweedFS resource requests#13454

Open
jeffspahr wants to merge 1 commit into
kubeflow:masterfrom
jeffspahr:codex/stabilize-seaweedfs-ci
Open

ci: stabilize SeaweedFS resource requests#13454
jeffspahr wants to merge 1 commit into
kubeflow:masterfrom
jeffspahr:codex/stabilize-seaweedfs-ci

Conversation

@jeffspahr
Copy link
Copy Markdown
Contributor

Summary

  • add a shared CI-only SeaweedFS resource request patch
  • apply it to standalone and multi-user CI overlays that deploy SeaweedFS
  • keep Ginkgo parallelism unchanged

Context

PR #13451 exposed repeated Argo wait-container artifact/log upload failures to SeaweedFS, for example failed to put file ... seaweedfs ... i/o timeout.

Validated on PR #13451 before splitting this into a separate PR:

  • E2EParallelNested cacheEnabled=false argoVersion=v4.0.4 passed with normal 3-worker parallelism
  • E2ECritical pod_to_pod_tls_enabled=true passed
  • Critical Scenario Multi User cacheEnabled=false passed
  • full test matrix was green except Tide pending

Local verification

  • git diff --check
  • rendered standalone overlays: default, cache-disabled, cache-disabled-proxy, proxy, tls-enabled
  • rendered multi-user overlays: default, cache-disabled, artifact-proxy
  • verified rendered SeaweedFS requests are cpu: 250m and memory: 256Mi

Copilot AI review requested due to automatic review settings May 30, 2026 15:41
@google-oss-prow google-oss-prow Bot requested review from droctothorpe and nsingla May 30, 2026 15:41
Signed-off-by: Jeff Spahr <spahrj@gmail.com>
@droctothorpe droctothorpe force-pushed the codex/stabilize-seaweedfs-ci branch from 3e4ba68 to 1537ba0 Compare May 31, 2026 23:55
@droctothorpe
Copy link
Copy Markdown
Collaborator

/approve
/lgtm

@google-oss-prow
Copy link
Copy Markdown

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: droctothorpe

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

The pull request process is described 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

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants