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/aspnetcore #4262

Merged
merged 5 commits into from
Aug 11, 2023

Conversation

dotnet-maestro[bot]
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Aug 10, 2023

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

  • Coherency Updates:
    • Microsoft.Bcl.TimeProvider: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.Caching.Abstractions: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.Caching.Memory: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.Configuration.Abstractions: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.Configuration.Binder: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.Configuration.CommandLine: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.Configuration.Json: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.Configuration: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.DependencyInjection.Abstractions: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.DependencyInjection: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.Hosting.Abstractions: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.Hosting: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.Http: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.Logging.Abstractions: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.Logging.Configuration: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.Logging.Console: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.Logging: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.Options.ConfigurationExtensions: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.Options.DataAnnotations: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.Options: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • Microsoft.Extensions.Primitives: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • System.Collections.Immutable: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • System.Configuration.ConfigurationManager: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • System.Diagnostics.DiagnosticSource: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • System.Diagnostics.PerformanceCounter: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • System.IO.Hashing: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • System.Net.Http.Json: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • System.Security.Cryptography.Pkcs: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • System.Security.Cryptography.Xml: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • System.Text.Encodings.Web: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • System.Text.Json: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)
    • System.Runtime.Caching: from 8.0.0-rc.1.23404.1 to 8.0.0-rc.1.23408.18 (parent: Microsoft.AspNetCore.App.Runtime.win-x64)

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

  • Subscription: de79120f-9533-4f2f-54c7-08db570beb04
  • Build: 20230809.22
  • Date Produced: August 10, 2023 2:09:36 AM UTC
  • Commit: 7834350746ecd0c5bde820d07b065ae3e82a4da3
  • Branch: refs/heads/main
Microsoft Reviewers: Open in CodeFlow

@joperezr
Copy link
Member

Interesting so looks like the Http Resilience tests are failing with this update with the following callstack:

