Relax enum restrictions on Storage bypass parameter (#4328)

Fixes #4247. Storage supports passing multiple values to the Bypass parameter separated by a comma, but that's not how Swagger enums work so the fact they've specified this field as an enum is technically incorrect. We just treat it as a string to allow the user to pass multiple values
Azure · Oct 9, 2024 · 67a21e7 · 67a21e7
1 parent fee7fed
commit 67a21e7
Show file tree

Hide file tree

Showing 20 changed files with 45 additions and 239 deletions.
diff --git a/v2/api/storage/v1api20210401/storage_account_spec_arm_types_gen.go b/v2/api/storage/v1api20210401/storage_account_spec_arm_types_gen.go
diff --git a/v2/api/storage/v1api20210401/storage_account_spec_arm_types_gen_test.go b/v2/api/storage/v1api20210401/storage_account_spec_arm_types_gen_test.go
diff --git a/v2/api/storage/v1api20210401/storage_account_types_gen.go b/v2/api/storage/v1api20210401/storage_account_types_gen.go
diff --git a/v2/api/storage/v1api20210401/storage_account_types_gen_test.go b/v2/api/storage/v1api20210401/storage_account_types_gen_test.go
diff --git a/v2/api/storage/v1api20210401/structure.txt b/v2/api/storage/v1api20210401/structure.txt
@@ -99,11 +99,7 @@ StorageAccount: Resource
 │   │   ├── "TLS1_1"
 │   │   └── "TLS1_2"
 │   ├── NetworkAcls: *Object (5 properties)
-│   │   ├── Bypass: *Enum (4 values)
-│   │   │   ├── "AzureServices"
-│   │   │   ├── "Logging"
-│   │   │   ├── "Metrics"
-│   │   │   └── "None"
+│   │   ├── Bypass: *string
 │   │   ├── DefaultAction: *Enum (2 values)
 │   │   │   ├── "Allow"
 │   │   │   └── "Deny"
@@ -721,11 +717,7 @@ StorageAccount_Spec_ARM: Object (8 properties)
 │   │   ├── "TLS1_1"
 │   │   └── "TLS1_2"
 │   ├── NetworkAcls: *Object (5 properties)
-│   │   ├── Bypass: *Enum (4 values)
-│   │   │   ├── "AzureServices"
-│   │   │   ├── "Logging"
-│   │   │   ├── "Metrics"
-│   │   │   └── "None"
+│   │   ├── Bypass: *string
 │   │   ├── DefaultAction: *Enum (2 values)
 │   │   │   ├── "Allow"
 │   │   │   └── "Deny"

diff --git a/v2/api/storage/v1api20210401/zz_generated.deepcopy.go b/v2/api/storage/v1api20210401/zz_generated.deepcopy.go
diff --git a/v2/api/storage/v1api20220901/storage_account_spec_arm_types_gen.go b/v2/api/storage/v1api20220901/storage_account_spec_arm_types_gen.go
diff --git a/v2/api/storage/v1api20220901/storage_account_spec_arm_types_gen_test.go b/v2/api/storage/v1api20220901/storage_account_spec_arm_types_gen_test.go
diff --git a/v2/api/storage/v1api20220901/storage_account_types_gen.go b/v2/api/storage/v1api20220901/storage_account_types_gen.go
diff --git a/v2/api/storage/v1api20220901/storage_account_types_gen_test.go b/v2/api/storage/v1api20220901/storage_account_types_gen_test.go
diff --git a/v2/api/storage/v1api20220901/structure.txt b/v2/api/storage/v1api20220901/structure.txt
@@ -125,11 +125,7 @@ StorageAccount: Resource
 │   │   ├── "TLS1_1"
 │   │   └── "TLS1_2"
 │   ├── NetworkAcls: *Object (5 properties)
-│   │   ├── Bypass: *Enum (4 values)
-│   │   │   ├── "AzureServices"
-│   │   │   ├── "Logging"
-│   │   │   ├── "Metrics"
-│   │   │   └── "None"
+│   │   ├── Bypass: *string
 │   │   ├── DefaultAction: *Enum (2 values)
 │   │   │   ├── "Allow"
 │   │   │   └── "Deny"
@@ -862,11 +858,7 @@ StorageAccount_Spec_ARM: Object (8 properties)
 │   │   ├── "TLS1_1"
 │   │   └── "TLS1_2"
 │   ├── NetworkAcls: *Object (5 properties)
-│   │   ├── Bypass: *Enum (4 values)
-│   │   │   ├── "AzureServices"
-│   │   │   ├── "Logging"
-│   │   │   ├── "Metrics"
-│   │   │   └── "None"
+│   │   ├── Bypass: *string
 │   │   ├── DefaultAction: *Enum (2 values)
 │   │   │   ├── "Allow"
 │   │   │   └── "Deny"

diff --git a/v2/api/storage/v1api20220901/zz_generated.deepcopy.go b/v2/api/storage/v1api20220901/zz_generated.deepcopy.go