Skip to content

Commit 53ad912

Browse files
Merge pull request #272 from MabOneSdk/swatim-dev1
Removing SCDPM from BackupManagementType list in Get Container.
2 parents b78baed + 7ebc459 commit 53ad912

File tree

3 files changed

+9
-8
lines changed

3 files changed

+9
-8
lines changed

src/ResourceManager/RecoveryServices.Backup/Commands.RecoveryServices.Backup.Models/CommonModels/Enums.cs

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -30,8 +30,7 @@ public enum ContainerType
3030
public enum BackupManagementType
3131
{
3232
AzureVM = 1,
33-
MARS,
34-
SCDPM
33+
MARS
3534
}
3635

3736
public enum BackupEngineType

src/ResourceManager/RecoveryServices.Backup/Commands.RecoveryServices.Backup.Providers/PsBackupProviderManager.cs

Lines changed: 6 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -48,10 +48,8 @@ public IPsBackupProvider GetProviderInstance(ContainerType containerType, Backup
4848
case ContainerType.Windows:
4949
if (backupManagementType == BackupManagementType.MARS)
5050
providerType = PsBackupProviderTypes.Mab;
51-
else if (backupManagementType == BackupManagementType.SCDPM)
52-
providerType = PsBackupProviderTypes.Dpm;
5351
else
54-
throw new ArgumentException(String.Format("BackupManagementType {0} is not expected for ContainerType {1}", backupManagementType.ToString(), containerType));
52+
throw new ArgumentException(String.Format("BackupManagementType is required for ContainerType {1}.", backupManagementType.ToString(), containerType));
5553
break;
5654
//case ContainerType.AzureSqlContainer:
5755
// if (backupManagementType.HasValue)
@@ -68,6 +66,11 @@ public IPsBackupProvider GetProviderInstance(ContainerType containerType, Backup
6866
return GetProviderInstance(providerType);
6967
}
7068

69+
public IPsBackupProvider GetProviderInstanceForBackupManagementServer()
70+
{
71+
return GetProviderInstance(PsBackupProviderTypes.Dpm);
72+
}
73+
7174
public IPsBackupProvider GetProviderInstance(WorkloadType workloadType)
7275
{
7376
PsBackupProviderTypes providerType = 0;

src/ResourceManager/RecoveryServices.Backup/Commands.RecoveryServices.Backup/Cmdlets/Container/GetAzureRmRecoveryServicesBackupManagementServer.cs

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -42,12 +42,11 @@ public override void ExecuteCmdlet()
4242

4343
PsBackupProviderManager providerManager = new PsBackupProviderManager(new Dictionary<System.Enum, object>()
4444
{
45-
{ContainerParams.ContainerType, ContainerType.Windows},
46-
{ContainerParams.BackupManagementType, BackupManagementType.SCDPM},
45+
{ContainerParams.ContainerType, ContainerType.Windows},
4746
{ContainerParams.Name, Name}
4847
}, HydraAdapter);
4948

50-
IPsBackupProvider psBackupProvider = providerManager.GetProviderInstance(ContainerType.Windows, BackupManagementType.SCDPM);
49+
IPsBackupProvider psBackupProvider = providerManager.GetProviderInstanceForBackupManagementServer();
5150

5251
var backupServerModels = psBackupProvider.ListBackupManagementServers();
5352
if (!string.IsNullOrEmpty(this.Name))

0 commit comments

Comments
 (0)