Skip to content

fix: disable aws-ebs-csi-driver by default on EKS#71

Merged
mchmarny merged 3 commits intoNVIDIA:mainfrom
yuanchen8911:fix/disable-ebs-csi-eks
Feb 6, 2026
Merged

fix: disable aws-ebs-csi-driver by default on EKS#71
mchmarny merged 3 commits intoNVIDIA:mainfrom
yuanchen8911:fix/disable-ebs-csi-eks

Conversation

@yuanchen8911
Copy link
Contributor

Summary

  • Disable aws-ebs-csi-driver subchart by default in the EKS overlay

Problem

EKS clusters typically have the EBS CSI driver installed as a managed addon. When eidos deploys it again, Helm fails with:

Error: INSTALLATION FAILED: unable to continue with install: ClusterRole "ebs-external-attacher-role"
exists and cannot be imported into the current release: invalid ownership metadata;
label validation error: key "app.kubernetes.io/managed-by" must equal "Helm": current value is "EKS"

Confirmed EBS CSI controller pods already running in kube-system as EKS-managed addon.

Changes

  • Modified: pkg/recipe/data/overlays/eks.yaml — added overrides: enabled: false to aws-ebs-csi-driver component

Users without the managed addon can re-enable with: --set aws-ebs-csi-driver.enabled=true

Test plan

  • go test ./pkg/recipe/... ./pkg/bundler/... — all pass
  • Generated bundle shows aws-ebs-csi-driver.enabled: false
  • helm install no longer fails with ClusterRole conflict on EKS

🤖 Generated with Claude Code

EKS clusters typically have the EBS CSI driver installed as a managed
addon. Deploying it again via the eidos bundle causes ClusterRole
conflicts ("managed-by: EKS" vs "managed-by: Helm").

Disable it by default in the EKS overlay. Users without the managed
addon can re-enable with: --set aws-ebs-csi-driver.enabled=true

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@yuanchen8911 yuanchen8911 requested a review from a team as a code owner February 6, 2026 02:17
@yuanchen8911 yuanchen8911 requested a review from mchmarny February 6, 2026 02:23
@github-actions
Copy link

github-actions bot commented Feb 6, 2026

Coverage Report ✅

Metric Value
Coverage 73.2%
Threshold 70%
Status Pass
Coverage Badge
![Coverage](https://img.shields.io/badge/coverage-73.2%25-green)

No Go source files changed in this PR.

@mchmarny mchmarny merged commit 6947c11 into NVIDIA:main Feb 6, 2026
15 checks passed
@mchmarny mchmarny deleted the fix/disable-ebs-csi-eks branch February 6, 2026 09:47
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.

3 participants