Throwing error if stderr is not null (#54)

* Throwing error if stderr is not null * Improved error message * Removed core.error
Azure · Jun 17, 2021 · 857ecaf · 857ecaf
1 parent 6212dde
commit 857ecaf
Show file tree

Hide file tree

Showing 3 changed files with 3 additions and 3 deletions.
diff --git a/src/deploy/scope_managementgroup.ts b/src/deploy/scope_managementgroup.ts
@@ -68,7 +68,7 @@ export async function DeployManagementGroupScope(azPath: string, region: string,
         core.info("Creating deployment...")
         var deploymentCode = await exec(`"${azPath}" deployment mg create ${azDeployParameters} -o json`, [], deployOptions);
         if (commandStdErr.trim().length !== 0) {
-            core.error(commandStdErr)
+            throw new Error(`Deployment process failed as some lines were written to stderr: ${commandStdErr}`)
         } else {
             if (deploymentCode != 0) {
                 core.error("Deployment failed.")

diff --git a/src/deploy/scope_resourcegroup.ts b/src/deploy/scope_resourcegroup.ts
@@ -70,7 +70,7 @@ export async function DeployResourceGroupScope(azPath: string, resourceGroupName
         core.info("Creating deployment...")
         var deploymentCode = await exec(`"${azPath}" deployment group create ${azDeployParameters} -o json`, [], deployOptions);
         if (commandStdErr.trim().length !== 0) {
-            core.error(commandStdErr)
+            throw new Error(`Deployment process failed as some lines were written to stderr: ${commandStdErr}`)
         } else {
             if (deploymentCode != 0) {
                 core.error("Deployment failed.")

diff --git a/src/deploy/scope_subscription.ts b/src/deploy/scope_subscription.ts
@@ -68,7 +68,7 @@ export async function DeploySubscriptionScope(azPath: string, region: string, te
         core.info("Creating deployment...")
         var deploymentCode = await exec(`"${azPath}" deployment sub create ${azDeployParameters} -o json`, [], deployOptions);
         if (commandStdErr.trim().length !== 0) {
-            core.error(commandStdErr)
+            throw new Error(`Deployment process failed as some lines were written to stderr: ${commandStdErr}`)
         } else {
             if (deploymentCode != 0) {
                 core.error("Deployment failed.")