linode / linode-blockstorage-csi-driver

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

[feat] Enable support for encrypted volumes for CSI driver #299

Closed prajwalvathreya closed 2 weeks ago

prajwalvathreya commented 4 weeks ago

General:

This PR focuses on enabling support for encrypted volumes for the CSI driver.

It addresses the following tasks:

Pull Request Guidelines:

  1. [ ] Does your submission pass tests?
  2. [ ] Have you added tests?
  3. [x] 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. [ ] Have you saved your large formatting changes for a different PR, so we can focus on your work?
  7. [ ] Have you explained your rationale for why this feature is needed?
  8. [ ] Have you linked your PR to an open issue
codecov[bot] commented 3 weeks ago

Codecov Report

Attention: Patch coverage is 84.28571% with 11 lines in your changes missing coverage. Please review.

Project coverage is 74.79%. Comparing base (c523c92) to head (02a5bd1). Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
internal/driver/controllerserver_helper.go 83.58% 8 Missing and 3 partials :warning:
Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #299 +/- ## ========================================== + Coverage 73.68% 74.79% +1.10% ========================================== Files 22 22 Lines 2356 2396 +40 ========================================== + Hits 1736 1792 +56 + Misses 522 499 -23 - Partials 98 105 +7 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.