Skip to content

[main] Update dependencies from dotnet/runtime#49075

Merged
Tratcher merged 3 commits intomainfrom
darc-main-486f9ac2-8f16-4224-9ecb-8a01883d73c3
Jun 29, 2023
Merged

[main] Update dependencies from dotnet/runtime#49075
Tratcher merged 3 commits intomainfrom
darc-main-486f9ac2-8f16-4224-9ecb-8a01883d73c3

Conversation

@dotnet-maestro
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Jun 28, 2023 •
edited
Loading

This pull request updates the following dependencies

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

  • Subscription: 32db3699-5666-45da-a1b7-08d8b804cd75
  • Build: 20230628.11
  • Date Produced: June 29, 2023 9:21:40 AM UTC
  • Commit: c88b3776d0fb3bb33fcd4192b476147c0895bd6f
  • Branch: refs/heads/main

@dotnet-maestro
Update dependencies from https://github.com/dotnet/runtime build 2023…
4f8e324 
…0628.1

Microsoft.Bcl.AsyncInterfaces , Microsoft.Bcl.TimeProvider , Microsoft.Extensions.Caching.Abstractions , Microsoft.Extensions.Caching.Memory , Microsoft.Extensions.Configuration , Microsoft.Extensions.Configuration.Abstractions , Microsoft.Extensions.Configuration.Binder , Microsoft.Extensions.Configuration.CommandLine , Microsoft.Extensions.Configuration.EnvironmentVariables , Microsoft.Extensions.Configuration.FileExtensions , Microsoft.Extensions.Configuration.Ini , Microsoft.Extensions.Configuration.Json , Microsoft.Extensions.Configuration.UserSecrets , Microsoft.Extensions.Configuration.Xml , Microsoft.Extensions.DependencyInjection , Microsoft.Extensions.DependencyInjection.Abstractions , Microsoft.Extensions.DependencyModel , Microsoft.Extensions.Diagnostics , Microsoft.Extensions.Diagnostics.Abstractions , Microsoft.Extensions.FileProviders.Abstractions , Microsoft.Extensions.FileProviders.Composite , Microsoft.Extensions.FileProviders.Physical , Microsoft.Extensions.FileSystemGlobbing , Microsoft.Extensions.HostFactoryResolver.Sources , Microsoft.Extensions.Hosting , Microsoft.Extensions.Hosting.Abstractions , Microsoft.Extensions.Http , Microsoft.Extensions.Logging , Microsoft.Extensions.Logging.Abstractions , Microsoft.Extensions.Logging.Configuration , Microsoft.Extensions.Logging.Console , Microsoft.Extensions.Logging.Debug , Microsoft.Extensions.Logging.EventLog , Microsoft.Extensions.Logging.EventSource , Microsoft.Extensions.Logging.TraceSource , Microsoft.Extensions.Options , Microsoft.Extensions.Options.ConfigurationExtensions , Microsoft.Extensions.Options.DataAnnotations , Microsoft.Extensions.Primitives , Microsoft.Internal.Runtime.AspNetCore.Transport , Microsoft.NET.Runtime.MonoAOTCompiler.Task , Microsoft.NET.Runtime.WebAssembly.Sdk , Microsoft.NETCore.App.Ref , Microsoft.NETCore.App.Runtime.AOT.win-x64.Cross.browser-wasm , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.BrowserDebugHost.Transport , Microsoft.NETCore.Platforms , System.Collections.Immutable , System.Configuration.ConfigurationManager , System.Diagnostics.DiagnosticSource , System.Diagnostics.EventLog , System.Diagnostics.PerformanceCounter , System.DirectoryServices.Protocols , System.IO.Hashing , System.IO.Pipelines , System.Net.Http.Json , System.Net.Http.WinHttpHandler , System.Reflection.Metadata , System.Resources.Extensions , System.Runtime.Caching , System.Security.Cryptography.Pkcs , System.Security.Cryptography.Xml , System.Security.Permissions , System.ServiceProcess.ServiceController , System.Text.Encodings.Web , System.Text.Json , System.Threading.AccessControl , System.Threading.Channels , System.Threading.RateLimiting
 From Version 8.0.0-preview.7.23326.5 -> To Version 8.0.0-preview.7.23328.1
