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

typo in nebula-cluster.yaml file causing deployment failure #170

Closed
porscheme opened this issue Jan 11, 2023 · 4 comments
Closed

typo in nebula-cluster.yaml file causing deployment failure #170

porscheme opened this issue Jan 11, 2023 · 4 comments

Comments

@porscheme
Copy link

Please check the FAQ documentation before raising an issue

Describe the bug (required)
Upgrade nebula cluster to v3.3.0 using helm chart failed

Upgrade nebula cluster to v3.3.0 using helm chart failed with below error

Error: UPGRADE FAILED: error validating "": error validating data: ValidationError(NebulaCluster.spec.storaged): unknown field "dataVolumeClaims" in io.nebula-graph.apps.v1alpha1.NebulaCluster.spec.storaged
helm.go:84: [debug] error validating "": error validating data: ValidationError(NebulaCluster.spec.storaged): unknown field "dataVolumeClaims" in io.nebula-graph.apps.v1alpha1.NebulaCluster.spec.storaged

Looking at the code...these seems a typo in nebula-cluster.yaml files

    dataVolumeClaims:
    - resources:
        requests:
          storage: {{ .Values.nebula.storaged.dataStorage }}
      {{- if .Values.nebula.storageClassName }}
      storageClassName: {{ .Values.nebula.storageClassName }}
      {{- end }}

Your Environments (required)

  • OS: uname -a
  • Commit id (e.g. a3ffc7d8)

How To Reproduce(required)

Steps to reproduce the behavior:

  1. Step 1
  2. Step 2
  3. Step 3

Expected behavior

Additional context

@wey-gu
Copy link
Contributor

wey-gu commented Jan 11, 2023

Dear @porscheme ,

we need to manually update the CRD after helm upgrade, as CRD is not controlled by helm.

please see #163

@wey-gu
Copy link
Contributor

wey-gu commented Jan 11, 2023

cc @MegaByte875 in case i am missing something regarding the typo part

@porscheme
Copy link
Author

Dear @porscheme ,

we need to manually update the CRD after helm upgrade, as CRD is not controlled by helm.

please see #163

After manually applying CRD it worked.
Thanks

@veezhang
Copy link
Contributor

@porscheme OK, 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

3 participants