Skip to content

Commit 986da78

Browse files
authored
[Storage] Removed StorageFileDataSmbShareOwner from value set of DefaultSharePermission (#15190)
1 parent e1f82c9 commit 986da78

File tree

8 files changed

+15
-12
lines changed

8 files changed

+15
-12
lines changed

src/Storage/Storage.Management.Test/ScenarioTests/StorageAccountTests.ps1

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -998,21 +998,21 @@ function Test-NewSetAzStorageAccountFileAADDS
998998
Assert-AreEqual 'AADDS' $sto.AzureFilesIdentityBasedAuth.DirectoryServiceOptions;
999999
Assert-AreEqual 'StorageFileDataSmbShareContributor' $sto.AzureFilesIdentityBasedAuth.DefaultSharePermission;
10001000

1001-
$sto = Set-AzStorageAccount -ResourceGroupName $rgname -Name $stoname -EnableAzureActiveDirectoryDomainServicesForFile $false -DefaultSharePermission StorageFileDataSmbShareOwner
1001+
$sto = Set-AzStorageAccount -ResourceGroupName $rgname -Name $stoname -EnableAzureActiveDirectoryDomainServicesForFile $false -DefaultSharePermission StorageFileDataSmbShareReader
10021002
Assert-AreEqual $stoname $sto.StorageAccountName;
10031003
Assert-AreEqual $stotype $sto.Sku.Name;
10041004
Assert-AreEqual $loc.ToLower().Replace(" ", "") $sto.Location;
10051005
Assert-AreEqual $kind $sto.Kind;
10061006
Assert-AreEqual 'None' $sto.AzureFilesIdentityBasedAuth.DirectoryServiceOptions;
1007-
Assert-AreEqual 'StorageFileDataSmbShareOwner' $sto.AzureFilesIdentityBasedAuth.DefaultSharePermission;
1007+
Assert-AreEqual 'StorageFileDataSmbShareReader' $sto.AzureFilesIdentityBasedAuth.DefaultSharePermission;
10081008

10091009
$sto = Get-AzStorageAccount -ResourceGroupName $rgname -Name $stoname;
10101010
Assert-AreEqual $stoname $sto.StorageAccountName;
10111011
Assert-AreEqual $stotype $sto.Sku.Name;
10121012
Assert-AreEqual $loc.ToLower().Replace(" ", "") $sto.Location;
10131013
Assert-AreEqual $kind $sto.Kind;
10141014
Assert-AreEqual 'None' $sto.AzureFilesIdentityBasedAuth.DirectoryServiceOptions;
1015-
Assert-AreEqual 'StorageFileDataSmbShareOwner' $sto.AzureFilesIdentityBasedAuth.DefaultSharePermission;
1015+
Assert-AreEqual 'StorageFileDataSmbShareReader' $sto.AzureFilesIdentityBasedAuth.DefaultSharePermission;
10161016

10171017
Retry-IfException { Remove-AzureRmStorageAccount -Force -ResourceGroupName $rgname -Name $stoname; }
10181018
}

src/Storage/Storage.Management.Test/SessionRecords/Microsoft.Azure.Commands.Management.Storage.Test.ScenarioTests.StorageAccountTests/TestNewSetAzStorageAccountFileAADDS.json

Lines changed: 5 additions & 5 deletions
Large diffs are not rendered by default.

src/Storage/Storage.Management/ChangeLog.md

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,9 @@
2424
* Supported enable Smb Multichannel on File service
2525
- `Update-AzStorageFileServiceProperty`
2626
* Fixed copy inside same account issue by access source with anonymous credential, when copy Blob inside same account with Oauth credential
27+
* Removed StorageFileDataSmbShareOwner from value set of parameter DefaultSharePermission in create/udpate storage account
28+
- `New-AzStorageAccount`
29+
- `Set-AzStorageAccount`
2730

2831
## Version 3.7.0
2932
* Supported file share snapshot

src/Storage/Storage.Management/StorageAccount/NewAzureStorageAccount.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -467,7 +467,6 @@ public bool AllowCrossTenantReplication
467467
DefaultSharePermissionType.StorageFileDataSmbShareContributor,
468468
DefaultSharePermissionType.StorageFileDataSmbShareReader,
469469
DefaultSharePermissionType.StorageFileDataSmbShareElevatedContributor,
470-
DefaultSharePermissionType.StorageFileDataSmbShareOwner,
471470
IgnoreCase = true)]
472471
public string DefaultSharePermission { get; set; }
473472

