linode / linode-blockstorage-csi-driver

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

[Feat] Adding CSI Sanity tests to GHA #195

Closed komer3 closed 1 month ago

komer3 commented 1 month ago

Reason for this PR: Kubernetes upstream has prewritten integration tests called csi-sanity which we can run against our CSI Driver to tests it functionality and also make sure we are meeting the CSI Spec (defined here). This PR aims to add these csi-sanity tests to GHA runs to make sure we are further adding testing to catch any unwanted bugs/changes before they are merged.

Updated go version to 1.22.5

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. [ ] Are your commits atomic, addressing one change per commit?
  5. [ ] 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 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 36.40%. Comparing base (e882f69) to head (e577a8a). Report is 1 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #195 +/- ## ========================================= + Coverage 0 36.40% +36.40% ========================================= Files 0 22 +22 Lines 0 1596 +1596 ========================================= + Hits 0 581 +581 - Misses 0 981 +981 - Partials 0 34 +34 ```

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