[xUnit.net 00:00:01.60]     Microsoft.Extensions.Http.Resilience.Test.HttpClientBuilderExtensionsTests.AddResilienceHandler_EnsureProperStrategyInstanceRetrieved(bySelector: True) [FAIL]
[xUnit.net 00:00:01.60]       System.MissingMethodException : Method not found: 'System.Diagnostics.Metrics.IMeterFactory System.Net.Http.SocketsHttpHandler.get_MeterFactory()'.
[xUnit.net 00:00:01.60]       Stack Trace:
[xUnit.net 00:00:01.60]            at Microsoft.Extensions.Http.MetricsFactoryHttpMessageHandlerFilter.<>c__DisplayClass2_0.<Configure>b__0(HttpMessageHandlerBuilder builder)
[xUnit.net 00:00:01.60]            at Microsoft.Extensions.Http.LoggingHttpMessageHandlerBuilderFilter.<>c__DisplayClass3_0.<Configure>b__0(HttpMessageHandlerBuilder builder)
[xUnit.net 00:00:01.60]            at Microsoft.Extensions.Http.DefaultHttpClientFactory.CreateHandlerEntry(String name)
[xUnit.net 00:00:01.60]            at Microsoft.Extensions.Http.DefaultHttpClientFactory.<>c__DisplayClass14_0.<.ctor>b__1()
[xUnit.net 00:00:01.60]            at System.Lazy`1.ViaFactory(LazyThreadSafetyMode mode)
[xUnit.net 00:00:01.60]            at System.Lazy`1.ExecutionAndPublication(LazyHelper executionAndPublication, Boolean useDefaultConstructor)
[xUnit.net 00:00:01.60]            at System.Lazy`1.CreateValue()
[xUnit.net 00:00:01.60]            at Microsoft.Extensions.Http.DefaultHttpClientFactory.CreateHandler(String name)
[xUnit.net 00:00:01.60]            at Microsoft.Extensions.Http.DefaultHttpClientFactory.CreateClient(String name)
[xUnit.net 00:00:01.60]         /_/test/Libraries/Microsoft.Extensions.Http.Resilience.Tests/Resilience/HttpClientBuilderExtensionsTests.Resilience.cs(197,0): at Microsoft.Extensions.Http.Resilience.Test.HttpClientBuilderExtensionsTests.AddResilienceHandler_EnsureProperStrategyInstanceRetrieved(Boolean bySelector)
[xUnit.net 00:00:01.60]         --- End of stack trace from previous location ---
[xUnit.net 00:00:01.60]     Microsoft.Extensions.Http.Resilience.Test.HttpClientBuilderExtensionsTests.AddResilienceHandler_EnsureProperStrategyInstanceRetrieved(bySelector: False) [FAIL]
[xUnit.net 00:00:01.60]       System.MissingMethodException : Method not found: 'System.Diagnostics.Metrics.IMeterFactory System.Net.Http.SocketsHttpHandler.get_MeterFactory()'.
[xUnit.net 00:00:01.60]       Stack Trace:
[xUnit.net 00:00:01.60]            at Microsoft.Extensions.Http.MetricsFactoryHttpMessageHandlerFilter.<>c__DisplayClass2_0.<Configure>b__0(HttpMessageHandlerBuilder builder)
[xUnit.net 00:00:01.60]            at Microsoft.Extensions.Http.LoggingHttpMessageHandlerBuilderFilter.<>c__DisplayClass3_0.<Configure>b__0(HttpMessageHandlerBuilder builder)
[xUnit.net 00:00:01.60]            at Microsoft.Extensions.Http.DefaultHttpClientFactory.CreateHandlerEntry(String name)
[xUnit.net 00:00:01.60]            at Microsoft.Extensions.Http.DefaultHttpClientFactory.<>c__DisplayClass14_0.<.ctor>b__1()
[xUnit.net 00:00:01.60]            at System.Lazy`1.ViaFactory(LazyThreadSafetyMode mode)
[xUnit.net 00:00:01.60]            at System.Lazy`1.ExecutionAndPublication(LazyHelper executionAndPublication, Boolean useDefaultConstructor)
[xUnit.net 00:00:01.60]            at System.Lazy`1.CreateValue()
[xUnit.net 00:00:01.60]            at Microsoft.Extensions.Http.DefaultHttpClientFactory.CreateHandler(String name)
[xUnit.net 00:00:01.60]            at Microsoft.Extensions.Http.DefaultHttpClientFactory.CreateClient(String name)
[xUnit.net 00:00:01.60]         /_/test/Libraries/Microsoft.Extensions.Http.Resilience.Tests/Resilience/HttpClientBuilderExtensionsTests.Resilience.cs(197,0): at Microsoft.Extensions.Http.Resilience.Test.HttpClientBuilderExtensionsTests.AddResilienceHandler_EnsureProperStrategyInstanceRetrieved(Boolean bySelector)
[xUnit.net 00:00:01.60]         --- End of stack trace from previous location ---

@noahfalk @MihaZupan @antonfirsov @tarekgh Was this method removed recently? Does this ring any bells? FYI @martintmk since the failing tests are in http resilience.

@joperezr
Copy link
Member

@MihaZupan
Copy link
Member

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

@joperezr
Copy link
Member

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.

@joperezr
Copy link
Member

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.

@RussKie
Copy link
Member

RussKie commented Aug 10, 2023

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.

@RussKie
Copy link
Member

RussKie commented Aug 11, 2023

image

dotnet-maestro bot and others added 4 commits August 11, 2023 11:54
…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
@RussKie RussKie force-pushed the darc-main-30268223-4571-4989-a0a2-3857af056c1c branch from 439d52c to c5c4bea Compare August 11, 2023 01:56
@dotnet-maestro dotnet-maestro bot merged commit cd82a9f into main Aug 11, 2023
@dotnet-maestro dotnet-maestro bot deleted the darc-main-30268223-4571-4989-a0a2-3857af056c1c branch August 11, 2023 03:18
@ghost ghost locked as resolved and limited conversation to collaborators Sep 10, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants