Skip to content

Allow overwrite “accessModes” to support AWS ebs for k8s volumes#74

Open
devopsmash wants to merge 1 commit intomage-ai:masterfrom
devopsmash:master
Open

Allow overwrite “accessModes” to support AWS ebs for k8s volumes#74
devopsmash wants to merge 1 commit intomage-ai:masterfrom
devopsmash:master

Conversation

@devopsmash
Copy link
Copy Markdown

@devopsmash devopsmash commented Feb 12, 2025

Summary

  • Allow overwrite “accessModes” to support AWS ebs for k8s volumes
  • Now if you choose persistence, it will attach the new PVC directly to the deployment

Tests

I've added additional custom values file (custom-values.yaml) with the following content:

persistence:
  enabled: true
  dynamicProvisioning: true
  storageClassName: gp3
  size: 10Gi
  accessModes:
  - ReadWriteOnce

Test commands

  • helm template -f .\values.yaml -f .\custom-values.yaml .
  • helm -n mageai upgrade --install mageai -f .\values.yaml -f .\custom-values.yaml .

Notes:

  • I believe that we can omit the .Values.volumes from the helm chart

closes #70

Inspired from the Jenkins helm chart

@devopsmash
Copy link
Copy Markdown
Author

Hi @wangxiaoyou1993 , please let me now if additional info is needed or if something is unclear.

@wangxiaoyou1993
Copy link
Copy Markdown
Member

Will review the PR today

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.

overwrite “accessModes” to support AWS ebs for k8s volumes

2 participants