Skip to content

Stabilize upgrade codes for host msi - #61606

Merged
safern merged 5 commits into
dotnet:release/6.0from
NikolaMilosavljevic:stable.uc.60
Nov 16, 2021
Merged

Stabilize upgrade codes for host msi#61606
safern merged 5 commits into
dotnet:release/6.0from
NikolaMilosavljevic:stable.uc.60

Conversation

@NikolaMilosavljevic

@NikolaMilosavljevicNikolaMilosavljevic commented Nov 15, 2021

Copy link
Copy Markdown
Contributor

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
Copy Markdown
Member

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
Copy Markdown
ContributorAuthor

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
Copy Markdown
Member

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
Copy Markdown
ContributorAuthor

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
Copy Markdown
ContributorAuthor

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

@NikolaMilosavljevic

Copy link
Copy Markdown
ContributorAuthor

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
Copy Markdown
ContributorAuthor

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.

Comment threadsrc/installer/pkg/sfx/installers/dotnet-host.proj
Comment threadsrc/installer/pkg/sfx/installers/dotnet-host.proj
@safern

Copy link
Copy Markdown
Member

@danmoseley is this approved to merge for this servicing?

@NikolaMilosavljevic

Copy link
Copy Markdown
ContributorAuthor

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

@NikolaMilosavljevic

Copy link
Copy Markdown
ContributorAuthor

@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
Copy Markdown
Member

Test failure is: #61636

@safern
safern merged commit 2165555 into dotnet:release/6.0Nov 16, 2021
@ghostghost locked as resolved and limited conversation to collaborators Dec 16, 2021
Sign up for freeto 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

@NikolaMilosavljevic@joeloff@safern@ericstj