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

storageClassName doesn't work #602

Open
alter opened this issue Mar 26, 2024 · 1 comment
Open

storageClassName doesn't work #602

alter opened this issue Mar 26, 2024 · 1 comment

Comments

@alter
Copy link

alter commented Mar 26, 2024

Description

I want to set a custom storageClassName, but it doesn't apply, looks like the cluster operator knows nothing about it

Importance

blocker

Location

apiVersion: "zookeeper.pravega.io/v1beta1"
kind: "ZookeeperCluster"
metadata:
  name: "zookeeper"
  namespace: "stands-template"
spec:
  replicas: 3
  storageType: persistence
  persistence:
    storageClassName: "efs-sc"

Suggestions for an improvement

add support of custom storageClassName

Additional info

template:

storageClassName: {{ .Values.persistence.storageClassName }}

values.yaml:

but when I look at PVC:

data-zookeeper-0                                        Bound    pvc-f31f3ca7-4edc-4ba6-af79-33a683576c8b   20Gi       RWO            gp2            5m3s
data-zookeeper-1                                        Bound    pvc-558694b2-b29b-4207-8413-9ae37796d958   20Gi       RWO            gp2            4m31s
data-zookeeper-2                                        Bound    pvc-404f60a1-e29b-43a6-bceb-06c5b985c15a   20Gi       RWO            gp2            3m58s

it set gp2, because it's default storage class in my cluster:

 k get sc
NAME                 PROVISIONER             RECLAIMPOLICY   VOLUMEBINDINGMODE      ALLOWVOLUMEEXPANSION   AGE
efs-sc               efs.csi.aws.com         Delete          Immediate              false                  109d
gp2 (default)        kubernetes.io/aws-ebs   Delete          WaitForFirstConsumer   true                   181d
@alter
Copy link
Author

alter commented Mar 26, 2024

I got that zookeeper-operator/charts/zookeeper/values.yaml wrong example, because it's not operator(but you put it into operator folder - wtf?)
there is another folder with CRD and there is no option to apply custom storageClassName, except defined in CRD, I don't get logic here...

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

No branches or pull requests

1 participant