-
Notifications
You must be signed in to change notification settings - Fork 241
feat(cloud-provider-azure): use Dalec for >= v1.32.11 #7591
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
base: main
Are you sure you want to change the base?
Conversation
| }, | ||
| { | ||
| "renovateTag": "registry=https://mcr.microsoft.com, name=oss/v2/kubernetes/azure-cloud-node-manager", | ||
| "latestVersion": "v1.33.6-1" |
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.
Should we remove the v1.33.6 (etc.) from oss/kubernetes/azure-cloud-node-manager above if we are caching the v2 (Dalec) ones? We are very careful about caching new versions because the VHD size often reach the disk size limit. Could you please make sure we only cache the necessary ones?
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.
I think we'll temporarily have multiple versions (Dalec and non-Dalec) while migrating. After the migrations is done, we'll clean up oss/kubernetes/azure-cloud-node-manager.
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.
need to add previousLatestVersion as well
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.
What would the previousLatestVersion be? This is the first time v1.32 and v1.33 are added to the entry. I see the latest tag before v1.32.11-1 is v1.32.3-4, v1.33.6-1 is the only tag for its minor version, and v1.34.3-1 replaces v1.34.3 (the latest tag before this is v1.34.2-1).
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.
sorry, I forgot dalec entries are different with the original ones. no need to add previousLatestVersion this time.
Signed-off-by: Liunardy <54991798+Liunardy@users.noreply.github.com>
e34f6db to
bc66334
Compare
Changes cached containers or packages on windows VHDsPlease get a Windows SIG member to approve. The following dif file shows any additions or deletions from what will be cached on windows VHDs organised by VHD type.
diff --git a/vhd_files/2019-containerd.txt b/vhd_files/2019-containerd.txt
index 80d24f0..9cd3775 100644
--- a/vhd_files/2019-containerd.txt
+++ b/vhd_files/2019-containerd.txt
@@ -3,0 +4,3 @@ c:\akse-cache\aks-secure-tls-bootstrap-client\: https://github.com/Azure/aks-sec
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.32.11/windows/amd64/azure-acr-credential-provider_1.32.11-1_amd64.zip
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.33.6/windows/amd64/azure-acr-credential-provider_1.33.6-1_amd64.zip
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.34.3/windows/amd64/azure-acr-credential-provider_1.34.3-1_amd64.zip
@@ -78 +81,3 @@ mcr.microsoft.com/oss/v2/kubernetes-csi/secrets-store/driver:v1.5.4
-mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.34.3
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.32.11-windows-hpc-1
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.33.6-windows-hpc-1
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.34.3-windows-hpc-1
diff --git a/vhd_files/2022-containerd-gen2.txt b/vhd_files/2022-containerd-gen2.txt
index 6bc4a5c..2f0a227 100644
--- a/vhd_files/2022-containerd-gen2.txt
+++ b/vhd_files/2022-containerd-gen2.txt
@@ -3,0 +4,3 @@ c:\akse-cache\aks-secure-tls-bootstrap-client\: https://github.com/Azure/aks-sec
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.32.11/windows/amd64/azure-acr-credential-provider_1.32.11-1_amd64.zip
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.33.6/windows/amd64/azure-acr-credential-provider_1.33.6-1_amd64.zip
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.34.3/windows/amd64/azure-acr-credential-provider_1.34.3-1_amd64.zip
@@ -128 +131,3 @@ mcr.microsoft.com/oss/v2/kubernetes-csi/secrets-store/driver:v1.5.4
-mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.34.3
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.32.11-windows-hpc-1
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.33.6-windows-hpc-1
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.34.3-windows-hpc-1
diff --git a/vhd_files/2022-containerd.txt b/vhd_files/2022-containerd.txt
index a8590c3..0624d0b 100644
--- a/vhd_files/2022-containerd.txt
+++ b/vhd_files/2022-containerd.txt
@@ -3,0 +4,3 @@ c:\akse-cache\aks-secure-tls-bootstrap-client\: https://github.com/Azure/aks-sec
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.32.11/windows/amd64/azure-acr-credential-provider_1.32.11-1_amd64.zip
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.33.6/windows/amd64/azure-acr-credential-provider_1.33.6-1_amd64.zip
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.34.3/windows/amd64/azure-acr-credential-provider_1.34.3-1_amd64.zip
@@ -128 +131,3 @@ mcr.microsoft.com/oss/v2/kubernetes-csi/secrets-store/driver:v1.5.4
-mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.34.3
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.32.11-windows-hpc-1
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.33.6-windows-hpc-1
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.34.3-windows-hpc-1
diff --git a/vhd_files/2025-gen2.txt b/vhd_files/2025-gen2.txt
index e27690e..3374953 100644
--- a/vhd_files/2025-gen2.txt
+++ b/vhd_files/2025-gen2.txt
@@ -3,0 +4,3 @@ c:\akse-cache\aks-secure-tls-bootstrap-client\: https://github.com/Azure/aks-sec
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.32.11/windows/amd64/azure-acr-credential-provider_1.32.11-1_amd64.zip
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.33.6/windows/amd64/azure-acr-credential-provider_1.33.6-1_amd64.zip
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.34.3/windows/amd64/azure-acr-credential-provider_1.34.3-1_amd64.zip
@@ -58 +61,3 @@ mcr.microsoft.com/oss/v2/kubernetes-csi/secrets-store/driver:v1.5.4
-mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.34.3
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.32.11-windows-hpc-1
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.33.6-windows-hpc-1
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.34.3-windows-hpc-1
diff --git a/vhd_files/2025.txt b/vhd_files/2025.txt
index fec8e92..677b345 100644
--- a/vhd_files/2025.txt
+++ b/vhd_files/2025.txt
@@ -3,0 +4,3 @@ c:\akse-cache\aks-secure-tls-bootstrap-client\: https://github.com/Azure/aks-sec
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.32.11/windows/amd64/azure-acr-credential-provider_1.32.11-1_amd64.zip
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.33.6/windows/amd64/azure-acr-credential-provider_1.33.6-1_amd64.zip
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.34.3/windows/amd64/azure-acr-credential-provider_1.34.3-1_amd64.zip
@@ -58 +61,3 @@ mcr.microsoft.com/oss/v2/kubernetes-csi/secrets-store/driver:v1.5.4
-mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.34.3
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.32.11-windows-hpc-1
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.33.6-windows-hpc-1
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.34.3-windows-hpc-1
diff --git a/vhd_files/23H2-gen2.txt b/vhd_files/23H2-gen2.txt
index 2440903..13d838a 100644
--- a/vhd_files/23H2-gen2.txt
+++ b/vhd_files/23H2-gen2.txt
@@ -3,0 +4,3 @@ c:\akse-cache\aks-secure-tls-bootstrap-client\: https://github.com/Azure/aks-sec
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.32.11/windows/amd64/azure-acr-credential-provider_1.32.11-1_amd64.zip
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.33.6/windows/amd64/azure-acr-credential-provider_1.33.6-1_amd64.zip
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.34.3/windows/amd64/azure-acr-credential-provider_1.34.3-1_amd64.zip
@@ -78 +81,3 @@ mcr.microsoft.com/oss/v2/kubernetes-csi/secrets-store/driver:v1.5.4
-mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.34.3
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.32.11-windows-hpc-1
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.33.6-windows-hpc-1
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.34.3-windows-hpc-1
diff --git a/vhd_files/23H2.txt b/vhd_files/23H2.txt
index 8664ed3..02092ca 100644
--- a/vhd_files/23H2.txt
+++ b/vhd_files/23H2.txt
@@ -3,0 +4,3 @@ c:\akse-cache\aks-secure-tls-bootstrap-client\: https://github.com/Azure/aks-sec
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.32.11/windows/amd64/azure-acr-credential-provider_1.32.11-1_amd64.zip
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.33.6/windows/amd64/azure-acr-credential-provider_1.33.6-1_amd64.zip
+c:\akse-cache\azure-acr-credential-provider\: https://packages.aks.azure.com/dalec-packages/azure-acr-credential-provider/1.34.3/windows/amd64/azure-acr-credential-provider_1.34.3-1_amd64.zip
@@ -78 +81,3 @@ mcr.microsoft.com/oss/v2/kubernetes-csi/secrets-store/driver:v1.5.4
-mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.34.3
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.32.11-windows-hpc-1
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.33.6-windows-hpc-1
+mcr.microsoft.com/oss/v2/kubernetes/azure-cloud-node-manager:v1.34.3-windows-hpc-1 |
What type of PR is this?
/kind feature
What this PR does / why we need it:
Migrate cloud-provider-azure >= v1.32.11 to use Dalec.
Which issue(s) this PR fixes:
Fixes #
Requirements:
Special notes for your reviewer:
Release note: