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

fix: PVC should use claimName key #369

Merged
merged 1 commit into from
Jul 26, 2023
Merged

Conversation

fuhrmannb
Copy link
Contributor

When using claim-name for persistent pack volumes, the following error is raised by Helm:

> helm upgrade my-st2 stackstorm/stackstorm-ha --values values.yaml --debug
upgrade.go:144: [debug] preparing upgrade for my-st2
Error: UPGRADE FAILED: error validating "": error validating data: [ValidationError(Deployment.spec.template.spec.volumes[2].persistentVolumeClaim): unknown field "claim-name" in io.k8s.api.core.v1.PersistentVolumeClaimVolumeSource, ValidationError(Deployment.spec.template.spec.volumes[2].persistentVolumeClaim): missing required field "claimName" in io.k8s.api.core.v1.PersistentVolumeClaimVolumeSource, ValidationError(Deployment.spec.template.spec.volumes[3].persistentVolumeClaim): unknown field "claim-name" in io.k8s.api.core.v1.PersistentVolumeClaimVolumeSource, ValidationError(Deployment.spec.template.spec.volumes[3].persistentVolumeClaim): missing required field "claimName" in io.k8s.api.core.v1.PersistentVolumeClaimVolumeSource, ValidationError(Deployment.spec.template.spec.volumes[4].persistentVolumeClaim): unknown field "claim-name" in io.k8s.api.core.v1.PersistentVolumeClaimVolumeSource, ValidationError(Deployment.spec.template.spec.volumes[4].persistentVolumeClaim): missing required field "claimName" in io.k8s.api.core.v1.PersistentVolumeClaimVolumeSource]

Fix by using claimName instead for persistentVolumeClaim sections.

When using `claim-name` for persistent pack volumes, the following error
is raised by Helm:

```
> helm upgrade my-st2 stackstorm/stackstorm-ha --values values.yaml --debug
upgrade.go:144: [debug] preparing upgrade for my-st2
Error: UPGRADE FAILED: error validating "": error validating data: [ValidationError(Deployment.spec.template.spec.volumes[2].persistentVolumeClaim): unknown field "claim-name" in io.k8s.api.core.v1.PersistentVolumeClaimVolumeSource, ValidationError(Deployment.spec.template.spec.volumes[2].persistentVolumeClaim): missing required field "claimName" in io.k8s.api.core.v1.PersistentVolumeClaimVolumeSource, ValidationError(Deployment.spec.template.spec.volumes[3].persistentVolumeClaim): unknown field "claim-name" in io.k8s.api.core.v1.PersistentVolumeClaimVolumeSource, ValidationError(Deployment.spec.template.spec.volumes[3].persistentVolumeClaim): missing required field "claimName" in io.k8s.api.core.v1.PersistentVolumeClaimVolumeSource, ValidationError(Deployment.spec.template.spec.volumes[4].persistentVolumeClaim): unknown field "claim-name" in io.k8s.api.core.v1.PersistentVolumeClaimVolumeSource, ValidationError(Deployment.spec.template.spec.volumes[4].persistentVolumeClaim): missing required field "claimName" in io.k8s.api.core.v1.PersistentVolumeClaimVolumeSource]
```

Fix by using `claimName` instead for `persistentVolumeClaim` sections.
@pull-request-size pull-request-size bot added the size/XS PR that changes 0-9 lines. Quick fix/merge. label Jul 25, 2023
@CLAassistant
Copy link

CLAassistant commented Jul 25, 2023

CLA assistant check
All committers have signed the CLA.

Copy link
Member

@arm4b arm4b left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the fix @fuhrmannb !

@arm4b arm4b merged commit 415e3a6 into StackStorm:master Jul 26, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation size/XS PR that changes 0-9 lines. Quick fix/merge.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants