Merge pull request #88360 from andyzhangx/delete-disk

fix: check disk status before delete azure disk
This commit is contained in:
Kubernetes Prow Robot 2020-02-24 06:42:47 -08:00 committed by GitHub
commit e4a5012de6
No known key found for this signature in database
GPG Key ID: 4AEE18F83AFDEB23

View File

@ -214,7 +214,16 @@ func (c *ManagedDiskController) DeleteManagedDisk(diskURI string) error {
return fmt.Errorf("failed to delete disk(%s) since it's in attaching or detaching state", diskURI)
}
rerr := c.common.cloud.DisksClient.Delete(ctx, resourceGroup, diskName)
disk, rerr := c.common.cloud.DisksClient.Get(ctx, resourceGroup, diskName)
if rerr != nil {
return rerr.Error()
}
if disk.ManagedBy != nil {
return fmt.Errorf("disk(%s) already attached to node(%s), could not be deleted", diskURI, *disk.ManagedBy)
}
rerr = c.common.cloud.DisksClient.Delete(ctx, resourceGroup, diskName)
if rerr != nil {
return rerr.Error()
}