Skip to content

[release/5.0] Allow Hosting Bundle to be upgraded by MU#37968

Merged
wtgodbe merged 8 commits intorelease/5.0from
backport/pr-37967-to-release/5.0
Nov 5, 2021
Merged

[release/5.0] Allow Hosting Bundle to be upgraded by MU#37968
wtgodbe merged 8 commits intorelease/5.0from
backport/pr-37967-to-release/5.0

Conversation

@github-actions
Copy link
Copy Markdown
Contributor

@github-actions github-actions Bot commented Nov 1, 2021

Backport of #37967 to release/5.0

/cc @wtgodbe

Customer Impact

Adds an .msi, dotnet-hosting-options, which writes to the registry the value of the switches used when last installing the hosting bundle for a given Major.Minor version. Subsequent installs of that band of the hosting bundle will choose what the install in the following way:

  • If the user passes any switches on the command line, use exactly those (and write them to the registry, along with 0 for any switches not passed).
  • Else use what's in the registry, if there are values in the registry.
  • Else set all switches to 0 and write 0 to the registry for each switch

Testing

Tested various scenarios with versions of the new hosting bundle across multiple Major.Minor.Patch versions, as well as how it interacts with previous Hosting Bundles without the Registry-writing behavior.

Risk

In December all customers who get this update thru MU will get a 1-time hit where, regardless of what they already had installed, the hosting bundle will install all components. We suspect most customers install everything anyways, and those who don't will be able to work around this by manually re-running the installer with whatever switches they passed the last time they installed the bundle manually. Additionally, there could be a scenario that we didn't think of (and therefore wasn't covered by our manual testing) that will break.

@github-actions github-actions Bot requested a review from Pilchie as a code owner November 1, 2021 00:57
@ghost ghost added the area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework label Nov 1, 2021
@ghost ghost added this to the 5.0.x milestone Nov 1, 2021
@ghost
Copy link
Copy Markdown

ghost commented Nov 1, 2021

Hi @github-actions[bot]. If this is not a tell-mode PR, please make sure to follow the instructions laid out in the servicing process document.
Otherwise, please add tell-mode label.

@wtgodbe wtgodbe requested review from a team, jamshedd and joeloff November 1, 2021 01:00
@wtgodbe wtgodbe added the Servicing-consider Shiproom approval is required for the issue label Nov 1, 2021
@ghost
Copy link
Copy Markdown

ghost commented Nov 1, 2021

Hi @github-actions[bot]. Please make sure you've updated the PR description to use the Shiproom Template. Also, make sure this PR is not marked as a draft and is ready-to-merge.

To learn more about how to prepare a servicing PR click here.

@wtgodbe wtgodbe changed the title [release/5.0] [release/6.0] Allow Hosting Bundle to be upgraded by MU [release/5.0] Allow Hosting Bundle to be upgraded by MU Nov 1, 2021
Copy link
Copy Markdown
Contributor

@dougbu dougbu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

All comments from #37969 apply here

@leecow leecow modified the milestones: 5.0.x, 5.0.13 Nov 2, 2021
@wtgodbe wtgodbe added Servicing-approved Shiproom has approved the issue and removed Servicing-consider Shiproom approval is required for the issue labels Nov 2, 2021
@wtgodbe
Copy link
Copy Markdown
Member

wtgodbe commented Nov 2, 2021

Approved in tactics

@dougbu
Copy link
Copy Markdown
Contributor

dougbu commented Nov 2, 2021

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 2 pipeline(s).

Comment thread src/Installers/Windows/HostOptions/Product.wxs Outdated
Comment thread src/Installers/Windows/HostOptions/Product.wxs
Comment thread src/Installers/Windows/HostOptions/Product.wxs Outdated
Comment thread src/Installers/Windows/WindowsHostingBundle/Bundle.wxs Outdated
@wtgodbe wtgodbe enabled auto-merge (squash) November 4, 2021 21:18
@wtgodbe wtgodbe disabled auto-merge November 4, 2021 22:00
@wtgodbe
Copy link
Copy Markdown
Member

wtgodbe commented Nov 5, 2021

Components E2E failures are unrelated

@wtgodbe wtgodbe merged commit c116150 into release/5.0 Nov 5, 2021
@wtgodbe wtgodbe deleted the backport/pr-37967-to-release/5.0 branch November 5, 2021 00:19
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-infrastructure Includes: MSBuild projects/targets, build scripts, CI, Installers and shared framework Servicing-approved Shiproom has approved the issue

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants