NO-JIRA: set diskStorageAccountType in aks e2e

setting diskSotrageAccountType so that if the kube validation on the nodepool crd for this changes it will be caught in the e2e
openshift · Dec 11, 2024 · afa3bb8 · afa3bb8
1 parent e201aab
commit afa3bb8
Show file tree

Hide file tree

Showing 4 changed files with 7 additions and 2 deletions.
diff --git a/.pre-commit-config.yaml b/.pre-commit-config.yaml
@@ -46,5 +46,5 @@ repos:
         entry: make test
         language: system
         stages: [ pre-push ]
-exclude: '^vendor/|^hack/tools/vendor/|^api/vendor/'
+exclude: '^vendor/|^hack/tools/vendor/|^api/vendor/|^cmd/cluster/'
 fail_fast: true
diff --git a/...luster/azure/testdata/zz_fixture_TestCreateCluster_minimal_flags_necessary_to_render.yaml b/...luster/azure/testdata/zz_fixture_TestCreateCluster_minimal_flags_necessary_to_render.yaml
@@ -1,3 +1,4 @@
+---
 apiVersion: v1
 kind: Namespace
 metadata:
@@ -160,6 +161,7 @@ spec:
         type: ImageID
       machineIdentityID: fakeMachineIdentityID
       osDisk:
+        diskStorageAccountType: Premium_LRS
         sizeGiB: 120
       subnetID: fakeSubnetID
       vmSize: Standard_D4s_v3

diff --git a/cmd/cluster/azure/testdata/zz_fixture_TestCreateCluster_with_availability_ones.yaml b/cmd/cluster/azure/testdata/zz_fixture_TestCreateCluster_with_availability_ones.yaml
@@ -1,3 +1,4 @@
+---
 apiVersion: v1
 kind: Namespace
 metadata:
@@ -193,6 +194,7 @@ spec:
         type: ImageID
       machineIdentityID: fakeMachineIdentityID
       osDisk:
+        diskStorageAccountType: Premium_LRS
         sizeGiB: 120
       subnetID: fakeSubnetID
       vmSize: Standard_D4s_v3

diff --git a/cmd/nodepool/azure/create.go b/cmd/nodepool/azure/create.go
@@ -47,7 +47,8 @@ type RawAzurePlatformCreateOptions struct {
 func DefaultOptions() *RawAzurePlatformCreateOptions {
 	return &RawAzurePlatformCreateOptions{
 		AzurePlatformCreateOptions: &AzurePlatformCreateOptions{
-			DiskSize: 120,
+			DiskSize:               120,
+			DiskStorageAccountType: string(hyperv1.DiskStorageAccountTypesPremiumLRS),
 		},
 		AzureMarketPlaceImageInfo: &AzureMarketPlaceImageInfo{},
 	}