-
Notifications
You must be signed in to change notification settings - Fork 38.8k
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
iscsi storage plugin: bkpPortal should be initialized beforehand #46968
iscsi storage plugin: bkpPortal should be initialized beforehand #46968
Conversation
This patch is a follow up patch for the PR#46239. The bkpPortal in DetachDisk() path should be initialized before using it.
Hi @mtanino. Thanks for your PR. I'm waiting for a kubernetes member to verify that this patch is reasonable to test. If it is, they should reply with 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/test-infra repository. I understand the commands that are listed here. |
@k8s-bot ok to test |
/assign |
@k8s-bot pull-kubernetes-e2e-kops-aws test this |
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: mtanino, rootfs The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these OWNERS Files:
You can indicate your approval by writing |
@k8s-bot pull-kubernetes-e2e-kops-aws test this |
Its a bug fix, assigning to 1.7 |
@k8s-bot pull-kubernetes-e2e-kops-aws test this |
/retest |
1 similar comment
/retest |
Automatic merge from submit-queue (batch tested with PRs 46112, 46764, 46727, 46974, 46968) |
…9-upstream-release-1.6 Automatic merge from submit-queue Automated cherry pick of #46239 Cherry pick of #46239 on release-1.6. #46239: Log out from multiple portals with iscsi storage plugin This Cherry pick also includes fix #46968(follow-up fix for #46239) #46968: iscsi storage plugin: bkpPortal should be initialized beforehand
What this PR does / why we need it:
This patch is a follow up patch for the PR #46239.
The bkpPortal in DetachDisk() path should be initialized before using it.
Special notes for your reviewer:
/cc @rootfs @childsb
Release note: