Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[main] Update dependencies from dotnet/runtime #22433

Merged
merged 12 commits into from
Nov 11, 2021

Conversation

dotnet-maestro[bot]
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Nov 2, 2021

This pull request updates the following dependencies

From https://github.com/dotnet/runtime

  • Subscription: aa69f164-2492-460a-3914-08d8e9750bf8
  • Build: 20211111.1
  • Date Produced: November 11, 2021 11:38:03 AM UTC
  • Commit: 6f5de0b2b979a70e6fe36904d0d4f087c32f9c7e
  • Branch: refs/heads/main

…1102.2

Microsoft.NETCore.Platforms , Microsoft.NETCore.DotNetHostResolver , Microsoft.NETCore.App.Runtime.win-x64 , System.CodeDom , System.Security.Cryptography.ProtectedData , System.Resources.Extensions , System.Reflection.MetadataLoadContext , Microsoft.NET.HostModel , Microsoft.Extensions.DependencyModel , Microsoft.NETCore.App.Host.win-x64 , Microsoft.NETCore.App.Ref , System.Text.Encoding.CodePages , VS.Redist.Common.NetCore.SharedFramework.x64.7.0 , VS.Redist.Common.NetCore.TargetingPack.x64.7.0
 From Version 7.0.0-alpha.1.21551.1 -> To Version 7.0.0-alpha.1.21552.2
@dotnet-maestro
Copy link
Contributor Author

dotnet-maestro bot commented Nov 2, 2021

Notification for subscribed users from https://github.com/dotnet/runtime:

@dnr-codeflow

Action requested: Please take a look at this failing automated dependency-flow pull request's checks; failures may be related to changes which originated in your repo.

  • This pull request contains changes from your source repo (https://github.com/dotnet/runtime) and seems to have failed checks in this PR. Please take a peek at the failures and comment if they seem relevant to your changes.
  • If you're being tagged in this comment it is due to an entry in the related Maestro Subscription of the Build Asset Registry. If you feel this entry has added your GitHub login or your GitHub team in error, please update the subscription to reflect this.
  • For more details, please read the Arcade Darc documentation

…1102.11

Microsoft.NETCore.Platforms , Microsoft.NETCore.DotNetHostResolver , Microsoft.NETCore.App.Runtime.win-x64 , System.CodeDom , System.Security.Cryptography.ProtectedData , System.Resources.Extensions , System.Reflection.MetadataLoadContext , Microsoft.NET.HostModel , Microsoft.Extensions.DependencyModel , Microsoft.NETCore.App.Host.win-x64 , Microsoft.NETCore.App.Ref , System.Text.Encoding.CodePages , VS.Redist.Common.NetCore.SharedFramework.x64.7.0 , VS.Redist.Common.NetCore.TargetingPack.x64.7.0
 From Version 7.0.0-alpha.1.21551.1 -> To Version 7.0.0-alpha.1.21552.11
dotnet-maestro bot and others added 5 commits November 4, 2021 12:20
…1103.11

Microsoft.NETCore.Platforms , Microsoft.NETCore.DotNetHostResolver , Microsoft.NETCore.App.Runtime.win-x64 , System.CodeDom , System.Security.Cryptography.ProtectedData , System.Resources.Extensions , System.Reflection.MetadataLoadContext , Microsoft.NET.HostModel , Microsoft.Extensions.DependencyModel , Microsoft.NETCore.App.Host.win-x64 , Microsoft.NETCore.App.Ref , System.Text.Encoding.CodePages , VS.Redist.Common.NetCore.SharedFramework.x64.7.0 , VS.Redist.Common.NetCore.TargetingPack.x64.7.0
 From Version 7.0.0-alpha.1.21551.1 -> To Version 7.0.0-alpha.1.21553.11
…1104.6

Microsoft.NETCore.Platforms , Microsoft.NETCore.DotNetHostResolver , Microsoft.NETCore.App.Runtime.win-x64 , System.CodeDom , System.Security.Cryptography.ProtectedData , System.Resources.Extensions , System.Reflection.MetadataLoadContext , Microsoft.NET.HostModel , Microsoft.Extensions.DependencyModel , Microsoft.NETCore.App.Host.win-x64 , Microsoft.NETCore.App.Ref , System.Text.Encoding.CodePages , VS.Redist.Common.NetCore.SharedFramework.x64.7.0 , VS.Redist.Common.NetCore.TargetingPack.x64.7.0
 From Version 7.0.0-alpha.1.21551.1 -> To Version 7.0.0-alpha.1.21554.6
…1105.7

Microsoft.NETCore.Platforms , Microsoft.NETCore.DotNetHostResolver , Microsoft.NETCore.App.Runtime.win-x64 , System.CodeDom , System.Security.Cryptography.ProtectedData , System.Resources.Extensions , System.Reflection.MetadataLoadContext , Microsoft.NET.HostModel , Microsoft.Extensions.DependencyModel , Microsoft.NETCore.App.Host.win-x64 , Microsoft.NETCore.App.Ref , System.Text.Encoding.CodePages , VS.Redist.Common.NetCore.SharedFramework.x64.7.0 , VS.Redist.Common.NetCore.TargetingPack.x64.7.0
 From Version 7.0.0-alpha.1.21551.1 -> To Version 7.0.0-alpha.1.21555.7
…1106.5

Microsoft.NETCore.Platforms , Microsoft.NETCore.DotNetHostResolver , Microsoft.NETCore.App.Runtime.win-x64 , System.CodeDom , System.Security.Cryptography.ProtectedData , System.Resources.Extensions , System.Reflection.MetadataLoadContext , Microsoft.NET.HostModel , Microsoft.Extensions.DependencyModel , Microsoft.NETCore.App.Host.win-x64 , Microsoft.NETCore.App.Ref , System.Text.Encoding.CodePages , VS.Redist.Common.NetCore.SharedFramework.x64.7.0 , VS.Redist.Common.NetCore.TargetingPack.x64.7.0
 From Version 7.0.0-alpha.1.21551.1 -> To Version 7.0.0-alpha.1.21556.5
@Tratcher
Copy link
Member

Tratcher commented Nov 8, 2021

Several Cryptography dlls have been merged and the publishing tests need to be updated. dotnet/runtime@df5b118

Microsoft.NET.Sdk.BlazorWebAssembly.Tests.WasmJsModulesIntegrationTests.Publish_DoesNotGenerateManifestJson_IncludesJSModulesOnBlazorBootJsonManifest

@javiercn

@javiercn
Copy link
Member

javiercn commented Nov 8, 2021

@Tratcher I believe @captainsafia was looking into it.

@captainsafia if that's not the case, let me know and I'll deal with it.

@captainsafia
Copy link
Member

@captainsafia if that's not the case, let me know and I'll deal with it.

I updated all of the Razor baselines in cb7d2f6 but missed the Blazor ones which are causing the issue.

@Tratcher If you want to take a stab at fixing it you'll need to change the below to true and rerun the Blazor test project.

public static bool GenerateBaselines = false;

Tratcher and others added 2 commits November 8, 2021 14:02
…1108.10

Microsoft.NETCore.Platforms , Microsoft.NETCore.DotNetHostResolver , Microsoft.NETCore.App.Runtime.win-x64 , System.CodeDom , System.Security.Cryptography.ProtectedData , System.Resources.Extensions , System.Reflection.MetadataLoadContext , Microsoft.NET.HostModel , Microsoft.Extensions.DependencyModel , Microsoft.NETCore.App.Host.win-x64 , Microsoft.NETCore.App.Ref , System.Text.Encoding.CodePages , VS.Redist.Common.NetCore.SharedFramework.x64.7.0 , VS.Redist.Common.NetCore.TargetingPack.x64.7.0
 From Version 7.0.0-alpha.1.21551.1 -> To Version 7.0.0-alpha.1.21558.10
@Tratcher
Copy link
Member

Tratcher commented Nov 9, 2021

@dougbu I fixed the blazor baselines, but there are a lot of other failures in this update that need investigation. I can help, but I don't see any so far that I understand.

@dougbu
Copy link
Member

dougbu commented Nov 9, 2021

I can help, but I don't see any so far that I understand.

Do those problems look like more breaks from dotnet/aspnetcore❔ Either way, please call on the people who may have more context.

@dougbu
Copy link
Member

dougbu commented Nov 9, 2021

Not sure about the inability to load an assembly but the file-locking issues are likely build flakiness. I reran failed jobs with 🤞

Other thoughts 💭 looking at first attempt of current build @dsplaisted @sfoslund @marcpopMSFT

@TanayParikh
Copy link
Contributor

Do those problems look like more breaks from dotnet/aspnetcore❔

Took a look through the failing build. Looks to mainly be infra / package ref failures.

https://dev.azure.com/dnceng/public/_build/results?buildId=1460809&view=ms.vss-test-web.build-test-results-tab&runId=41987324&paneView=debug&resultId=100061

Either way, please call on the people who may have more context.

Who would have the appropriate context in this area / repo?

/ping @dsplaisted @sfoslund @marcpopMSFT

This is a higher priority fix as it's holding up other ingestions as well ex. dotnet/installer#12589.

@TanayParikh
Copy link
Contributor

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@dsplaisted
Copy link
Member

A lot of failures were in GivenThatWeWantToUseAnalyzers.It_resolves_analyzers_correctly, and were failing because a new analyzer was added in dotnet/runtime#61329. I've pushed c5bd2f5 to handle this (and better cover analyzers coming from targeting packs).

FYI @joperezr @safern @ericstj

Also FYI @joeloff as domestic cat

@TanayParikh
Copy link
Contributor

Thanks @dsplaisted, that definitely helped.

Still seeing package related failures like the following (not limited to just these packages):

C:\h\w\94840842\t\dotnetSdkTests\ohqkbyxf.lyy\It_resolves_a---950EE480\NETFrameworkLibrary\NETFrameworkLibrary.csproj : warning NU3003: Package 'System.Runtime.Numerics 4.3.0' from source 'https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet-public/nuget/v3/index.json': The package signature is invalid or cannot be verified on this platform.
C:\h\w\94840842\t\dotnetSdkTests\rl3aupts.yxk\ChangeCompile---B0CCC827\WatchGlobbingApp.csproj : error NU3003: Package 'Microsoft.AspNetCore.App.Ref 5.0.0' from source 'https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet-public/nuget/v3/index.json': The package signature is invalid or cannot be verified on this platform.

Test run here: https://dev.azure.com/dnceng/public/_build/results?buildId=1462556&view=ms.vss-test-web.build-test-results-tab&runId=42020158&resultId=100185&paneView=debug

Does anyone have context on these package sources? The package signature is invalid or cannot be verified on this platform. seems to be an AZDO issue potentially?

@joeloff
Copy link
Member

joeloff commented Nov 10, 2021

I'm looking into this.

@joeloff
Copy link
Member

joeloff commented Nov 10, 2021

the signature is valid, could be a problem with the agents

image

dotnet-maestro bot and others added 2 commits November 11, 2021 13:12
…1111.1

Microsoft.NETCore.Platforms , Microsoft.NETCore.DotNetHostResolver , Microsoft.NETCore.App.Runtime.win-x64 , System.CodeDom , System.Security.Cryptography.ProtectedData , System.Resources.Extensions , System.Reflection.MetadataLoadContext , Microsoft.NET.HostModel , Microsoft.Extensions.DependencyModel , Microsoft.NETCore.App.Host.win-x64 , Microsoft.NETCore.App.Ref , System.Text.Encoding.CodePages , VS.Redist.Common.NetCore.SharedFramework.x64.7.0 , VS.Redist.Common.NetCore.TargetingPack.x64.7.0
 From Version 7.0.0-alpha.1.21551.1 -> To Version 7.0.0-alpha.1.21561.1
@dotnet-maestro dotnet-maestro bot merged commit de390be into main Nov 11, 2021
@dotnet-maestro dotnet-maestro bot deleted the darc-main-7ffb3e29-bf6f-430a-a354-fb89cd8385b0 branch November 11, 2021 20:41
@TanayParikh
Copy link
Contributor

Awesome, thanks everyone! 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants