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

Cannot create PVC #325

Open
zhangluva opened this issue Dec 9, 2022 · 2 comments
Open

Cannot create PVC #325

zhangluva opened this issue Dec 9, 2022 · 2 comments

Comments

@zhangluva
Copy link
Contributor

Has #167 been re-introduced?
I am on operator version 0.0.9, and trying to create PVC on historical and middle manager. The snippet for historical looks like below

nodes:
    historicals:
      druid.port: 8088
      nodeConfigMountPath: /opt/druid/conf/druid/cluster/data/historical
      nodeType: historical
      replicas: 5
      volumeClaimTemplates:
      - apiVersion: v1
        kind: PersistentVolumeClaim
        metadata:
          name: historical
        spec:
          accessModes:
          - ReadWriteOnce
          resources:
            requests:
              storage: 50Gi
          storageClassName: gp2

However, when I looked at the druid definition that the operator created on the cluster, the volumeClaimTemplates.metadata.name was removed. The generated snippet for the PVC

      volumeClaimTemplates:
      - apiVersion: v1
        kind: PersistentVolumeClaim
        metadata: {}
        spec:
          accessModes:
          - ReadWriteOnce
          resources:
            requests:
              storage: 50Gi
          storageClassName: gp2
        status: {}

And describe sts historical shows

│ Events:                                                                                                                                                         │
│   Type     Reason        Age                   From                    Message                                                                                  │
│   ----     ------        ----                  ----                    -------                                                                                  │
│   Warning  FailedCreate  4m23s (x20 over 42m)  statefulset-controller  create Claim -druid-druid-staging-historicals-0 for Pod druid-druid-staging-historicals- │
│ 0 in StatefulSet druid-druid-staging-historicals failed error: PersistentVolumeClaim "-druid-druid-staging-historicals-0" is invalid: metadata.name: Invalid va │
│ lue: "-druid-druid-staging-historicals-0": a lowercase RFC 1123 subdomain must consist of lower case alphanumeric characters, '-' or '.', and must start and en │
│ d with an alphanumeric character (e.g. 'example.com', regex used for validation is '[a-z0-9]([-a-z0-9]*[a-z0-9])?(\.[a-z0-9]([-a-z0-9]*[a-z0-9])?)*')
@AdheipSingh
Copy link
Contributor

Yes, can you try to run with this PR #321 change.
And update your crd definitions.

@zhangluva
Copy link
Contributor Author

Patched CRD fixed it. Thanks!

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

2 participants