Skip to content

chore(deps): add 2604 acr-mirror ref - #9309

Open
Peter Damianov (pdamianov-dev) wants to merge 2 commits into
mainfrom
pd/u26/acr-mirror-pkg
Open

chore(deps): add 2604 acr-mirror ref#9309
Peter Damianov (pdamianov-dev) wants to merge 2 commits into
mainfrom
pd/u26/acr-mirror-pkg

Conversation

@pdamianov-dev

Copy link
Copy Markdown
Contributor

What this PR does / why we need it: chore(deps): add 2604 acr-mirror ref for artifact streaming support on 2604

Which issue(s) this PR fixes: chore(deps): add 2604 acr-mirror ref

Fixes #

Copilot AI balanced review requested due to automatic review settings August 25, 2026 23:19
@github-actions github-actions Bot added the components This pull request updates cached components on Linux or Windows VHDs label Aug 25, 2026
@github-actions

github-actions Bot commented Aug 25, 2026

Copy link
Copy Markdown
Contributor

Windows Unit Test Results

  3 files   13 suites   45s ⏱️
404 tests 404 ✅ 0 💤 0 ❌
407 runs  407 ✅ 0 💤 0 ❌

Results for commit a0437b0.

♻️ This comment has been updated with latest results.

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Adds acr-mirror 1.0.0 support for Ubuntu 26.04 VHDs, enabling artifact streaming.

Changes:

  • Adds Ubuntu 26.04 package metadata and download URL.
  • Keeps acr-mirror pinned against Renovate updates.

Package Update Analysis: acr-mirror

Version change: Unsupported → 1.0.0 (new OS enablement)
OS affected: Ubuntu 26.04 amd64 and arm64
Risk: 🟡 Medium — upstream changelog and artifact URLs could not be independently verified, and Ubuntu 26.04 lacks artifact-streaming E2E coverage.


💡 Configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

Comment thread parts/common/components.json

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

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

Suppressed comments (1)

e2e/scenario_test.go:1273

  • 🟡 Medium Risk — 🧪 Test Coverage: The ARM64 package is fetched from a distinct derived URL (acr-mirror-2604-arm64.deb), but this validation only checks that its services remain running. A broken ARM64 streaming path can still fall back silently to overlayfs and pass. Exercise ValidateArtifactStreamingImagePull on Ubuntu 26.04 ARM64 (using a compatible anonymous Premium ACR cluster) to validate the architecture-specific artifact end to end.
			Validator: func(ctx context.Context, s *Scenario) error {
				return errors.Join(
					ValidateNonEmptyDirectory(ctx, s, "/etc/overlaybd"),
					ValidateSystemdUnitIsRunning(ctx, s, "overlaybd-snapshotter.service"),
					ValidateSystemdUnitIsRunning(ctx, s, "overlaybd-tcmu.service"),
					ValidateSystemdUnitIsRunning(ctx, s, "acr-mirror.service"),
					ValidateSystemdUnitIsRunning(ctx, s, "containerd.service"),

Comment thread e2e/scenario_test.go
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

components This pull request updates cached components on Linux or Windows VHDs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants