Skip to content

Conversation

@jiashun0011
Copy link
Contributor

What type of PR is this?

/kind feature

What this PR does / why we need it:

Which issue(s) this PR fixes:
Fallback mcr.azk8s.cn to mcr.azure.cn in Windows containerd

Requirements:

  • uses conventional commit messages
  • includes documentation
  • adds unit tests
  • tested upgrade from previous version
  • commits are GPG signed and Github marks them as verified

Special notes for your reviewer:

Release note:

[feat] Fallback mcr.azk8s.cn to mcr.azure.cn in Windows containerd

func Test_Windows2022_McrChinaCloud_Windows(t *testing.T) {
RunScenario(t, &Scenario{
Tags: Tags{
MockAzureChinaCloud: true,
Copy link
Contributor

Choose a reason for hiding this comment

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

can we try to change the e2e daemonset test image from mcr.microsoft.com to mcr.azk8s.cn if detect MockAzureChinaCloud=true

@fseldow
Copy link
Contributor

fseldow commented Jan 13, 2026

incorrect title
starts with feat: instead of [feat]
The purpose of pr is not fallback mcr.azk8s.cn. It is to redirect the host

@jiashun0011 jiashun0011 changed the title [feat] Fallback mcr.azk8s.cn to mcr.azure.cn in Windows containerd feat: redirect mcr.azk8s.cn to mcr.azure.cn in Windows containerd Jan 14, 2026
@jiashun0011 jiashun0011 force-pushed the jiashunliu/mcr-migration-cse branch from e09841e to feb7075 Compare January 14, 2026 02:30
Copy link
Contributor

@timmy-wright timmy-wright left a comment

Choose a reason for hiding this comment

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

As per comments

@qweeah
Copy link
Contributor

qweeah commented Jan 15, 2026

Minor comment on the release note:

[feat] Fallback mcr.azk8s.cn to mcr.azure.cn in Windows containerd

What is actually done should be Mirroring mcr.azk8s.cn through mcr.azure.cn with auto fallback in Windows containerd.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants