Darc incorrectly constructing download url for internal manifest symbols package
See original GitHub issue- This issue is blocking
- This issue is causing unreasonable pain
This is an internal build, so the symbol package was uploaded to a nuget feed. The format of the package name appears to be confusing darc’s gather-drop, because it’s incorrectly constructing the package download URL:
It attempts https://pkgs.dev.azure.com/dnceng/internal/_apis/packaging/feeds/dotnet6-internal-symbols/nuget/packages/Microsoft.NET.Workload.Mono.ToolChain.Manifest-6.0.100.Msi.x64.6.0.0-preview.7.21377.20/versions/6.0.100/content
. This appears to mean that it thinks the version number is 6.0.100
, rather than 6.0.0-preview.7.21377.20
My hunch is that the - in the middle of the package name is confusing things. It’s unclear whether the error is on the arcade-services side or the version identification side. BAR appears to understand the version number correctly, according to the darc get-asset output, but it’s unclear why it would choose 6.0.100 later.
PS C:\r\arcade> darc get-asset --name "assets/symbols/Microsoft.NET.Workload.Mono.ToolChain.Manifest-6.0.100.6.0.0-preview.7.21377.20.symbols.nupkg"
Looking up assets with name 'assets/symbols/Microsoft.NET.Workload.Mono.ToolChain.Manifest-6.0.100.6.0.0-preview.7.21377.20.symbols.nupkg' in the last 30 days
assets/symbols/Microsoft.NET.Workload.Mono.ToolChain.Manifest-6.0.100.6.0.0-preview.7.21377.20.symbols.nupkg @ 6.0.0-preview.7.21377.20
Repository: https://dev.azure.com/dnceng/internal/_git/dotnet-runtime
Branch: refs/heads/internal/release/6.0-preview7
Commit: 01019e12c1292fc51496c2c3b52fb5141d0f0602
Build Number: 20210727.20
Date Produced: 7/27/2021 9:03 PM
Build Link: https://dev.azure.com/dnceng/internal/_build/results?buildId=1262709
BAR Build Id: 99704
Released: False
Channels:
- .NET 6 Preview 7 Internal
Locations:
- https://dev.azure.com/dnceng/internal/_apis/build/builds/1262709/artifacts (Container)
- https://pkgs.dev.azure.com/dnceng/internal/_packaging/dotnet6-internal-symbols/nuget/v3/index.json (Container)
Issue Analytics
- State:
- Created 2 years ago
- Comments:18 (18 by maintainers)
Top GitHub Comments
The sdk band version has to be in the package name, since there will eventually be multiple of them. I think we either need to change the version identifier (with a new test of course, since this stuff can be fragile) or change the way the package name is constructed so that it still contains 6.0.100, but perhaps with different delimiters. That may require reaction on the VS or installer side.
/cc @joeloff
Ok, I think I have a change that might fix the version parser. I will put up a PR and we can discuss.