Skip to content

Commit f26f8cd

Browse files
author
dragonfly91
committed
Fix for bug 3892210: Get-AzureBackupContainer with NotRegistered filter gives registered containers
1 parent 81dd937 commit f26f8cd

File tree

1 file changed

+11
-0
lines changed

1 file changed

+11
-0
lines changed

src/ResourceManager/AzureBackup/Commands.AzureBackup/Cmdlets/Container/GetAzureBackupContainer.cs

Lines changed: 11 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,17 @@ public override void ExecuteCmdlet()
8282
private List<AzureBackupContainer> GetMachineContainers()
8383
{
8484
List<MarsContainerResponse> marsContainerResponses = new List<MarsContainerResponse>();
85+
86+
// Machine containers are always registered.
87+
// So if requested Status is not Registered, return an empty list.
88+
// Machine containers don't have a resource group.
89+
// So, if a resource group is passed, return an empty list.
90+
if (Status != AzureBackupContainerRegistrationStatus.Registered ||
91+
!string.IsNullOrEmpty(ManagedResourceGroupName))
92+
{
93+
return new List<AzureBackupContainer>();
94+
}
95+
8596
if (string.IsNullOrEmpty(Name))
8697
{
8798
marsContainerResponses.AddRange(AzureBackupClient.ListMachineContainers());

0 commit comments

Comments
 (0)