src/Storage/Storage.Management/StorageAccount/SetAzureStorageAccount.cs

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -448,7 +448,6 @@ public bool AllowCrossTenantReplication
448448
DefaultSharePermissionType.StorageFileDataSmbShareContributor,
449449
DefaultSharePermissionType.StorageFileDataSmbShareReader,
450450
DefaultSharePermissionType.StorageFileDataSmbShareElevatedContributor,
451-
DefaultSharePermissionType.StorageFileDataSmbShareOwner,
452451
IgnoreCase = true)]
453452
public string DefaultSharePermission { get; set; }
454453

src/Storage/Storage.Management/help/New-AzStorageAccount.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -467,7 +467,7 @@ Default share permission for users using Kerberos authentication if RBAC role is
467467
Type: System.String
468468
Parameter Sets: (All)
469469
Aliases:
470-
Accepted values: None, StorageFileDataSmbShareContributor, StorageFileDataSmbShareReader, StorageFileDataSmbShareElevatedContributor, StorageFileDataSmbShareOwner
470+
Accepted values: None, StorageFileDataSmbShareContributor, StorageFileDataSmbShareReader, StorageFileDataSmbShareElevatedContributor
471471

472472
Required: False
473473
Position: Named

src/Storage/Storage.Management/help/Set-AzStorageAccount.md

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -541,7 +541,7 @@ Default share permission for users using Kerberos authentication if RBAC role is
541541
Type: System.String
542542
Parameter Sets: (All)
543543
Aliases:
544-
Accepted values: None, StorageFileDataSmbShareContributor, StorageFileDataSmbShareReader, StorageFileDataSmbShareElevatedContributor, StorageFileDataSmbShareOwner
544+
Accepted values: None, StorageFileDataSmbShareContributor, StorageFileDataSmbShareReader, StorageFileDataSmbShareElevatedContributor
545545

546546
Required: False
547547
Position: Named

tools/StaticAnalysis/Exceptions/Az.Storage/BreakingChangeIssues.csv

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -10,3 +10,5 @@
1010
"Az.Storage","Microsoft.WindowsAzure.Commands.Storage.File.Cmdlet.SetAzureStorageFileContent","Set-AzStorageFileContent","0","1050","The parameter set 'Share' for cmdlet 'Set-AzStorageFileContent' has been removed.","Add parameter set 'Share' back to cmdlet 'Set-AzStorageFileContent'."
1111
"Az.Storage","Microsoft.WindowsAzure.Commands.Storage.File.Cmdlet.SetAzureStorageFileContent","Set-AzStorageFileContent","0","1050","The parameter set 'Directory' for cmdlet 'Set-AzStorageFileContent' has been removed.","Add parameter set 'Directory' back to cmdlet 'Set-AzStorageFileContent'."
1212
"Az.Storage","Microsoft.WindowsAzure.Commands.Storage.File.Cmdlet.SetAzureStorageFileContent","Set-AzStorageFileContent","0","1050","The parameter set '__AllParameterSets' for cmdlet 'Set-AzStorageFileContent' has been removed.","Add parameter set '__AllParameterSets' back to cmdlet 'Set-AzStorageFileContent'."
13+
"Az.Storage","Microsoft.Azure.Commands.Management.Storage.NewAzureStorageAccountCommand","New-AzStorageAccount","0","2040","The validation set for parameter 'DefaultSharePermission' for cmdlet 'New-AzStorageAccount' no longer contains the value 'StorageFileDataSmbShareOwner'.","Add 'StorageFileDataSmbShareOwner' back to the validation set for 'DefaultSharePermission'."
14+
"Az.Storage","Microsoft.Azure.Commands.Management.Storage.SetAzureStorageAccountCommand","Set-AzStorageAccount","0","2040","The validation set for parameter 'DefaultSharePermission' for cmdlet 'Set-AzStorageAccount' no longer contains the value 'StorageFileDataSmbShareOwner'.","Add 'StorageFileDataSmbShareOwner' back to the validation set for 'DefaultSharePermission'."

0 commit comments

Comments
 (0)