Update dependencies from https://github.com/dotnet/arcade build 20221221.1

dotnet-maestro[bot] · dotnet-maestro[bot] · commit 38c980794116 · 2022-12-22T13:23:02.000Z
Microsoft.DotNet.Arcade.Sdk
 From Version 8.0.0-beta.22616.1 -&gt; To Version 8.0.0-beta.22621.1
diff --git a/eng/Version.Details.xml b/eng/Version.Details.xml
@@ -3,9 +3,9 @@
   <ProductDependencies>
   </ProductDependencies>
   <ToolsetDependencies>
-    <Dependency Name="Microsoft.DotNet.Arcade.Sdk" Version="8.0.0-beta.22620.3">
+    <Dependency Name="Microsoft.DotNet.Arcade.Sdk" Version="8.0.0-beta.22621.1">
       <Uri>https://github.com/dotnet/arcade</Uri>
-      <Sha>0cc119808e4affdda4b02cb81153de27bce9202e</Sha>
+      <Sha>3003926e4126f827bca50d5b3ee179afc86d8a7b</Sha>
       <SourceBuild RepoName="arcade" ManagedOnly="true" />
     </Dependency>
   </ToolsetDependencies>
diff --git a/eng/common/tools.ps1 b/eng/common/tools.ps1
@@ -815,7 +815,8 @@ function MSBuild-Core() {
       Write-Host "See log: $buildLog" -ForegroundColor DarkGray
     }
 
-    if ($ci) {
+    # When running on Azure Pipelines, override the returned exit code to avoid double logging.
+    if ($ci -and $env:SYSTEM_TEAMPROJECT -ne $null) {
       Write-PipelineSetResult -Result "Failed" -Message "msbuild execution failed."
       # Exiting with an exit code causes the azure pipelines task to log yet another "noise" error
       # The above Write-PipelineSetResult will cause the task to be marked as failure without adding yet another error
diff --git a/eng/common/tools.sh b/eng/common/tools.sh
@@ -472,7 +472,9 @@ function MSBuild-Core {
       # We should not Write-PipelineTaskError here because that message shows up in the build summary
       # The build already logged an error, that's the reason it failed. Producing an error here only adds noise.
       echo "Build failed with exit code $exit_code. Check errors above."
-      if [[ "$ci" == "true" ]]; then
+
+      # When running on Azure Pipelines, override the returned exit code to avoid double logging.
+      if [[ "$ci" == "true" && -n ${SYSTEM_TEAMPROJECT:-} ]]; then
         Write-PipelineSetResult -result "Failed" -message "msbuild execution failed."
         # Exiting with an exit code causes the azure pipelines task to log yet another "noise" error
         # The above Write-PipelineSetResult will cause the task to be marked as failure without adding yet another error
diff --git a/global.json b/global.json
@@ -18,6 +18,6 @@
     "rollForward": "patch"
   },
   "msbuild-sdks": {
-    "Microsoft.DotNet.Arcade.Sdk": "8.0.0-beta.22620.3"
+    "Microsoft.DotNet.Arcade.Sdk": "8.0.0-beta.22621.1"
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -815,7 +815,8 @@ function MSBuild-Core() {`
`815`	`815`	`Write-Host "See log: $buildLog" -ForegroundColor DarkGray`
`816`	`816`	`}`
`817`	`817`
`818`		`- if ($ci) {`
	`818`	`+ # When running on Azure Pipelines, override the returned exit code to avoid double logging.`
	`819`	`+ if ($ci -and $env:SYSTEM_TEAMPROJECT -ne $null) {`
`819`	`820`	`Write-PipelineSetResult -Result "Failed" -Message "msbuild execution failed."`
`820`	`821`	`# Exiting with an exit code causes the azure pipelines task to log yet another "noise" error`
`821`	`822`	`# The above Write-PipelineSetResult will cause the task to be marked as failure without adding yet another error`
Original file line number	Diff line number	Diff line change
`@@ -18,6 +18,6 @@`
`18`	`18`	`"rollForward": "patch"`
`19`	`19`	`},`
`20`	`20`	`"msbuild-sdks": {`
`21`		`- "Microsoft.DotNet.Arcade.Sdk": "8.0.0-beta.22620.3"`
	`21`	`+ "Microsoft.DotNet.Arcade.Sdk": "8.0.0-beta.22621.1"`
`22`	`22`	`}`
`23`	`23`	`}`