@dotnet-maestro dotnet-maestro bot requested review from a team and wtgodbe as code owners June 28, 2023 12:02
@ghost ghost added area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework Type: Dependency Update 🔼 labels Jun 28, 2023
ghost
ghost approved these changes Jun 28, 2023
View reviewed changes
Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Auto-approving dependency update.

@ghost
Copy link

ghost commented Jun 28, 2023

Hey @dotnet/aspnet-build, looks like this PR is something you want to take a look at.

@surayya-MS
Copy link
Member

surayya-MS commented Jun 28, 2023

Microsoft.AspNetCore.Server.Kestrel.Core.Tests.SniOptionsSelectorTests.CloneSslOptionsClonesAllProperties fails:

Assert.Empty() Failure
Expected: <empty>
Actual:   ["AllowTlsResume"]

dotnet/runtime#86047 could be related.

@wfurt, @stephentoub could you please look into this?

@stephentoub
Copy link
Member

stephentoub commented Jun 28, 2023 •
edited
Loading

Microsoft.AspNetCore.Server.Kestrel.Core.Tests.SniOptionsSelectorTests.CloneSslOptionsClonesAllProperties fails:

Assert.Empty() Failure
Expected: <empty>
Actual:   ["AllowTlsResume"]

dotnet/runtime#86047 could be related.

@wfurt, @stephentoub could you please look into this?

The type now has an additional property. The test is written expecting it doesn't:

aspnetcore/src/Servers/Kestrel/Core/test/SniOptionsSelectorTests.cs

Line 904 in f0be130

// Ensure we've checked every property. When new properties get added, we'll have to update this test along with the CloneSslOptions implementation.

The test needs to be updated (and, according to the comment, maybe something in the ASP.NET implementation as well)

@surayya-MS
Copy link
Member

surayya-MS commented Jun 28, 2023

cc @adityamandaleeka

@dotnet-maestro
Update dependencies from https://github.com/dotnet/runtime build 2023…
e7379f8 
…0628.11

Microsoft.Bcl.AsyncInterfaces , Microsoft.Bcl.TimeProvider , Microsoft.Extensions.Caching.Abstractions , Microsoft.Extensions.Caching.Memory , Microsoft.Extensions.Configuration , Microsoft.Extensions.Configuration.Abstractions , Microsoft.Extensions.Configuration.Binder , Microsoft.Extensions.Configuration.CommandLine , Microsoft.Extensions.Configuration.EnvironmentVariables , Microsoft.Extensions.Configuration.FileExtensions , Microsoft.Extensions.Configuration.Ini , Microsoft.Extensions.Configuration.Json , Microsoft.Extensions.Configuration.UserSecrets , Microsoft.Extensions.Configuration.Xml , Microsoft.Extensions.DependencyInjection , Microsoft.Extensions.DependencyInjection.Abstractions , Microsoft.Extensions.DependencyModel , Microsoft.Extensions.Diagnostics , Microsoft.Extensions.Diagnostics.Abstractions , Microsoft.Extensions.FileProviders.Abstractions , Microsoft.Extensions.FileProviders.Composite , Microsoft.Extensions.FileProviders.Physical , Microsoft.Extensions.FileSystemGlobbing , Microsoft.Extensions.HostFactoryResolver.Sources , Microsoft.Extensions.Hosting , Microsoft.Extensions.Hosting.Abstractions , Microsoft.Extensions.Http , Microsoft.Extensions.Logging , Microsoft.Extensions.Logging.Abstractions , Microsoft.Extensions.Logging.Configuration , Microsoft.Extensions.Logging.Console , Microsoft.Extensions.Logging.Debug , Microsoft.Extensions.Logging.EventLog , Microsoft.Extensions.Logging.EventSource , Microsoft.Extensions.Logging.TraceSource , Microsoft.Extensions.Options , Microsoft.Extensions.Options.ConfigurationExtensions , Microsoft.Extensions.Options.DataAnnotations , Microsoft.Extensions.Primitives , Microsoft.Internal.Runtime.AspNetCore.Transport , Microsoft.NET.Runtime.MonoAOTCompiler.Task , Microsoft.NET.Runtime.WebAssembly.Sdk , Microsoft.NETCore.App.Ref , Microsoft.NETCore.App.Runtime.AOT.win-x64.Cross.browser-wasm , Microsoft.NETCore.App.Runtime.win-x64 , Microsoft.NETCore.BrowserDebugHost.Transport , Microsoft.NETCore.Platforms , System.Collections.Immutable , System.Configuration.ConfigurationManager , System.Diagnostics.DiagnosticSource , System.Diagnostics.EventLog , System.Diagnostics.PerformanceCounter , System.DirectoryServices.Protocols , System.IO.Hashing , System.IO.Pipelines , System.Net.Http.Json , System.Net.Http.WinHttpHandler , System.Reflection.Metadata , System.Resources.Extensions , System.Runtime.Caching , System.Security.Cryptography.Pkcs , System.Security.Cryptography.Xml , System.Security.Permissions , System.ServiceProcess.ServiceController , System.Text.Encodings.Web , System.Text.Json , System.Threading.AccessControl , System.Threading.Channels , System.Threading.RateLimiting
 From Version 8.0.0-preview.7.23326.5 -> To Version 8.0.0-preview.7.23328.11
