-
Notifications
You must be signed in to change notification settings - Fork 806
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
Update to CSI 1.0 #122
Update to CSI 1.0 #122
Conversation
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: bertinatto The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
989ba61
to
d2be26b
Compare
Pull Request Test Coverage Report for Build 191
💛 - Coveralls |
/assign @leakingtapan |
/hold |
d2be26b
to
2350aff
Compare
2350aff
to
ff25c76
Compare
/hold cancel Apparently we have to keep the |
Is this PR tested against v1.13? |
Yes, it' s only tested against v1.13. |
98a5ccc
to
6c86ea3
Compare
I just changed the base branch for this PR from |
/lgtm |
Fixes: kubernetes-sigs#137 I use latest instead of v1.13 to highlight that is the latest version under development which might have issues. And any other versioned manifest files should not break.
Partially revert PR #122 and add latest manifest files
Fixes: kubernetes-sigs#137 I use latest instead of v1.13 to highlight that is the latest version under development which might have issues. And any other versioned manifest files should not break.
Is this a bug fix or adding new feature?
/kind feature
What is this PR about? / Why do we need it?
This PR updates the driver to the new CSI 1.0 spec.