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

[net7.0] Update dependencies from dotnet/aspnetcore #9014

Closed

Conversation

dotnet-maestro[bot]
Copy link
Contributor

@dotnet-maestro dotnet-maestro bot commented Jul 26, 2022

This pull request updates the following dependencies

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

  • Subscription: 99c89091-cd95-45ad-0b04-08da677c2adc
  • Build: 20220726.6
  • Date Produced: July 26, 2022 8:27:30 PM UTC
  • Commit: cccab50fba1c7335fdb48a735246374b5ddd7e69
  • Branch: refs/heads/release/7.0-preview7

…0220726.4

Microsoft.AspNetCore.Authorization , Microsoft.AspNetCore.Components , Microsoft.AspNetCore.Components.Analyzers , Microsoft.AspNetCore.Components.Forms , Microsoft.AspNetCore.Components.Web , Microsoft.AspNetCore.Components.WebView , Microsoft.AspNetCore.Metadata , Microsoft.JSInterop
 From Version 7.0.0-preview.7.22369.8 -> To Version 7.0.0-preview.7.22376.4
@ghost ghost added the area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions label Jul 26, 2022
…0220726.6

Microsoft.AspNetCore.Authorization , Microsoft.AspNetCore.Components , Microsoft.AspNetCore.Components.Analyzers , Microsoft.AspNetCore.Components.Forms , Microsoft.AspNetCore.Components.Web , Microsoft.AspNetCore.Components.WebView , Microsoft.AspNetCore.Metadata , Microsoft.JSInterop
 From Version 7.0.0-preview.7.22369.8 -> To Version 7.0.0-preview.7.22376.6
@dougbu
Copy link
Member

dougbu commented Jul 27, 2022

Something odd here. Why do both #9013 and this PR exist❔

@TanayParikh
Copy link
Contributor

Something odd here. Why do both #9013 and this PR exist❔

This is:

Subscription: 99c89091-cd95-45ad-0b04-08da677c2adc

#9013 is:

Subscription: b26f9119-2306-4d94-1d05-08da47a7d2f9

This is the one we want (Preview 7). #9013 is RC1 and thus likely a different channel.

@mattleibow
Copy link
Member

/azp run

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@TanayParikh
Copy link
Contributor

This is failing as it's resulting in package downgrades of dependencies:

D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.csproj : error NU1605: Detected package downgrade: Microsoft.Extensions.Logging.Abstractions from 7.0.0-preview.7.22375.6 to 7.0.0-preview.7.22369.4. Reference the package directly from the project to select a different version.  [D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.sln]
D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.csproj : error NU1605:  mauiblazor_DotNet -> Microsoft.AspNetCore.Components.WebView.Maui 7.0.0-preview.7.9930 -> Microsoft.AspNetCore.Authorization 7.0.0-preview.7.22376.6 -> Microsoft.Extensions.Logging.Abstractions (>= 7.0.0-preview.7.22375.6)  [D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.sln]
D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.csproj : error NU1605:  mauiblazor_DotNet -> Microsoft.Extensions.Logging.Abstractions (>= 7.0.0-preview.7.22369.4) [D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.sln]
D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.csproj : error NU1605: Detected package downgrade: Microsoft.Extensions.Logging from 7.0.0-preview.7.22375.6 to 7.0.0-preview.7.22369.4. Reference the package directly from the project to select a different version.  [D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.sln]
D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.csproj : error NU1605:  mauiblazor_DotNet -> Microsoft.AspNetCore.Components.WebView.Maui 7.0.0-preview.7.9930 -> Microsoft.AspNetCore.Components.WebView 7.0.0-preview.7.22376.6 -> Microsoft.Extensions.Logging (>= 7.0.0-preview.7.22375.6)  [D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.sln]
D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.csproj : error NU1605:  mauiblazor_DotNet -> Microsoft.Extensions.Logging (>= 7.0.0-preview.7.22369.4) [D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.sln]
D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.csproj : error NU1605: Detected package downgrade: Microsoft.Extensions.DependencyInjection from 7.0.0-preview.7.22375.6 to 7.0.0-preview.7.22369.4. Reference the package directly from the project to select a different version.  [D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.sln]
D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.csproj : error NU1605:  mauiblazor_DotNet -> Microsoft.AspNetCore.Components.WebView.Maui 7.0.0-preview.7.9930 -> Microsoft.AspNetCore.Components.WebView 7.0.0-preview.7.22376.6 -> Microsoft.AspNetCore.Components.Web 7.0.0-preview.7.22376.6 -> Microsoft.Extensions.DependencyInjection (>= 7.0.0-preview.7.22375.6)  [D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.sln]
D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.csproj : error NU1605:  mauiblazor_DotNet -> Microsoft.Extensions.DependencyInjection (>= 7.0.0-preview.7.22369.4) [D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.sln]
D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.csproj : error NU1605: Detected package downgrade: Microsoft.Extensions.Configuration from 7.0.0-preview.7.22375.6 to 7.0.0-preview.7.22369.4. Reference the package directly from the project to select a different version.  [D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.sln]
D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.csproj : error NU1605:  mauiblazor_DotNet -> Microsoft.AspNetCore.Components.WebView.Maui 7.0.0-preview.7.9930 -> Microsoft.AspNetCore.Components.WebView 7.0.0-preview.7.22376.6 -> Microsoft.Extensions.Configuration.Json 7.0.0-preview.7.22375.6 -> Microsoft.Extensions.Configuration (>= 7.0.0-preview.7.22375.6)  [D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.sln]
D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.csproj : error NU1605:  mauiblazor_DotNet -> Microsoft.Extensions.Configuration (>= 7.0.0-preview.7.22369.4) [D:\a\_work\_temp\templatesTest\mauiblazor_DotNet\mauiblazor_DotNet.sln]