@Tratcher Tratcher self-assigned this Jun 29, 2023
@Tratcher
Copy link
Member

Tratcher commented Jun 29, 2023

Fixed

@Tratcher Tratcher enabled auto-merge (squash) June 29, 2023 18:06
@Tratcher
Copy link
Member

Tratcher commented Jun 29, 2023

/azp run

@azure-pipelines
Copy link

azure-pipelines bot commented Jun 29, 2023

Azure Pipelines successfully started running 3 pipeline(s).

@Tratcher Tratcher merged commit d07f4d4 into main Jun 29, 2023
@Tratcher Tratcher deleted the darc-main-486f9ac2-8f16-4224-9ecb-8a01883d73c3 branch June 29, 2023 21:50
@ghost ghost added this to the 8.0-preview7 milestone Jun 29, 2023
@javiercn
Copy link
Member

javiercn commented Jun 30, 2023 •
edited
Loading

@Tratcher this update broke the build. Why did we merge on red?

javiercn added a commit that referenced this pull request Jun 30, 2023
@javiercn
Revert "[main] Update dependencies from dotnet/runtime (#49075)"
2deb0c0 
This reverts commit d07f4d4.
@javiercn javiercn mentioned this pull request Jun 30, 2023
Closed
@Tratcher
Copy link
Member

Tratcher commented Jun 30, 2023

It was on auto-merge, the check must not be required, and there's nothing preventing the next build from resubmitting the same change.

@javiercn
Copy link
Member

javiercn commented Jun 30, 2023

@wtgodbe can you take a look at what @Tratcher is suggesting and switching the check to be required if its currently not?

@Tratcher Tratcher mentioned this pull request Jun 30, 2023
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wtgodbe wtgodbe Awaiting requested review from wtgodbe wtgodbe is a code owner

@Tratcher Tratcher Awaiting requested review from Tratcher

@halter73 halter73 Awaiting requested review from halter73 halter73 is a code owner

@BrennanConroy BrennanConroy Awaiting requested review from BrennanConroy BrennanConroy is a code owner

@JamesNK JamesNK Awaiting requested review from JamesNK

@mgravell mgravell Awaiting requested review from mgravell

1 more reviewer
Reviewers whose approvals may not affect merge requirements

Assignees

@Tratcher Tratcher

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework Type: Dependency Update 🔼

Projects

None yet

Milestone

8.0-preview7

Development

Successfully merging this pull request may close these issues.

4 participants

@surayya-MS @stephentoub @Tratcher @javiercn