Skip to content

Commit

Permalink
[main] Update dependencies from dotnet/arcade (dotnet#10369)
Browse files Browse the repository at this point in the history
* Update dependencies from https://github.com/dotnet/arcade build 20201029.1

Microsoft.DotNet.Arcade.Sdk
 From Version 6.0.0-beta.20527.10 -> To Version 6.0.0-beta.20529.1

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

Microsoft.DotNet.Arcade.Sdk
 From Version 6.0.0-beta.20527.10 -> To Version 6.0.0-beta.20530.2

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

Microsoft.DotNet.Arcade.Sdk
 From Version 6.0.0-beta.20527.10 -> To Version 6.0.0-beta.20552.5

* retain stable sdk

Co-authored-by: dotnet-maestro[bot] <dotnet-maestro[bot]@users.noreply.github.com>
Co-authored-by: Brett V. Forsgren <brettfo@microsoft.com>
  • Loading branch information
3 people authored and nosami committed Feb 22, 2021
1 parent f84d529 commit 6ebb769
Show file tree
Hide file tree
Showing 3 changed files with 55 additions and 27 deletions.
4 changes: 2 additions & 2 deletions eng/Version.Details.xml
Original file line number Diff line number Diff line change
Expand Up @@ -3,9 +3,9 @@
<ProductDependencies>
</ProductDependencies>
<ToolsetDependencies>
<Dependency Name="Microsoft.DotNet.Arcade.Sdk" Version="6.0.0-beta.20527.10">
<Dependency Name="Microsoft.DotNet.Arcade.Sdk" Version="6.0.0-beta.20552.5">
<Uri>https://github.com/dotnet/arcade</Uri>
<Sha>51f14c7da74e5c3d2ce0bab077461da399333896</Sha>
<Sha>72b28b7e023d4c3fffa0a0b9748a7d4e8cc799be</Sha>
</Dependency>
</ToolsetDependencies>
</Dependencies>
76 changes: 52 additions & 24 deletions eng/common/post-build/symbols-validation.ps1
Original file line number Diff line number Diff line change
Expand Up @@ -9,9 +9,16 @@ param(
# Maximum number of jobs to run in parallel
$MaxParallelJobs = 6

# Max number of retries
$MaxRetry = 5

# Wait time between check for system load
$SecondsBetweenLoadChecks = 10

# Set error codes
Set-Variable -Name "ERROR_BADEXTRACT" -Option Constant -Value -1
Set-Variable -Name "ERROR_FILEDOESNOTEXIST" -Option Constant -Value -2

$CountMissingSymbols = {
param(
[string] $PackagePath # Path to a NuGet package
Expand All @@ -21,10 +28,15 @@ $CountMissingSymbols = {

Add-Type -AssemblyName System.IO.Compression.FileSystem

Write-Host "Validating $PackagePath "

# Ensure input file exist
if (!(Test-Path $PackagePath)) {
Write-PipelineTaskError "Input file does not exist: $PackagePath"
return -2
return [pscustomobject]@{
result = $using:ERROR_FILEDOESNOTEXIST
packagePath = $PackagePath
}
}

# Extensions for which we'll look for symbols
Expand All @@ -45,7 +57,7 @@ $CountMissingSymbols = {
Write-Host "Something went wrong extracting $PackagePath"
Write-Host $_
return [pscustomobject]@{
result = -1
result = $using:ERROR_BADEXTRACT
packagePath = $PackagePath
}
}
Expand Down Expand Up @@ -91,26 +103,37 @@ $CountMissingSymbols = {
$dotnetSymbolExe = "$env:USERPROFILE\.dotnet\tools"
$dotnetSymbolExe = Resolve-Path "$dotnetSymbolExe\dotnet-symbol.exe"

& $dotnetSymbolExe --symbols --modules --windows-pdbs $TargetServerParam $FullPath -o $SymbolsPath | Out-Null
$totalRetries = 0

if (Test-Path $PdbPath) {
return 'PDB'
}
elseif (Test-Path $NGenPdb) {
return 'NGen PDB'
}
elseif (Test-Path $SODbg) {
return 'DBG for SO'
}
elseif (Test-Path $DylibDwarf) {
return 'Dwarf for Dylib'
}
elseif (Test-Path $SymbolPath) {
return 'Module'
}
else {
return $null
while ($totalRetries -lt $using:MaxRetry) {
# Save the output and get diagnostic output
$output = & $dotnetSymbolExe --symbols --modules --windows-pdbs $TargetServerParam $FullPath -o $SymbolsPath --diagnostics | Out-String

if (Test-Path $PdbPath) {
return 'PDB'
}
elseif (Test-Path $NGenPdb) {
return 'NGen PDB'
}
elseif (Test-Path $SODbg) {
return 'DBG for SO'
}
elseif (Test-Path $DylibDwarf) {
return 'Dwarf for Dylib'
}
elseif (Test-Path $SymbolPath) {
return 'Module'
}
elseif ($output.Contains("503 Service Unavailable")) {
# If we got a 503 error, we should retry.
$totalRetries++
}
else {
return $null
}
}

return $null
}

$SymbolsOnMSDL = & $FirstMatchingSymbolDescriptionOrDefault $FileName '--microsoft-symbol-server' $SymbolsPath
Expand Down Expand Up @@ -155,14 +178,21 @@ function CheckJobResult(
$packagePath,
[ref]$DupedSymbols,
[ref]$TotalFailures) {
if ($result -eq '-1') {
if ($result -eq $ERROR_BADEXTRACT) {
Write-PipelineTelemetryError -Category 'CheckSymbols' -Message "$packagePath has duplicated symbol files"
$DupedSymbols.Value++
}
elseif ($jobResult.result -ne '0') {
elseif ($result -eq $ERROR_FILEDOESNOTEXIST) {
Write-PipelineTelemetryError -Category 'CheckSymbols' -Message "$packagePath does not exist"
$TotalFailures.Value++
}
elseif ($result -gt '0') {
Write-PipelineTelemetryError -Category 'CheckSymbols' -Message "Missing symbols for $result modules in the package $packagePath"
$TotalFailures.Value++
}
else {
Write-Host "All symbols verified for package $packagePath"
}
}

function CheckSymbolsAvailable {
Expand Down Expand Up @@ -192,8 +222,6 @@ function CheckSymbolsAvailable {
return
}

Write-Host "Validating $FileName "

Start-Job -ScriptBlock $CountMissingSymbols -ArgumentList $FullName | Out-Null

$NumJobs = @(Get-Job -State 'Running').Count
Expand Down
2 changes: 1 addition & 1 deletion global.json
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
}
},
"msbuild-sdks": {
"Microsoft.DotNet.Arcade.Sdk": "6.0.0-beta.20527.10",
"Microsoft.DotNet.Arcade.Sdk": "6.0.0-beta.20552.5",
"Microsoft.DotNet.Helix.Sdk": "2.0.0-beta.19069.2"
}
}

0 comments on commit 6ebb769

Please sign in to comment.