fix: resource group issue in disk attach

[andyzhangx]

What type of PR is this?

/kind bug

What this PR does / why we need it:

fix: resource group issue in disk attach

Original behavior is using disk resource group as node resource group which is wrong, there would be issue when disk is not in node resource group, this PR fixes the issue.

cloud-provider-azure/pkg/provider/azure_controller_common.go

Lines 270 to 274 in c905fb1

	resourceGroup, _, err := getInfoFromDiskURI(diskURI)
	if err != nil {
	return -1, err
	}
	return lun, vmset.WaitForUpdateResult(ctx, future, nodeName, resourceGroup, "attach_disk")

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?

fix: resource group issue in disk attach

Additional documentation e.g., KEPs (Kubernetes Enhancement Proposals), usage docs, etc.:

fix: resource group issue in disk attach

[netlify]

✅ Deploy Preview for kubernetes-sigs-cloud-provide-azure canceled.

Name	Link
🔨 Latest commit	2fcc699
🔍 Latest deploy log	https://app.netlify.com/sites/kubernetes-sigs-cloud-provide-azure/deploys/637e2f61a349710008597a10

[coveralls]

Coverage decreased (-0.03%) to 79.596% when pulling 2fcc699 on andyzhangx:rg-issue into c905fb1 on kubernetes-sigs:master.

[mukhoakash]

/lgtm
@andyzhangx I have written an e2e test case with similar changes to fix the resource group fix in the csi driver. I will discard by branch and test with your changes to verify that this issue gets covered in the csi driver e2e test.

[k8s-ci-robot]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: andyzhangx, mukhoakash

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:

• OWNERS [andyzhangx]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

[mukhoakash]

/lgtm @andyzhangx I have written an e2e test case with similar changes to fix the resource group fix in the csi driver. I will discard by branch and test with your changes to verify that this issue gets covered in the csi driver e2e test.

This was my change for reference: mukhoakash@f81141f