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
[manila-csi-plugin]: Remove unused manila code #2299
Conversation
1a03585
to
69de40f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM. Thanks!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for the clean up, @kayrus! The interfaces in compatibility and capability packages were there to fill in for missing Manila features, specifically share-from-snapshot with cephfs. The implementation for that never landed as it was not very performant, and Manila supports it in recent versions anyway. I'm ok for removing this.
/lgtm
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jichenjc 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 |
What this PR does / why we need it:
Resolves #2298
Which issue this PR fixes(if applicable):
fixes #2298
Special notes for reviewers:
Release note: