Skip to content

fix: size k8s worker pvcs from artifact metadata#89

Merged
MarcStdt merged 1 commit into
masterfrom
fix/k8s-worker-pvc-min-disk
Jun 12, 2026
Merged

fix: size k8s worker pvcs from artifact metadata#89
MarcStdt merged 1 commit into
masterfrom
fix/k8s-worker-pvc-min-disk

Conversation

@MarcStdt

Copy link
Copy Markdown
Contributor

Summary

  • resolve scroll OCI manifest annotations before create-time worker materialization
  • pass gg.druid.scroll.minDisk into Kubernetes pull worker actions
  • request that storage size for newly created runtime PVCs, falling back to 1Gi when absent/invalid

Why

Fresh prod prebuild E2E CRs were getting 1Gi runtime PVCs even when artifacts declare larger minDisk values, causing large prebuild pulls to fail before routing.

Tests

  • GOCACHE=/private/tmp/druid-go-cache go test ./internal/runtime/kubernetes
  • GOCACHE=/private/tmp/druid-go-cache go test ./apps/druid/adapters/cli
  • GOCACHE=/private/tmp/druid-go-cache go test ./internal/core/services/registry
  • GOCACHE=/private/tmp/druid-go-cache go test ./apps/druid/core/services
  • GOCACHE=/private/tmp/druid-go-cache go test ./...

@druid-infra

Copy link
Copy Markdown
Contributor
Error: This repo is not allowlisted for Atlantis.

@MarcStdt MarcStdt merged commit a448108 into master Jun 12, 2026
10 checks passed
@MarcStdt MarcStdt deleted the fix/k8s-worker-pvc-min-disk branch June 12, 2026 14:19
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.

2 participants