Revert "Setting up Azure Active cloud corresponding to service endpoi…

…nt (#9003) (#9006)" This reverts commit 8ebe8db.
microsoft · Dec 12, 2018 · e815460 · e815460
1 parent 6c3c07b
commit e815460
Show file tree

Hide file tree

Showing 4 changed files with 13 additions and 23 deletions.
diff --git a/Tasks/AzureCLIV1/Strings/resources.resjson/en-US/resources.resjson b/Tasks/AzureCLIV1/Strings/resources.resjson/en-US/resources.resjson
@@ -28,6 +28,5 @@
   "loc.messages.FailedToLogout": "The following error occurred while logging out: %s",
   "loc.messages.LoginFailed": "Azure login failed",
   "loc.messages.ErrorInSettingUpSubscription": "Error in setting up subscription",
-  "loc.messages.SettingAzureConfigDir": "Setting AZURE_CONFIG_DIR env variable to: %s",
-  "loc.messages.SettingAzureCloud": "Setting active cloud to: %s"
+  "loc.messages.SettingAzureConfigDir": "Setting AZURE_CONFIG_DIR env variable to: %s"
 }
diff --git a/Tasks/AzureCLIV1/azureclitask.ts b/Tasks/AzureCLIV1/azureclitask.ts
@@ -57,9 +57,7 @@ export class azureclitask {
                 tool = tl.tool(tl.which(scriptPath, true));
             }
             this.throwIfError(tl.execSync("az", "--version"));
-            // set az cli config dir
-            this.setConfigDirectory();
-            this.setAzureCloudBasedOnServiceEndpoint();
+            this.useGlobalConfig = tl.getBoolInput("useGlobalConfig");
             this.loginAzure();
 
             tool.line(args); // additional args should always call line. line() parses quoted arg strings
@@ -106,6 +104,7 @@ export class azureclitask {
     private static isLoggedIn: boolean = false;
     private static cliPasswordPath: string = null;
     private static azCliConfigPath: string;
+    private static useGlobalConfig: boolean = true;
 
     private static loginAzure() {
         var connectedService: string = tl.getInput("connectedServiceNameARM", true);
@@ -132,6 +131,9 @@ export class azureclitask {
         var tenantId: string = tl.getEndpointAuthorizationParameter(connectedService, "tenantid", false);
         var subscriptionID: string = tl.getEndpointDataParameter(connectedService, "SubscriptionID", true);
 
+        // set az cli config dir
+        this.setConfigDirectory();
+
         //login using svn
         this.throwIfError(tl.execSync("az", "login --service-principal -u \"" + servicePrincipalId + "\" -p \"" + cliPassword + "\" --tenant \"" + tenantId + "\""), tl.loc("LoginFailed"));
         this.isLoggedIn = true;
@@ -140,7 +142,7 @@ export class azureclitask {
     }
 
     private static setConfigDirectory(): void {
-        if (tl.getBoolInput("useGlobalConfig")) {
+        if (this.useGlobalConfig) {
             return;
         }
 
@@ -160,15 +162,6 @@ export class azureclitask {
         }
     }
 
-    private static setAzureCloudBasedOnServiceEndpoint(): void {
-        var connectedService: string = tl.getInput("connectedServiceNameARM", true);
-        var environment = tl.getEndpointDataParameter(connectedService, 'environment', true);
-        if(!!environment) {
-            console.log(tl.loc('SettingAzureCloud', environment));
-            this.throwIfError(tl.execSync("az", "cloud set -n " + environment));
-        }
-    }
-
     private static logoutAzure() {
         try {
             tl.execSync("az", " account clear");

diff --git a/Tasks/AzureCLIV1/task.json b/Tasks/AzureCLIV1/task.json
@@ -19,7 +19,7 @@
     "version": {
         "Major": 1,
         "Minor": 143,
-        "Patch": 5
+        "Patch": 4
     },
     "minimumAgentVersion": "2.0.0",
     "instanceNameFormat": "Azure CLI $(scriptPath)",
@@ -94,7 +94,7 @@
             "name": "useGlobalConfig",
             "type": "boolean",
             "label": "Use global Azure CLI configuration",
-            "defaultValue": "false",
+            "defaultValue": "true",
             "required": false,
             "helpMarkDown": "If this is false, this task will use its own separate [Azure CLI configuration directory](https://docs.microsoft.com/en-us/cli/azure/azure-cli-configuration?view=azure-cli-latest#cli-configuration-file). This can be used to run Azure CLI tasks in *parallel* releases",
             "groupName": "advanced"
@@ -135,7 +135,6 @@
         "FailedToLogout": "The following error occurred while logging out: %s",
         "LoginFailed": "Azure login failed",
         "ErrorInSettingUpSubscription": "Error in setting up subscription",
-        "SettingAzureConfigDir": "Setting AZURE_CONFIG_DIR env variable to: %s",
-        "SettingAzureCloud": "Setting active cloud to: %s"
+        "SettingAzureConfigDir": "Setting AZURE_CONFIG_DIR env variable to: %s"
     }
 }
diff --git a/Tasks/AzureCLIV1/task.loc.json b/Tasks/AzureCLIV1/task.loc.json
@@ -19,7 +19,7 @@
   "version": {
     "Major": 1,
     "Minor": 143,
-    "Patch": 5
+    "Patch": 4
   },
   "minimumAgentVersion": "2.0.0",
   "instanceNameFormat": "ms-resource:loc.instanceNameFormat",
@@ -94,7 +94,7 @@
       "name": "useGlobalConfig",
       "type": "boolean",
       "label": "ms-resource:loc.input.label.useGlobalConfig",
-      "defaultValue": "false",
+      "defaultValue": "true",
       "required": false,
       "helpMarkDown": "ms-resource:loc.input.help.useGlobalConfig",
       "groupName": "advanced"
@@ -135,7 +135,6 @@
     "FailedToLogout": "ms-resource:loc.messages.FailedToLogout",
     "LoginFailed": "ms-resource:loc.messages.LoginFailed",
     "ErrorInSettingUpSubscription": "ms-resource:loc.messages.ErrorInSettingUpSubscription",
-    "SettingAzureConfigDir": "ms-resource:loc.messages.SettingAzureConfigDir",
-    "SettingAzureCloud": "ms-resource:loc.messages.SettingAzureCloud"
+    "SettingAzureConfigDir": "ms-resource:loc.messages.SettingAzureConfigDir"
   }
 }