Are those packages updated via a separate subscription?

@TanayParikh
Copy link
Contributor

Microsoft.AspNetCore.Components.WebView.Maui 7.0.0-preview.7.9930 -> Microsoft.AspNetCore.Components.WebView 7.0.0-preview.7.22376.6

7.0.0-preview.7.9930 seems a bit unexpected. cc/ @dougbu

@mattleibow
Copy link
Member

the 9930 is the version in the maui repo. It is the maui blazor view package.

@dougbu
Copy link
Member

dougbu commented Jul 29, 2022

7.0.0-preview.7.9930 seems a bit unexpected.

Given @mattleibow's comment, this is completely expected given dotnet/runtime packages are brought in through coherency from dotnet/aspnetcore and xamarin/xamarin-macios. Probably need to update dotnet/efcore, dotnet/aspnetcore, dotnet/sdk, dotnet/installer, and xamarin/xamarin-macios to have enough in eng/Version.Details.xml for strict coherency everywhere. This repo would then only need a xamarin/xamarin-macios subscription. (As-is the xamarin/xamarin-andriod subscription is unnecessary because that repo doesn't push versions into any channel.)

/cc @mmitche @riarenas since I mentioned strict coherency, something we probably lost after 3.1.x or 5.0.x ☹️

@mattleibow mattleibow added the do-not-merge Don't merge this PR label Aug 4, 2022
dotnet-maestro bot and others added 4 commits August 5, 2022 13:11
…0220726.6

Microsoft.AspNetCore.Authorization , Microsoft.AspNetCore.Components , Microsoft.AspNetCore.Components.Analyzers , Microsoft.AspNetCore.Components.Forms , Microsoft.AspNetCore.Components.Web , Microsoft.AspNetCore.Components.WebView , Microsoft.AspNetCore.Metadata , Microsoft.JSInterop
 From Version 7.0.0-preview.7.22369.8 -> To Version 7.0.0-preview.7.22376.6
This reverts commit 3fd5d18.
This reverts commit 6011303.
@mattleibow
Copy link
Member

Closing this PR because of branch changes and there are new maestro builds:

@mattleibow mattleibow closed this Aug 8, 2022
@mattleibow mattleibow deleted the darc-net7.0-549250a9-0dd1-46e3-8877-fd4a804ecccb branch August 8, 2022 19:14
@github-actions github-actions bot locked and limited conversation to collaborators Dec 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
area-infrastructure CI, Maestro / Coherency, upstream dependencies/versions do-not-merge Don't merge this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants