Skip to content
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

Snapshot Engine: volumeSnapshotClassName is hardcoded #417

Closed
harryttd opened this issue Apr 13, 2022 · 0 comments · Fixed by #411
Closed

Snapshot Engine: volumeSnapshotClassName is hardcoded #417

harryttd opened this issue Apr 13, 2022 · 0 comments · Fixed by #411
Assignees

Comments

@harryttd
Copy link
Collaborator

harryttd commented Apr 13, 2022

volumeSnapshotClassName: test-snapclass

Right now the volume snapshot class is manually created in the cluster. Pulumi needs to be creating it, because it is cluster scoped and we only want to create it once. So we don’t want it in tezos-k8s which would want to create it for every Helm deployment we do to the cluster.
So Pulumi needs to create it and it would give it a name. That name needs to be passed into values.y which would then have to get it into the VolumeSnapshot resource defined in createVolumeSnapshot.yaml
See https://aws.amazon.com/blogs/containers/using-ebs-snapshots-for-persistent-storage-with-your-eks-cluster/#:~:text=Step%204%3A[…]ass%20and%20Snapshot

nicolasochem added a commit that referenced this issue Apr 13, 2022
from kube doc

A volume snapshot can request a particular class by specifying the name
of a VolumeSnapshotClass using the attribute volumeSnapshotClassName. If
nothing is set, then the default class is used if available.

https://kubernetes.io/docs/concepts/storage/volume-snapshots/
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 a pull request may close this issue.

2 participants