-
Notifications
You must be signed in to change notification settings - Fork 351
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Can't use "volume.shared" in Compontent Chart #1868
Comments
@johnnypea thanks for creating this issue! Yes seems like this is not part of the latest spec in devspace, we'll fix that for the next release! |
@FabianKramm thanks! Please, could you give me working example how can I use this sharing a volume between the pods. It is not very clear from the docs. Should "shared" be set for every pod which is using that shared volume? |
@johnnypea As far as I see it in the chart's templates, the EDIT: Yes, |
@LukasGentele @FabianKramm I finally got to test it on 5.18.3 but still no luck getting it work...
When I removed the volume definition I just got:
|
@johnnypea when you get the resource already exist error, can you do a |
@FabianKramm I did This pod yaml if it would be any help. I can't see "shared" definition there, is that correct?
|
@johnnypea thanks for the info, can you share your devspace.yaml that fails during |
@LukasGentele Yes, I do. I sent it in my previous comment. Do you need any more details? |
@johnnypea thanks for the info, I guess thats currently a limitation of the component chart as it tries to create 2 PVC's here. You are probably better off to create your own custom helm chart that does this the way you want with a single PVC and multiple deployments |
@FabianKramm would it be possible to use it like this?
pwa/deploy/pwa-dist-pvc.yaml
Getting this error:
|
@johnnypea no, the problem is that the pvc already exists and you need to separate the deployments, where 1 deployment wouldn't create the persistent volume claim |
https://devspace.sh/component-chart/docs/configuration/containers#volumeshared
The text was updated successfully, but these errors were encountered: