[azd update] Improve error message by replacing non-standard install language with administrator-managed messaging#7417
Conversation
There was a problem hiding this comment.
Pull request overview
Updates Windows MSI update messaging to avoid “non-standard install” wording and instead indicate the install may be administrator-managed, while documenting per-machine update expectations for MSI installs.
Changes:
- Adjusted
isStandardMSIInstall()error text to recommend contacting an administrator and added anAZD_SKIP_UPDATE_CHECK=1suppression hint. - Updated the corresponding unit test assertion to match the new error message text.
- Documented in the installer README that per-machine MSI installs (
ALLUSERS=1) require administrator-managed updates and notedAZD_SKIP_UPDATE_CHECK=1for suppressing notifications.
Reviewed changes
Copilot reviewed 3 out of 3 changed files in this pull request and generated no comments.
| File | Description |
|---|---|
| cli/installer/README.md | Documents per-machine MSI install update responsibility and how to suppress update notifications. |
| cli/azd/pkg/update/msi_windows.go | Rewords non-standard MSI install error messaging and adds AZD_SKIP_UPDATE_CHECK=1 hint. |
| cli/azd/pkg/update/msi_windows_test.go | Aligns test assertion with the updated error message text. |
jongio
left a comment
There was a problem hiding this comment.
Messaging changes look good and match the issue requirements. One nit outside the diff:
- msi_windows.go:129-131 - doc comment for
isStandardMSIInstall()still saysadvising the user to reinstall- worth updating to also mention thecontact administratorguidance
Azure Dev CLI Install InstructionsInstall scriptsMacOS/Linux
bash: pwsh: WindowsPowerShell install MSI install Standalone Binary
MSI
Documentationlearn.microsoft.com documentationtitle: Azure Developer CLI reference
|
- Add 1.24.0 (Unreleased) CHANGELOG section documenting: - --non-interactive flag alias and AZD_NON_INTERACTIVE env var (Azure#7392) - Hooks per provisioning layer in azure.yaml (Azure#7382) - Docker build network option in azure.yaml (Azure#7361) - Improved azd update error message for admin-managed installs (Azure#7417) - Deprecation of -e short flag in AI extensions (Azure#7313) - Add AZD_NON_INTERACTIVE to environment-variables.md Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Description
Fixes #7414
Changes
cli/azd/pkg/update/msi_windows.go - Updated
isStandardMSIInstall()error message:azd is installed in a non-standard locationwithazd installation might be managed by an administratorAZD_SKIP_UPDATE_CHECK=1hint to suppress update notificationscli/azd/pkg/update/msi_windows_test.go - Updated test assertion to match new message text
cli/installer/README.md - Added note after MSI configuration table documenting that per-machine installs (
ALLUSERS=1) require the administrator to manage updates, withAZD_SKIP_UPDATE_CHECK=1to suppress notifications