Add error handling for CI steps (Azure#22423)

Author: wyunchi-ms

tools/ExecuteCIStep.ps1

@@ -394,13 +394,49 @@ If ($StaticAnalysis)
     {
         $Parameters["TargetModule"] = $TargetModule
     }
-    .("$PSScriptRoot/ExecuteCIStep.ps1") -StaticAnalysisBreakingChange @Parameters
-    .("$PSScriptRoot/ExecuteCIStep.ps1") -StaticAnalysisDependency @Parameters
-    .("$PSScriptRoot/ExecuteCIStep.ps1") -StaticAnalysisSignature @Parameters
-    .("$PSScriptRoot/ExecuteCIStep.ps1") -StaticAnalysisHelp @Parameters
-    .("$PSScriptRoot/ExecuteCIStep.ps1") -StaticAnalysisUX @Parameters
-    .("$PSScriptRoot/ExecuteCIStep.ps1") -StaticAnalysisCmdletDiff @Parameters
-    .("$PSScriptRoot/ExecuteCIStep.ps1") -StaticAnalysisGeneratedSdk @Parameters
+    $FailedTasks = @()
+    $ErrorLogPath = "$StaticAnalysisOutputDirectory/error.log"
+    .("$PSScriptRoot/ExecuteCIStep.ps1") -StaticAnalysisBreakingChange @Parameters 2>$ErrorLogPath
+    If ($LASTEXITCODE -ne 0)
+    {
+        $FailedTasks += "BreakingChange"
+    }
+    .("$PSScriptRoot/ExecuteCIStep.ps1") -StaticAnalysisDependency @Parameters 2>>$ErrorLogPath
+    If ($LASTEXITCODE -ne 0)
+    {
+        $FailedTasks += "Dependency"
+    }
+    .("$PSScriptRoot/ExecuteCIStep.ps1") -StaticAnalysisSignature @Parameters 2>>$ErrorLogPath
+    If ($LASTEXITCODE -ne 0)
+    {
+        $FailedTasks += "Signature"
+    }
+    .("$PSScriptRoot/ExecuteCIStep.ps1") -StaticAnalysisHelp @Parameters 2>>$ErrorLogPath
+    If ($LASTEXITCODE -ne 0)
+    {
+        $FailedTasks += "Help"
+    }
+    .("$PSScriptRoot/ExecuteCIStep.ps1") -StaticAnalysisUX @Parameters 2>>$ErrorLogPath
+    If ($LASTEXITCODE -ne 0)
+    {
+        $FailedTasks += "UXMetadata"
+    }
+    .("$PSScriptRoot/ExecuteCIStep.ps1") -StaticAnalysisCmdletDiff @Parameters 2>>$ErrorLogPath
+    If ($LASTEXITCODE -ne 0)
+    {
+        $FailedTasks += "CmdletDiff"
+    }
+    .("$PSScriptRoot/ExecuteCIStep.ps1") -StaticAnalysisGeneratedSdk @Parameters 2>>$ErrorLogPath
+    If ($LASTEXITCODE -ne 0)
+    {
+        $FailedTasks += "GenertedSdk"
+    }
+    If ($FailedTasks.Length -ne 0)
+    {
+        Write-Host "There are failed tasks: $FailedTasks"
+        $ErrorLog = Get-Content -Path $ErrorLogPath | Join-String -Separator "`n"
+        Write-Error $ErrorLog
+    }
     Return
 }
 
@@ -418,6 +454,10 @@ If ($StaticAnalysisBreakingChange)
     {
         Write-Host "Running static analysis for breaking change..."
         dotnet $RepoArtifacts/StaticAnalysis/StaticAnalysis.Netcore.dll -p $RepoArtifacts/$Configuration -r $StaticAnalysisOutputDirectory --analyzers breaking-change -u -m $BreakingChangeCheckModuleList
+        If ($LASTEXITCODE -ne 0)
+        {
+            Return $LASTEXITCODE
+        }
     }
     Return
 }
@@ -435,6 +475,10 @@ If ($StaticAnalysisDependency)
     {
         Write-Host "Running static analysis for dependency..."
         dotnet $RepoArtifacts/StaticAnalysis/StaticAnalysis.Netcore.dll -p $RepoArtifacts/$Configuration -r $StaticAnalysisOutputDirectory --analyzers dependency -u -m $DependencyCheckModuleList
+        If ($LASTEXITCODE -ne 0)
+        {
+            Return $LASTEXITCODE
+        }
         .($PSScriptRoot + "/CheckAssemblies.ps1") -BuildConfig $Configuration
     }
     Return
@@ -454,6 +498,10 @@ If ($StaticAnalysisSignature)
     {
         Write-Host "Running static analysis for signature..."
         dotnet $RepoArtifacts/StaticAnalysis/StaticAnalysis.Netcore.dll -p $RepoArtifacts/$Configuration -r $StaticAnalysisOutputDirectory --analyzers signature -u -m $SignatureCheckModuleList
+        If ($LASTEXITCODE -ne 0)
+        {
+            Return $LASTEXITCODE
+        }
     }
     Return
 }
@@ -472,6 +520,10 @@ If ($StaticAnalysisHelp)
     {
         Write-Host "Running static analysis for help..."
         dotnet $RepoArtifacts/StaticAnalysis/StaticAnalysis.Netcore.dll -p $RepoArtifacts/$Configuration -r $StaticAnalysisOutputDirectory --analyzers help -u -m $HelpCheckModuleList
+        If ($LASTEXITCODE -ne 0)
+        {
+            Return $LASTEXITCODE
+        }
     }
     Return
 }
@@ -490,6 +542,10 @@ If ($StaticAnalysisUX)
     {
         Write-Host "Running static analysis for UX metadata..."
         dotnet $RepoArtifacts/StaticAnalysis/StaticAnalysis.Netcore.dll -p $RepoArtifacts/$Configuration -r $StaticAnalysisOutputDirectory --analyzers ux -u -m $UXModuleList
+        If ($LASTEXITCODE -ne 0)
+        {
+            Return $LASTEXITCODE
+        }
     }
     Return
 }
@@ -508,6 +564,10 @@ If ($StaticAnalysisCmdletDiff)
     {
         Write-Host "Running static analysis for cmdlet diff..."
         dotnet $RepoArtifacts/StaticAnalysis/StaticAnalysis.Netcore.dll -p $RepoArtifacts/$Configuration -r $StaticAnalysisOutputDirectory --analyzers cmdlet-diff -u -m $CmdletDiffModuleList
+        If ($LASTEXITCODE -ne 0)
+        {
+            Return $LASTEXITCODE
+        }
     }
     Return
 }
@@ -527,6 +587,10 @@ If ($StaticAnalysisGeneratedSdk)
         Write-Host "Running static analysis to verify generated sdk..."
         $result = .($PSScriptRoot + "/StaticAnalysis/GeneratedSdkAnalyzer/SDKGeneratedCodeVerify.ps1")
         Write-Host "Static analysis to verify generated sdk result: $result"
+        If ($LASTEXITCODE -ne 0)
+        {
+            Return $LASTEXITCODE
+        }
     }
     Return
 }