[Platform][Kubernetes] Use az.updateConfig for incluster credentials (#…

…10928) c0b9786 changed the behavior of az.setConfig and added az.updateConfig. This change uses the correct method to add `{"KUBECONFIG": ""}` to the existing configuration instead of overwriting the whole AZ configuration. This was missed when I rebased #8371 against latest changes. Fixes #10860 Signed-off-by: Bhavin Gandhi <[email protected]>
yugabyte · Feb 10, 2022 · 42d85c4 · 42d85c4
1 parent d29d316
commit 42d85c4
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/managed/src/main/java/com/yugabyte/yw/controllers/handlers/CloudProviderHandler.java b/managed/src/main/java/com/yugabyte/yw/controllers/handlers/CloudProviderHandler.java
@@ -222,7 +222,7 @@ public Provider createKubernetes(Customer customer, KubernetesProviderFormData f
         boolean isConfigInZone = updateKubeConfigForZone(provider, region, az, zoneConfig, false);
         if (!(isConfigInProvider || isConfigInRegion || isConfigInZone)) {
           // Use in-cluster ServiceAccount credentials
-          az.setConfig(ImmutableMap.of("KUBECONFIG", ""));
+          az.updateConfig(ImmutableMap.of("KUBECONFIG", ""));
           az.save();
         }
       }
@@ -290,7 +290,7 @@ public Provider createKubernetesNew(Customer customer, Provider reqProvider) thr
         boolean isConfigInZone = updateKubeConfigForZone(provider, region, az, zoneConfig, false);
         if (!(isConfigInProvider || isConfigInRegion || isConfigInZone)) {
           // Use in-cluster ServiceAccount credentials
-          az.setConfig(ImmutableMap.of("KUBECONFIG", ""));
+          az.updateConfig(ImmutableMap.of("KUBECONFIG", ""));
           az.save();
         }
       }