Skip to content

Conversation

@joperezr
Copy link
Member

cc: @dsplaisted @marcpopMSFT @baronfel

This change is making it so that the 9.0 SDK will have an 8.0 Aspire workload, as opposed to the 9.0 previews that it had before. Aspire decided to not have 9.0 previews and instead only maintain a single stable 8.0 train, so given this decision, this change will make it so the SDK will carry the latest 8.0 workload as opposed to a dead-ended 9.0 preview one.

We have validated that this will still allow users to use the 9.0 SDK to build Aspire apps, even if they are targeting 9.0 TFMs in their individual services. After this change ships, we'll be able to un-list all of the (now dead-ended) 9.0 preview packages for Aspire workloads, in favor of pushing folks to use to the live 8.0 train.

@joperezr joperezr requested a review from marcpopMSFT June 12, 2024 20:43
@ghost ghost added Area-CodeFlow untriaged Request triage from a team member labels Jun 12, 2024
@marcpopMSFT
Copy link
Member

@joperezr any customer who keeps an older preview installed will be broken for aspire, correct? If so, we'll probably want to either breaking change or relnote this so that there's a place to point customers to tell them to uninstall their old sdk. Would it be worth detecting that a 9.0 aspire manifest is installed (any way to do that?) and provide a warning to help customers unwind this?

@joperezr
Copy link
Member Author

Wouldn't installing a newer preview of the SDK replace a previous installation? (and therefore remove the 9.0 manifest)

If so I'm fine with doing a relnote to basically just tell customers that if they want to use Aspire they should just install latest 9.0 SDK

@marcpopMSFT
Copy link
Member

We only replace the earlier version on windows admin installs. Zip installs and mac/linux installs would not replace the prior version. We've definitely run into problems of people having months old previews still installed fwiw.

@marcpopMSFT
Copy link
Member

/azp run sdk-source-build,sdk-unified-build

@azure-pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@joperezr
Copy link
Member Author

We only replace the earlier version on windows admin installs. Zip installs and mac/linux installs would not replace the prior version. We've definitely run into problems of people having months old previews still installed fwiw.

I see, in those cases I think we can suggest either uninstalling manually or alternatively using rollback files (which is the current workaround we advertise to folks using the 9.0 SDK) to force the use of the 8.0 workload as opposed to newer manifests.

@marcpopMSFT
Copy link
Member

That's a reasonable approach. I just wanted to ensure there was documentation we could point people at. Remerging as the vmr builds were having issues and let's see if that helps.

@joperezr
Copy link
Member Author

@marcpopMSFT trying to understand the source build failures but not sure I'm following what the issue is. Do you have any clue or do you know who can I work with to resolve? I'd really like this to ship in Preview 6 and I know snap is soon.

@marcpopMSFT
Copy link
Member

@marcpopMSFT trying to understand the source build failures but not sure I'm following what the issue is. Do you have any clue or do you know who can I work with to resolve? I'd really like this to ship in Preview 6 and I know snap is soon.

@dotnet/source-build-internal and @dotnet/product-construction for the source build and VMR failures. Looks like a similar failure in both around merged manifests and not sure how that's related to this PR:
/vmr/eng/merge-asset-manifests.proj(21,5): error MSB4018: The "Microsoft.DotNet.UnifiedBuild.Tasks.MergeAssetManifests" task failed unexpectedly.

@mmitche
Copy link
Member

mmitche commented Jun 19, 2024

The error is about how the manifest produced by aspire's publishing doesn't have the same root attributes as the other repos. Because you rolled the aspire sha back to an older 8.0 version, it's highly likely that it's missing some fixes that were made in mian to work with the VMR.

@joperezr
Copy link
Member Author

I see, is there a way to know which fixes are those? Aspire builds with the 8.0 arcade, so if the fixes are done in arcade main then that would explain it.

@mmitche mmitche requested review from a team as code owners June 21, 2024 18:57
@joperezr
Copy link
Member Author

/backport to release/9.0.1xx-preview6

@github-actions
Copy link
Contributor

Started backporting to release/9.0.1xx-preview6: https://github.com/dotnet/sdk/actions/runs/9667085593

@joperezr
Copy link
Member Author

/azp run sdk-unified-build

@azure-pipelines
Copy link

Azure Pipelines successfully started running 1 pipeline(s).

@joperezr joperezr enabled auto-merge June 28, 2024 23:20
@joperezr
Copy link
Member Author

Looks like this is ready to go @marcpopMSFT 🙂

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Area-CodeFlow untriaged Request triage from a team member

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants