Skip to content

fix: increase minio PVC to 2Ti to match UAT#1196

Merged
Gkrumbach07 merged 1 commit intomainfrom
fix/minio-pvc-size-uat
Apr 3, 2026
Merged

fix: increase minio PVC to 2Ti to match UAT#1196
Gkrumbach07 merged 1 commit intomainfrom
fix/minio-pvc-size-uat

Conversation

@Gkrumbach07
Copy link
Copy Markdown
Contributor

@Gkrumbach07 Gkrumbach07 commented Apr 3, 2026

Summary

  • UAT cluster already has a 2TB minio-data PVC, but the base manifest specifies 500Gi
  • Kubernetes rejects shrinking a PVC, causing oc apply to fail with: spec.resources.requests.storage: Forbidden: field can not be less than previous value
  • Adds a production overlay patch to set minio-data PVC to 2Ti

Fixes https://github.com/ambient-code/platform/actions/runs/23954839404/job/69871041808

Test plan

  • Verify kustomize build components/manifests/overlays/production renders minio-data PVC with storage: 2Ti
  • Deploy to UAT succeeds without PVC error

🤖 Generated with Claude Code

Summary by CodeRabbit

  • Chores
    • Updated production environment configuration to allocate 2Ti of storage for MinIO object storage service.

UAT cluster already has a 2TB minio-data PVC. The base manifest specifies
500Gi, causing deploy failures with "field can not be less than previous
value".

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Apr 3, 2026

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

Run ID: 143e34c5-02c7-4fa2-a8a1-51bb15977f33

📥 Commits

Reviewing files that changed from the base of the PR and between d9106f3 and 236ce24.

📒 Files selected for processing (2)
  • components/manifests/overlays/production/kustomization.yaml
  • components/manifests/overlays/production/minio-pvc-patch.yaml

📝 Walkthrough

Walkthrough

Production overlay now includes a Kustomize patch for MinIO's PersistentVolumeClaim, specifying 2Ti storage. The patch is registered in kustomization.yaml and defined in a new dedicated manifest file.

Changes

Cohort / File(s) Summary
MinIO PVC Configuration
components/manifests/overlays/production/kustomization.yaml, components/manifests/overlays/production/minio-pvc-patch.yaml
Adds Kustomize patch targeting MinIO PVC resource with 2Ti storage allocation via new patch manifest.
🚥 Pre-merge checks | ✅ 6
✅ Passed checks (6 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title check ✅ Passed The title follows Conventional Commits format with 'fix' type and clearly describes the main change: increasing MinIO PVC storage to 2Ti for production UAT environment compatibility.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Performance And Algorithmic Complexity ✅ Passed PR contains only Kubernetes manifest changes—Kustomize patch and PersistentVolumeClaim—with no algorithmic operations or performance risks.
Security And Secret Handling ✅ Passed PR modifies only two files with zero sensitive data: kustomization.yaml references a patch file, and minio-pvc-patch.yaml is a minimal Kubernetes manifest specifying only storage capacity without credentials or tokens.
Kubernetes Resource Safety ✅ Passed Minimal strategic merge patch updating minio-data PVC storage (500Gi→2Ti) without introducing child resources, modifying containers, altering RBAC, or affecting namespace scoping/security contexts.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch fix/minio-pvc-size-uat
✨ Simplify code
  • Create PR with simplified code
  • Commit simplified code in branch fix/minio-pvc-size-uat

Comment @coderabbitai help to get the list of available commands and usage tips.

@Gkrumbach07 Gkrumbach07 merged commit 440f5b9 into main Apr 3, 2026
41 checks passed
@Gkrumbach07 Gkrumbach07 deleted the fix/minio-pvc-size-uat branch April 3, 2026 17:41
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.

1 participant