-
Notifications
You must be signed in to change notification settings - Fork 199
Only attach the volumes which need to be attached #3770
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
Only attach the volumes which need to be attached #3770
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: skogta The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
Hi @skogta. Thanks for your PR. I'm waiting for a github.com member to verify that this patch is reasonable to test. If it is, they should reply with Once the patch is verified, the new status will be reflected by the I understand the commands that are listed here. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. |
ef83dbd to
aae08b0
Compare
e090c56 to
d846a3b
Compare
|
/ok-to-test |
pkg/syncer/cnsoperator/controller/cnsnodevmbatchattachment/cnsnodevmbatchattachment_helper.go
Show resolved
Hide resolved
d846a3b to
b1f037a
Compare
|
/lgtm |
|
Triggering CSI-WCP Pre-checkin Pipeline for this PR... Job takes approximately an hour to complete |
|
SUCCESS --- Jenkins Build #680 |
What this PR does / why we need it:
Update CnsNodeVMBatchAttachment reconciliation to only attach those volumes which are not already attached to the VM.
It is also important to ensure that the CR's status reflects the VM on the VC. So if a VM has a volume attached on the VC but the CR's status does not reflect the same, then that volume must be re-attached again.
Testing done:
Created a batchattach CR with 1 volume:
Added 1 more volume and observed only 1 volume got attached:
Detached 1 volume and observed no volumes were attached:
Deleted the instance, observed that volume was detached and the CR was also deleted:
WCP precheckin: https://jenkins-vcf-csifvt.devops.broadcom.net/job/wcp-instapp-e2e-pre-checkin/680/
VKS precheckin: https://jenkins-vcf-csifvt.devops.broadcom.net/job/vks-instapp-e2e-pre-checkin/619/