-
Notifications
You must be signed in to change notification settings - Fork 3.3k
{Packaging} Remove aio and dist-info folders in MSI #17317
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
azure-pipelines.yml
Outdated
| }else { | ||
| $reinstall_option="" | ||
| } | ||
| Start-Process "msiexec.exe" -ArgumentList "/i `"$env:SYSTEM_ARTIFACTSDIRECTORY\msi\Microsoft Azure CLI.msi`" $reinstall_option /q /norestart /l*v .\install_logs.txt" -Wait -NoNewWindow |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
$InstallArgs looks more readable to me.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Reverted
| if ($pre_installed_version -eq $to_be_installed_version){ | ||
| # See https://docs.microsoft.com/windows/win32/msi/reinstallmode about options of REINSTALLMODE | ||
| $reinstall_option="REINSTALL=ALL REINSTALLMODE=emus" | ||
| $InstallArgs += $reinstall_option | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ADO agent has a pre-installed Azure CLI MSI, so this job tries to "reinstall" with the artifact MSI. If the artifact MSI has the same version as the pre-installed Azure CLI, L226 adds REINSTALL=ALL REINSTALLMODE=emus.
If I run the msiexec.exe command locally (with or without /q):
msiexec.exe /i "Microsoft Azure CLI.msi" REINSTALL=ALL REINSTALLMODE=emusit succeeds very fast but doesn't install anything, so the "reinstall" actually never works.
I guess it is because the artifact MSI is actually a different product as the pre-installed one, so it has nothing to "reinstall". I am not sure if this is related to Product.Id being *:
| <Product Id="*" |
After removing REINSTALL=ALL REINSTALLMODE=emus, the artifact MSI can install successfully, but is installed side-by-side with the pre-installed one.
|
After removing the |
Description
azure-clidoes not useaiofiles in azure management SDKs.dist-infois not used in MSI for installation or update. Remove them in this PR.This PR also adds back the MSI test task to resolve #9270
Testing Guide
History Notes
[Component Name 1] BREAKING CHANGE: az command a: Make some customer-facing breaking change.
[Component Name 2] az command b: Add some customer-facing feature.
This checklist is used to make sure that common guidelines for a pull request are followed.
The PR title and description has followed the guideline in Submitting Pull Requests.
I adhere to the Command Guidelines.
I adhere to the Error Handling Guidelines.