Skip cleanup when not configured with a valid client (#488)

jenkinsci · Dec 16, 2023 · 7309274 · 7309274
1 parent 3600669
commit 7309274
Showing 1 changed file with 8 additions and 0 deletions.
diff --git a/src/main/java/com/microsoft/azure/vmagent/AzureVMAgentCleanUpTask.java b/src/main/java/com/microsoft/azure/vmagent/AzureVMAgentCleanUpTask.java
@@ -329,6 +329,14 @@ public void cleanLeakedResources(
         try {
             final List<String> validVMs = getValidVMs();
             final AzureResourceManager azureClient = cloud.getAzureClient();
+
+            if (azureClient == null) {
+                LOGGER.log(getNormalLoggingLevel(),
+                        "cleanLeakedResources: Skipping cleanup as cloud is not configured with a "
+                                + "valid credential");
+                return;
+            }
+
             final AzureVMManagementServiceDelegate serviceDelegate = cloud.getServiceDelegate();
             // can't use listByTag because for some reason that method strips all the tags from the outputted resources
             // (https://github.com/Azure/azure-sdk-for-java/issues/1436)