Skip to content
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

refact(deps): use lib-csi v0.6.0 for csi pv leak protection #90

Merged
merged 1 commit into from
Jun 8, 2021

Conversation

iyashu
Copy link
Contributor

@iyashu iyashu commented Jun 7, 2021

Why is this PR required? What issue does it fix?:
Leak protection controller moved to openebs/lib-csi#11

What this PR does?:
Use lib-csi v0.6.0

Does this PR require any upgrade changes?:
No

If the changes in this PR are manually verified, list down the scenarios covered::

Any additional information for your reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs

Checklist:

  • Fixes #
  • PR Title follows the convention of <type>(<scope>): <subject>
  • Has the change log section been updated?
  • Commit has unit tests
  • Commit has integration tests
  • (Optional) Are upgrade changes included in this PR? If not, mention the issue/PR to track:
  • (Optional) If documentation changes are required, which issue on https://github.com/openebs/openebs-docs is used to track them:

Signed-off-by: Yashpal Choudhary <yashpal.c1995@gmail.com>
@iyashu iyashu requested a review from akhilerm June 7, 2021 21:45
Copy link
Contributor

@akhilerm akhilerm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@prateekpandey14 prateekpandey14 merged commit aa439a1 into openebs:master Jun 8, 2021
akhilerm pushed a commit to akhilerm/lvm-localpv that referenced this pull request Jun 18, 2021
Signed-off-by: Yashpal Choudhary <yashpal.c1995@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants