Skip to content

Conversation

@dotnet-maestro
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Aug 5, 2022

This pull request updates the following dependencies

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

  • Subscription: aa69f164-2492-460a-3914-08d8e9750bf8
  • Build: 20220806.5
  • Date Produced: August 7, 2022 5:08:16 AM UTC
  • Commit: e43e31527db9f1a2dde3095a1651279b272c8c09
  • Branch: refs/heads/main

…0805.1

Microsoft.DotNet.ILCompiler , Microsoft.Extensions.DependencyModel , Microsoft.NET.HostModel , Microsoft.NETCore.App.Host.win-x64 , Microsoft.NETCore.App.Ref , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.DotNetHostResolver , Microsoft.NETCore.Platforms , System.CodeDom , System.Reflection.MetadataLoadContext , System.Resources.Extensions , System.Security.Cryptography.ProtectedData , 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-rc.1.22403.8 -> To Version 7.0.0-rc.1.22405.1
@ghost ghost added the Area-CodeFlow label Aug 5, 2022
@dotnet-maestro
Copy link
Contributor Author

dotnet-maestro bot commented Aug 5, 2022

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

@jeffschwMSFT
Copy link
Member

All the failures have this failure signature.

Microsoft.NET.Sdk.BlazorWebAssembly.Tests.BlazorWasmStaticWebAssetsIntegrationTest.StaticWebAssets_BuildMinimal_Works [FAIL]
  Expected subject to be a collection with 408 item(s), but found 410.

@steveisok can you take a look?

@steveisok
Copy link
Member

All the failures have this failure signature.

Microsoft.NET.Sdk.BlazorWebAssembly.Tests.BlazorWasmStaticWebAssetsIntegrationTest.StaticWebAssets_BuildMinimal_Works [FAIL]
  Expected subject to be a collection with 408 item(s), but found 410.

@lewing @radical Any ideas?

@kasperk81
Copy link
Contributor

to ping, names should be outside the code block
@lewing @radical

@radical
Copy link
Member

radical commented Aug 6, 2022

this is likely about changed set of deployed files. Cc @pavelsavara

…0805.9

Microsoft.DotNet.ILCompiler , Microsoft.Extensions.DependencyModel , Microsoft.NET.HostModel , Microsoft.NETCore.App.Host.win-x64 , Microsoft.NETCore.App.Ref , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.DotNetHostResolver , Microsoft.NETCore.Platforms , System.CodeDom , System.Reflection.MetadataLoadContext , System.Resources.Extensions , System.Security.Cryptography.ProtectedData , 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-rc.1.22403.8 -> To Version 7.0.0-rc.1.22405.9
@pavelsavara
Copy link
Member

possibly new dotnet-legacy.d.ts

@pavelsavara
Copy link
Member

We have related pending PR here https://github.com/dotnet/sdk/pull/26966/files#diff-576ff8fae44da14d0ffe3820de926dbe0f8df13c8881dc5fd98bd851101c4180R305 which is filtering dotnet-legacy.d.ts from wwwroot

@kasperk81
Copy link
Contributor

two extra static files shouldn't be a failure or at least to save rountrips every other day, these "expected" lists should come from runtime where the change can make or break this test.

to unblock the rest of us mortals, maybe push a commit here to account the additional static files it is seeing or disable this test until that pr lands?

@radical
Copy link
Member

radical commented Aug 6, 2022

We are working on some changes, so blazor gets the list of files from the runtime.

@pavelsavara
Copy link
Member

@javiercn would merging #26966 help to unblock this ? I'm not sure

…0806.5

Microsoft.DotNet.ILCompiler , Microsoft.Extensions.DependencyModel , Microsoft.NET.HostModel , Microsoft.NETCore.App.Host.win-x64 , Microsoft.NETCore.App.Ref , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.DotNetHostResolver , Microsoft.NETCore.Platforms , System.CodeDom , System.Reflection.MetadataLoadContext , System.Resources.Extensions , System.Security.Cryptography.ProtectedData , 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-rc.1.22403.8 -> To Version 7.0.0-rc.1.22406.5
@dotnet-maestro dotnet-maestro bot merged commit 7ec61a2 into main Aug 7, 2022
@dotnet-maestro dotnet-maestro bot deleted the darc-main-3c74eb5f-0a2f-445a-85bf-cc20edb7896a branch August 7, 2022 13:50
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.

6 participants