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
Add migration shim for verifyvolumeattachment and bulk verify #80443
Conversation
7228ef0
to
d0aaaf6
Compare
/priority important-soon |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: davidz627 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 |
/assign @jsafrane |
/retest |
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, only the logging looks too verbose.
d0aaaf6
to
8eb2147
Compare
8eb2147
to
aa91547
Compare
aa91547
to
290a7f1
Compare
/retest |
1 similar comment
/retest |
/lgtm |
VerifyVolumesAreAttached
andBulkVolumeVerify
were not shimmed to CSI when migration was enabled for the verification plugin. This implements the shim layer for those functions.However,
VerifyVolumesAreAttached
for CSI is broken since it checks theVolumeAttachment
object which is not currently necessarily representative of what's attached in the backend (if something was detached out of band VolumeAttachment wont see that). andBulkVolumeVerify
is not implemented for CSI.Those are separate issues that can be worked on after this PR.
/kind feature
/sig storage
/assign @msau42 @ddebroy @leakingtapan @saad-ali
/cc @gnufied @jsafrane @andrewsykim @adisky @andyzhangx