Skip to content

[ci] Fix maestro publishing for stable packages (#23664)#23681

Merged
rmarinho merged 1 commit into
mainfrom
fix-maestro-main
Jul 18, 2024
Merged

[ci] Fix maestro publishing for stable packages (#23664)#23681
rmarinho merged 1 commit into
mainfrom
fix-maestro-main

Conversation

@rmarinho

Copy link
Copy Markdown
Member

Backport to main of #23664

We've seen the build promotion pipeline fail when trying to publish
stable package versions:

    error : Package 'Microsoft.AspNetCore.Components.WebView.Maui' has stable version '8.0.60' but is targeted at a non-isolated feed 'https://pkgs.dev.azure.com/dnceng/public/_packaging/dotnet8/nuget/v3/index.json'

This is because we were not declaring these packages as stable when
building the build asset registry manifest.

Fix this by passing the `$(StabilizePackageVersion)` property to the
build asset manifest creation task. This property needs to be updated
manually when switching to stable package versioning (see 4eab956220b).

When `$(StabilizePackageVersion)` is set to true, packages will be
pushed to an isolated feed during publishing, such as:

    Package Microsoft.Android.Ref.34@34.0.125 (Shipping) should go to https://pkgs.dev.azure.com/dnceng/public/_packaging/darc-pub-dotnet-android-b8317b6f/nuget/v3/index.json (Isolated, Public)
@rmarinho rmarinho requested a review from a team as a code owner July 18, 2024 13:50
@rmarinho rmarinho merged commit bb2819e into main Jul 18, 2024
@rmarinho rmarinho deleted the fix-maestro-main branch July 18, 2024 13:51
@samhouts samhouts added fixed-in-net9.0-nightly This may be available in a nightly release! fixed-in-8.0.80 and removed fixed-in-net9.0-nightly This may be available in a nightly release! labels Aug 2, 2024
@github-actions github-actions Bot locked and limited conversation to collaborators Sep 8, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

fixed-in-8.0.80 fixed-in-net9.0-nightly This may be available in a nightly release!

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants