Merge pull request #135 from rishabh-11/master

Add logs for successful deletion of NIC, Disk and VM
gardener · Mar 1, 2024 · a2ae616 · a2ae616
2 parents 1d9430c + 5a60e1e
commit a2ae616
Show file tree

Hide file tree

Showing 3 changed files with 6 additions and 0 deletions.
diff --git a/pkg/azure/access/helpers/disk.go b/pkg/azure/access/helpers/disk.go
@@ -16,6 +16,7 @@ package helpers
 
 import (
 	"context"
+	"k8s.io/klog/v2"
 
 	"github.com/Azure/azure-sdk-for-go/sdk/azcore/runtime"
 	"github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/compute/armcompute/v5"
@@ -44,5 +45,6 @@ func DeleteDisk(ctx context.Context, client *armcompute.DisksClient, resourceGro
 	if err != nil {
 		errors.LogAzAPIError(err, "Polling failed while waiting for Deleting for [resourceGroup: %s, Name: %s]", diskName, resourceGroup)
 	}
+	klog.Infof("Successfully deleted Disk: %s, for ResourceGroup: %s", diskName, resourceGroup)
 	return
 }
diff --git a/pkg/azure/access/helpers/nic.go b/pkg/azure/access/helpers/nic.go
@@ -16,6 +16,7 @@ package helpers
 
 import (
 	"context"
+	"k8s.io/klog/v2"
 	"time"
 
 	"github.com/Azure/azure-sdk-for-go/sdk/azcore/runtime"
@@ -55,6 +56,7 @@ func DeleteNIC(ctx context.Context, client *armnetwork.InterfacesClient, resourc
 	if err != nil {
 		errors.LogAzAPIError(err, "Polling failed while waiting for Deleting of NIC [ResourceGroup: %s, Name: %s]", resourceGroup, nicName)
 	}
+	klog.Infof("Successfully deleted NIC: %s, for ResourceGroup: %s", nicName, resourceGroup)
 	return
 }
 

diff --git a/pkg/azure/access/helpers/vm.go b/pkg/azure/access/helpers/vm.go
@@ -16,6 +16,7 @@ package helpers
 
 import (
 	"context"
+	"k8s.io/klog/v2"
 	"time"
 
 	"github.com/Azure/azure-sdk-for-go/sdk/resourcemanager/compute/armcompute/v5"
@@ -76,6 +77,7 @@ func DeleteVirtualMachine(ctx context.Context, vmAccess *armcompute.VirtualMachi
 		errors.LogAzAPIError(err, "Polling failed while waiting for delete of VM: %s for ResourceGroup: %s", vmName, resourceGroup)
 		return
 	}
+	klog.Infof("Successfully deleted VM: %s, for ResourceGroup: %s", vmName, resourceGroup)
 	return
 }