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 storageclass encryption config. #42

Merged
merged 1 commit into from
Jan 20, 2022

Conversation

MarvinCai
Copy link
Contributor

In newly create AWS poolmember, PVC stuck in pending status with events:

Events:
  Type     Reason                Age                    From                                                                                      Message
  ----     ------                ----                   ----                                                                                      -------
  Normal   WaitForFirstConsumer  43m                    persistentvolume-controller                                                               waiting for first consumer to be created before binding
  Warning  ProvisioningFailed    14m (x15 over 43m)     ebs.csi.aws.com_ebs-csi-controller-86d995b7c7-hw9h8_6dfba1d0-c1a4-4576-af9d-debab8944b08  failed to provision volume with StorageClass "sn-default": rpc error: code = InvalidArgument desc = Invalid parameter key encryption for CreateVolume

Seems it should be encrypted not encryption: https://kubernetes.io/docs/concepts/storage/storage-classes/#aws-ebs

@MarvinCai MarvinCai requested a review from jrsdav January 20, 2022 10:57
Copy link
Contributor

@jrsdav jrsdav 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 finding and fixing this!

@jrsdav jrsdav merged commit b5f10f1 into master Jan 20, 2022
@delete-merged-branch delete-merged-branch bot deleted the zxc/fix-storageclass-encryption branch January 20, 2022 16:37
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 this pull request may close these issues.

2 participants