-
Notifications
You must be signed in to change notification settings - Fork 10.4k
[master] Update dependencies from dotnet/efcore #20532
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
[master] Update dependencies from dotnet/efcore #20532
Conversation
…403.6 - Microsoft.EntityFrameworkCore.Tools - 5.0.0-preview.4.20203.6 - Microsoft.EntityFrameworkCore.SqlServer - 5.0.0-preview.4.20203.6 - dotnet-ef - 5.0.0-preview.4.20203.6 - Microsoft.EntityFrameworkCore - 5.0.0-preview.4.20203.6 - Microsoft.EntityFrameworkCore.InMemory - 5.0.0-preview.4.20203.6 - Microsoft.EntityFrameworkCore.Relational - 5.0.0-preview.4.20203.6 - Microsoft.EntityFrameworkCore.Sqlite - 5.0.0-preview.4.20203.6
There was a problem hiding this 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.
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
…403.5 - Microsoft.EntityFrameworkCore.Tools - 5.0.0-preview.4.20203.5 - Microsoft.EntityFrameworkCore.SqlServer - 5.0.0-preview.4.20203.5 - dotnet-ef - 5.0.0-preview.4.20203.5 - Microsoft.EntityFrameworkCore - 5.0.0-preview.4.20203.5 - Microsoft.EntityFrameworkCore.InMemory - 5.0.0-preview.4.20203.5 - Microsoft.EntityFrameworkCore.Relational - 5.0.0-preview.4.20203.5 - Microsoft.EntityFrameworkCore.Sqlite - 5.0.0-preview.4.20203.5
…403.8 - Microsoft.EntityFrameworkCore.Tools - 5.0.0-preview.4.20203.8 - Microsoft.EntityFrameworkCore.SqlServer - 5.0.0-preview.4.20203.8 - dotnet-ef - 5.0.0-preview.4.20203.8 - Microsoft.EntityFrameworkCore - 5.0.0-preview.4.20203.8 - Microsoft.EntityFrameworkCore.InMemory - 5.0.0-preview.4.20203.8 - Microsoft.EntityFrameworkCore.Relational - 5.0.0-preview.4.20203.8 - Microsoft.EntityFrameworkCore.Sqlite - 5.0.0-preview.4.20203.8
…404.3 - Microsoft.EntityFrameworkCore.Tools - 5.0.0-preview.4.20204.3 - Microsoft.EntityFrameworkCore.SqlServer - 5.0.0-preview.4.20204.3 - dotnet-ef - 5.0.0-preview.4.20204.3 - Microsoft.EntityFrameworkCore - 5.0.0-preview.4.20204.3 - Microsoft.EntityFrameworkCore.InMemory - 5.0.0-preview.4.20204.3 - Microsoft.EntityFrameworkCore.Relational - 5.0.0-preview.4.20204.3 - Microsoft.EntityFrameworkCore.Sqlite - 5.0.0-preview.4.20204.3
…404.8 - Microsoft.EntityFrameworkCore.Tools - 5.0.0-preview.4.20204.8 - Microsoft.EntityFrameworkCore.SqlServer - 5.0.0-preview.4.20204.8 - dotnet-ef - 5.0.0-preview.4.20204.8 - Microsoft.EntityFrameworkCore - 5.0.0-preview.4.20204.8 - Microsoft.EntityFrameworkCore.InMemory - 5.0.0-preview.4.20204.8 - Microsoft.EntityFrameworkCore.Relational - 5.0.0-preview.4.20204.8 - Microsoft.EntityFrameworkCore.Sqlite - 5.0.0-preview.4.20204.8
@ajcvickers Did you guys make a breaking change?
|
@davidfowl Yes. @AndriySvyryd? |
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
Did it get approval? Seems like it breaks identity server. Did we do analysis to see what other popular packages it breaks in the ecosystem? |
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
This is a binary breaking change, the return type changed. Recompiling will fix it. |
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
Looks as if recompilation isn't enough. The tests that are currently failing are template tests and they're failing while doing EF migrations e.g.
|
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
…406.1 - Microsoft.EntityFrameworkCore.Tools - 5.0.0-preview.4.20206.1 - Microsoft.EntityFrameworkCore.SqlServer - 5.0.0-preview.4.20206.1 - dotnet-ef - 5.0.0-preview.4.20206.1 - Microsoft.EntityFrameworkCore - 5.0.0-preview.4.20206.1 - Microsoft.EntityFrameworkCore.InMemory - 5.0.0-preview.4.20206.1 - Microsoft.EntityFrameworkCore.Relational - 5.0.0-preview.4.20206.1 - Microsoft.EntityFrameworkCore.Sqlite - 5.0.0-preview.4.20206.1 Dependency coherency updates - Microsoft.Extensions.Caching.Abstractions - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Caching.Memory - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration.Abstractions - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration.Binder - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration.CommandLine - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration.EnvironmentVariables - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration.FileExtensions - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration.Ini - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration.Json - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration.UserSecrets - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration.Xml - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.DependencyInjection.Abstractions - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.DependencyInjection - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.FileProviders.Abstractions - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.FileProviders.Composite - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.FileProviders.Physical - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.FileSystemGlobbing - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Hosting.Abstractions - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Hosting - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Http - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Logging.Abstractions - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Logging.Configuration - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Logging.Console - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Logging.Debug - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Logging.EventSource - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Logging.EventLog - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Logging.TraceSource - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Options.ConfigurationExtensions - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Options.DataAnnotations - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Options - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Primitives - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Logging - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language)
@dougbu I investigated the templates tests and locally I get this generated by the test:
The failure is caused by the mismatch of referenced versions. Not sure where these are coming from. |
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
I found that it's generated by https://github.com/dotnet/aspnetcore/blob/3787d7e7f070543cc9368d589a504fa8c4bd4830/src/ProjectTemplates/Web.Spa.ProjectTemplates/Microsoft.DotNet.Web.Spa.ProjectTemplates.csproj |
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
…406.2 - Microsoft.EntityFrameworkCore.Tools - 5.0.0-preview.4.20206.2 - Microsoft.EntityFrameworkCore.SqlServer - 5.0.0-preview.4.20206.2 - dotnet-ef - 5.0.0-preview.4.20206.2 - Microsoft.EntityFrameworkCore - 5.0.0-preview.4.20206.2 - Microsoft.EntityFrameworkCore.InMemory - 5.0.0-preview.4.20206.2 - Microsoft.EntityFrameworkCore.Relational - 5.0.0-preview.4.20206.2 - Microsoft.EntityFrameworkCore.Sqlite - 5.0.0-preview.4.20206.2 Dependency coherency updates - Microsoft.Extensions.Caching.Abstractions - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Caching.Memory - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration.Abstractions - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration.Binder - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration.CommandLine - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration.EnvironmentVariables - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration.FileExtensions - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration.Ini - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration.Json - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration.UserSecrets - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration.Xml - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Configuration - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.DependencyInjection.Abstractions - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.DependencyInjection - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.FileProviders.Abstractions - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.FileProviders.Composite - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.FileProviders.Physical - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.FileSystemGlobbing - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Hosting.Abstractions - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Hosting - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Http - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Logging.Abstractions - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Logging.Configuration - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Logging.Console - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Logging.Debug - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Logging.EventSource - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Logging.EventLog - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Logging.TraceSource - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Options.ConfigurationExtensions - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Options.DataAnnotations - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Options - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Primitives - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language) - Microsoft.Extensions.Logging - 5.0.0-preview.4.20201.2 (parent: Microsoft.AspNetCore.Razor.Language)
Disregard my previous 2 comments, the templates are fine. When identity server is rebuilt this will be fixed. |
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
How did you fix it? Doesn’t it require a new identity server ? |
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
What's the expected timing for "When identity server is rebuilt this will be fixed."❔ If it's not nearly-immediate, we should pin EFCore dependencies at their previous versions to cut down on the noise. @davidfowl I think @AndriySvyryd's comment above answers your earlier question about breaking the ecosystem. |
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
Filed DuendeArchive/IdentityServer4#4259
👍 |
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
Updated this PR to do nothing except pin the EFCore dependencies. |
@AndriySvyryd and @ajcvickers if there's a build between 5.0.0-preview.4.20203.1 and 5.0.0-preview.4.20206.2 that you'd prefer we stick with, please let us know. It's not that hard to run /fyi @Pilchie |
Auto-Merge StatusThis pull request has not been merged because Maestro++ is waiting on the following merge policies.
|
@JunTaoLuo if you get a chance to "Squash and merge" this PR, please update the description to describe pinning instead of an actual dependency update. |
Auto-Merge StatusThis pull request will be merged because the following merge policies have succeeded.
|
This pull request updates the following dependencies
From https://github.com/dotnet/efcore
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