Skip to content

Conversation

@erik-bershel
Copy link
Contributor

Description

Follow up: #13290

Related issue:

#13279

Check list

  • Related issue / work item is attached
  • Tests are written (if applicable)
  • Documentation is updated (if applicable)
  • Changes are tested and related VM images are successfully generated

---

Co-authored-by: PrashantRaj18198 <PrashantRaj18198@gmail.com>
Copilot AI review requested due to automatic review settings November 12, 2025 20:22
Copilot finished reviewing on behalf of erik-bershel November 12, 2025 20:24
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR updates the AzCopy installation process for both macOS and Ubuntu platforms to download from GitHub releases instead of using Microsoft's aka.ms redirect URLs. This change provides more control over version selection and improves reliability by using the organization's standard GitHub release resolution helper functions.

Key Changes:

  • Replaced hardcoded aka.ms URLs with dynamic GitHub release asset URL resolution
  • Added platform/architecture-specific filtering for macOS (arm64 and amd64)
  • Improved shell variable quoting in the macOS script for better safety

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
images/ubuntu/scripts/build/install-azcopy.sh Replaced aka.ms URL with GitHub release resolution using resolve_github_release_asset_url for Linux amd64 assets
images/macos/scripts/build/install-azcopy.sh Replaced hardcoded aka.ms URLs with architecture-aware GitHub release resolution and improved variable quoting

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant