diff --git a/staging/src/k8s.io/legacy-cloud-providers/azure/azure_controller_standard.go b/staging/src/k8s.io/legacy-cloud-providers/azure/azure_controller_standard.go index 5ffafc81ce1..ecb972b7379 100644 --- a/staging/src/k8s.io/legacy-cloud-providers/azure/azure_controller_standard.go +++ b/staging/src/k8s.io/legacy-cloud-providers/azure/azure_controller_standard.go @@ -17,7 +17,6 @@ limitations under the License. package azure import ( - "fmt" "net/http" "strings" @@ -132,7 +131,8 @@ func (as *availabilitySet) DetachDisk(diskName, diskURI string, nodeName types.N } if !bFoundDisk { - return nil, fmt.Errorf("detach azure disk failure, disk %s not found, diskURI: %s", diskName, diskURI) + // only log here, next action is to update VM status with original meta data + klog.Errorf("detach azure disk: disk %s not found, diskURI: %s", diskName, diskURI) } newVM := compute.VirtualMachine{ diff --git a/staging/src/k8s.io/legacy-cloud-providers/azure/azure_controller_vmss.go b/staging/src/k8s.io/legacy-cloud-providers/azure/azure_controller_vmss.go index 2d551af3f69..5236cd0aa0b 100644 --- a/staging/src/k8s.io/legacy-cloud-providers/azure/azure_controller_vmss.go +++ b/staging/src/k8s.io/legacy-cloud-providers/azure/azure_controller_vmss.go @@ -17,7 +17,6 @@ limitations under the License. package azure import ( - "fmt" "net/http" "strings" @@ -136,7 +135,8 @@ func (ss *scaleSet) DetachDisk(diskName, diskURI string, nodeName types.NodeName } if !bFoundDisk { - return nil, fmt.Errorf("detach azure disk failure, disk %s not found, diskURI: %s", diskName, diskURI) + // only log here, next action is to update VM status with original meta data + klog.Errorf("detach azure disk: disk %s not found, diskURI: %s", diskName, diskURI) } newVM := compute.VirtualMachineScaleSetVM{