-
Notifications
You must be signed in to change notification settings - Fork 762
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/aspnetcore #4262
[main] Update dependencies from dotnet/aspnetcore #4262
Conversation
Interesting so looks like the Http Resilience tests are failing with this update with the following callstack:
@noahfalk @MihaZupan @antonfirsov @tarekgh Was this method removed recently? Does this ring any bells? FYI @martintmk since the failing tests are in http resilience. |
mmm that is very weird actually, I do see the method is still there: |
Don't think we changed anything around this property since it was introduced, it definitely wasn't removed. cc: @CarnaViire in case it's related to the client factory |
mm seems like it is just a mismatch from the runtime we use. I'm trying to update the SDK used by the repo and I believe that should fix this issue. |
Yup, looks like the issue was with the SDK as bumping it fixed the tests for me at least locally. Pushing the change to see if this passes CI now. Sorry for the randomization folks. |
In general, we should be at least at the same version as dotnet/aspnetcore. |
…0230809.22 Microsoft.AspNetCore.App.Runtime.win-x64 , Microsoft.AspNetCore.Mvc.Testing , Microsoft.AspNetCore.TestHost , Microsoft.Extensions.Caching.StackExchangeRedis , Microsoft.Extensions.Diagnostics.HealthChecks , Microsoft.Extensions.Diagnostics.HealthChecks.Abstractions , Microsoft.Extensions.Features , Microsoft.Extensions.Http.Polly , Microsoft.Extensions.ObjectPool From Version 8.0.0-rc.1.23409.1 -> To Version 8.0.0-rc.1.23409.22 Dependency coherency updates Microsoft.Bcl.TimeProvider,Microsoft.Extensions.Caching.Abstractions,Microsoft.Extensions.Caching.Memory,Microsoft.Extensions.Configuration.Abstractions,Microsoft.Extensions.Configuration.Binder,Microsoft.Extensions.Configuration.CommandLine,Microsoft.Extensions.Configuration.Json,Microsoft.Extensions.Configuration,Microsoft.Extensions.DependencyInjection.Abstractions,Microsoft.Extensions.DependencyInjection,Microsoft.Extensions.Hosting.Abstractions,Microsoft.Extensions.Hosting,Microsoft.Extensions.Http,Microsoft.Extensions.Logging.Abstractions,Microsoft.Extensions.Logging.Configuration,Microsoft.Extensions.Logging.Console,Microsoft.Extensions.Logging,Microsoft.Extensions.Options.ConfigurationExtensions,Microsoft.Extensions.Options.DataAnnotations,Microsoft.Extensions.Options,Microsoft.Extensions.Primitives,System.Collections.Immutable,System.Configuration.ConfigurationManager,System.Diagnostics.DiagnosticSource,System.Diagnostics.PerformanceCounter,System.IO.Hashing,System.Net.Http.Json,System.Security.Cryptography.Pkcs,System.Security.Cryptography.Xml,System.Text.Encodings.Web,System.Text.Json,System.Runtime.Caching From Version 8.0.0-rc.1.23404.1 -> To Version 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64
439d52c
to
c5c4bea
Compare
This pull request updates the following dependencies
Coherency Updates
The following updates ensure that dependencies with a CoherentParentDependency
attribute were produced in a build used as input to the parent dependency's build.
See Dependency Description Format
From https://github.com/dotnet/aspnetcore
Microsoft Reviewers: Open in CodeFlow