Skip to content

Conversation

@NikolaMilosavljevic
Copy link
Member

@NikolaMilosavljevic NikolaMilosavljevic commented Nov 15, 2021

Stable upgrade codes and dependency key to allow proper msi upgrade experience.

We are using values from latest official release (6.0.0).

@joeloff
Copy link
Member

joeloff commented Nov 15, 2021

We should probably look at the dependency provider key and figure out what to do. Below is from RC2 and RTM. When upgrading, we need to ensure stable key so that chainer dependencies can be maintained, otherwise we risk losing the ref count against the MSI itself.

image

@NikolaMilosavljevic
Copy link
Member Author

We should probably look at the dependency provider key and figure out what to do. Below is from RC2 and RTM. When upgrading, we need to ensure stable key so that chainer dependencies can be maintained, otherwise we risk losing the ref count against the MSI itself.

image

Thanks. I'm going to see what the best option is to override the auto-generated values.

@joeloff
Copy link
Member

joeloff commented Nov 15, 2021

We should probably look at the dependency provider key and figure out what to do. Below is from RC2 and RTM. When upgrading, we need to ensure stable key so that chainer dependencies can be maintained, otherwise we risk losing the ref count against the MSI itself.
image

Thanks. I'm going to see what the best option is to override the auto-generated values.
Something like Dotnet_CLI_SharedHost,$(MajorVersion).$(MinorVersion),$(Platform) should work (or whatever the relevant properties are). The tricky bit is whether we decide to leave the older versions or if we allow them to upgrade, allow the ref count to become inaccurate as part of the transition.

@NikolaMilosavljevic
Copy link
Member Author

We should probably look at the dependency provider key and figure out what to do. Below is from RC2 and RTM. When upgrading, we need to ensure stable key so that chainer dependencies can be maintained, otherwise we risk losing the ref count against the MSI itself.
image

Thanks. I'm going to see what the best option is to override the auto-generated values.
Something like Dotnet_CLI_SharedHost,$(MajorVersion).$(MinorVersion),$(Platform) should work (or whatever the relevant properties are). The tricky bit is whether we decide to leave the older versions or if we allow them to upgrade, allow the ref count to become inaccurate as part of the transition.

I was thinking of allowing override of ProviderKey value, which works in my tests and is in my new Arcade PR: dotnet/arcade#8181

@NikolaMilosavljevic
Copy link
Member Author

I've pushed the new change to stabilize ProviderKey value by overriding the Candle variable.

@NikolaMilosavljevic
Copy link
Member Author

I've pushed the new change to stabilize ProviderKey value by overriding the Candle variable.

Will also need to update a reference to Arcade's Installers package once Arcade PR is approved and we get a build: dotnet/arcade#8181

@NikolaMilosavljevic
Copy link
Member Author

I've updated the Arcade package (Microsoft.DotNet.Build.Tasks.Installers). This PR can now be merged, once I get the approving review and it finishes building.

@safern
Copy link
Member

safern commented Nov 16, 2021

@danmoseley is this approved to merge for this servicing?

@NikolaMilosavljevic
Copy link
Member Author

This test failure seems to be a known issue, that is breaking other servicing PRs: #61636

@NikolaMilosavljevic
Copy link
Member Author

@danmoseley is this approved to merge for this servicing?

This is all part of the same setup work, for 6.0, 5.0 and 3.1. Here are few other examples:
#61619
dotnet/arcade#8181
dotnet/arcade#8174

@safern
Copy link
Member

safern commented Nov 16, 2021

Test failure is: #61636

@safern safern merged commit 2165555 into dotnet:release/6.0 Nov 16, 2021
@ghost ghost locked as resolved and limited conversation to collaborators Dec 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants