linode / linode-blockstorage-csi-driver

Container Storage Interface (CSI) Driver for Linode Block Storage
Apache License 2.0
66 stars 55 forks source link

Update README.md #134

Closed johnybradshaw closed 11 months ago

johnybradshaw commented 12 months ago

In the encrypted StorageClass example the namespace set in the Secret and the namespace referenced by the StorageClass do not match. This will cause pods to fail to deploy.

Namespace mismatch, was default, should be csi-encrypt-example

General:

Pull Request Guidelines:

  1. [ ] Does your submission pass tests?
  2. [ ] Have you added tests?
  3. [ ] Are you addressing a single feature in this PR?
  4. [x] Are your commits atomic, addressing one change per commit?
  5. [x] Are you following the conventions of the language?
  6. [x] Have you saved your large formatting changes for a different PR, so we can focus on your work?
  7. [x] Have you explained your rationale for why this feature is needed?
  8. [ ] Have you linked your PR to an open issue
luthermonson commented 12 months ago

sorry about that! nice catch