[RS Backup] Fixing 7349803 (documentation fixes) in Provider Layer

Author: dragonfly91

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

@@ -32,6 +32,11 @@ public class DpmPsBackupProvider : IPsBackupProvider
         Dictionary<System.Enum, object> ProviderData { get; set; }
         ServiceClientAdapter ServiceClientAdapter { get; set; }
 
+        /// <summary>
+        /// Initializes the provider with the data recieved from the cmdlet layer
+        /// </summary>
+        /// <param name="providerData">Data from the cmdlet layer intended for the provider</param>
+        /// <param name="serviceClientAdapter">Service client adapter for communicating with the backend service</param>
         public void Initialize(Dictionary<System.Enum, object> providerData, ServiceClientAdapter serviceClientAdapter)
         {
             this.ProviderData = providerData;
@@ -88,6 +93,10 @@ public ServiceClientModel.ProtectionPolicyResponse ModifyPolicy()
             throw new NotImplementedException();
         }
 
+        /// <summary>
+        /// Lists backup management servers registered with the recovery services vault
+        /// </summary>
+        /// <returns></returns>
         public List<Models.BackupEngineBase> ListBackupManagementServers()
         {
             string name = (string)this.ProviderData[ContainerParams.Name];

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

@@ -41,12 +41,21 @@ public class IaasVmPsBackupProvider : IPsBackupProvider
         Dictionary<System.Enum, object> ProviderData { get; set; }
         ServiceClientAdapter ServiceClientAdapter { get; set; }
 
+        /// <summary>
+        /// Initializes the provider with the data recieved from the cmdlet layer
+        /// </summary>
+        /// <param name="providerData">Data from the cmdlet layer intended for the provider</param>
+        /// <param name="serviceClientAdapter">Service client adapter for communicating with the backend service</param>
         public void Initialize(Dictionary<System.Enum, object> providerData, ServiceClientAdapter serviceClientAdapter)
         {
             this.ProviderData = providerData;
             this.ServiceClientAdapter = serviceClientAdapter;
         }
 
+        /// <summary>
+        /// Triggers the enable protection operation for the given item
+        /// </summary>
+        /// <returns>The job response returned from the service</returns>
         public BaseRecoveryServicesJobResponse EnableProtection()
         {
             string azureVMName = (string)ProviderData[ItemParams.AzureVMName];
@@ -130,6 +139,10 @@ public BaseRecoveryServicesJobResponse EnableProtection()
                                 serviceClientRequest);
         }
 
+        /// <summary>
+        /// Triggers the disable protection operation for the given item
+        /// </summary>
+        /// <returns>The job response returned from the service</returns>
         public BaseRecoveryServicesJobResponse DisableProtection()
         {
             bool deleteBackupData = (bool)ProviderData[ItemParams.DeleteBackupData];
@@ -189,6 +202,10 @@ public BaseRecoveryServicesJobResponse DisableProtection()
             }
         }
 
+        /// <summary>
+        /// Triggers the backup operation for the given item
+        /// </summary>
+        /// <returns>The job response returned from the service</returns>
         public BaseRecoveryServicesJobResponse TriggerBackup()
         {
             ItemBase item = (ItemBase)ProviderData[ItemParams.Item];
@@ -197,6 +214,10 @@ public BaseRecoveryServicesJobResponse TriggerBackup()
                 IdUtils.GetValueByName(iaasVmItem.Id, IdUtils.IdNames.ProtectedItemName));
         }
 
+        /// <summary>
+        /// Triggers the recovery operation for the given recovery point
+        /// </summary>
+        /// <returns>The job response returned from the service</returns>
         public BaseRecoveryServicesJobResponse TriggerRestore()
         {
             AzureVmRecoveryPoint rp = ProviderData[RestoreBackupItemParams.RecoveryPoint]
@@ -216,6 +237,10 @@ public ProtectedItemResponse GetProtectedItem()
             throw new NotImplementedException();
         }
 
+        /// <summary>
+        /// Fetches the detail info for the given recovery point
+        /// </summary>
+        /// <returns>Recovery point detail as returned by the service</returns>
         public CmdletModel.RecoveryPointBase GetRecoveryPointDetails()
         {
             AzureVmItem item = ProviderData[GetRecoveryPointParams.Item]
@@ -231,6 +256,10 @@ public CmdletModel.RecoveryPointBase GetRecoveryPointDetails()
             return RecoveryPointConversions.GetPSAzureRecoveryPoints(rpResponse, item);
         }
 
+        /// <summary>
+        /// Lists recovery points generated for the given item
+        /// </summary>
+        /// <returns>List of recovery point PowerShell model objects</returns>
         public List<CmdletModel.RecoveryPointBase> ListRecoveryPoints()
         {
             DateTime startDate = (DateTime)(ProviderData[GetRecoveryPointParams.StartDate]);
@@ -258,6 +287,10 @@ public CmdletModel.RecoveryPointBase GetRecoveryPointDetails()
             return RecoveryPointConversions.GetPSAzureRecoveryPoints(rpListResponse, item);
         }
 
+        /// <summary>
+        /// Creates policy given the provider data
+        /// </summary>
+        /// <returns>Created policy object as returned by the service</returns>
         public ProtectionPolicyResponse CreatePolicy()
         {
             string policyName = (string)ProviderData[PolicyParams.PolicyName];
@@ -312,6 +345,10 @@ public ProtectionPolicyResponse CreatePolicy()
                                  serviceClientRequest);
         }
 
+        /// <summary>
+        /// Modifies policy using the provider data
+        /// </summary>
+        /// <returns>Modified policy object as returned by the service</returns>
         public ProtectionPolicyResponse ModifyPolicy()
         {
             RetentionPolicyBase retentionPolicy =
@@ -383,6 +420,10 @@ public ProtectionPolicyResponse ModifyPolicy()
                                                                serviceClientRequest);
         }
 
+        /// <summary>
+        /// Lists protection containers according to the provider data
+        /// </summary>
+        /// <returns>List of protection containers</returns>
         public List<ContainerBase> ListProtectionContainers()
         {
             Models.ContainerType containerType = 
@@ -433,6 +474,10 @@ public List<ContainerBase> ListProtectionContainers()
             throw new NotImplementedException();
         }
 
+        /// <summary>
+        /// Lists protected items protected by the recovery services vault according to the provider data
+        /// </summary>
+        /// <returns>List of protected items</returns>
         public List<ItemBase> ListProtectedItems()
         {
             ContainerBase container =
@@ -550,6 +595,10 @@ public List<ItemBase> ListProtectedItems()
             return itemModels;
         }
 
+        /// <summary>
+        /// Constructs the schedule policy object with default inits
+        /// </summary>
+        /// <returns>Default schedule policy object</returns>
         public SchedulePolicyBase GetDefaultSchedulePolicyObject()
         {
             CmdletModel.SimpleSchedulePolicy defaultSchedule = new CmdletModel.SimpleSchedulePolicy();
@@ -566,6 +615,10 @@ public SchedulePolicyBase GetDefaultSchedulePolicyObject()
             return defaultSchedule;
         }
 
+        /// <summary>
+        /// Constructs the retention policy object with default inits
+        /// </summary>
+        /// <returns>Default retention policy object</returns>
         public RetentionPolicyBase GetDefaultRetentionPolicyObject()
         {
             CmdletModel.LongTermRetentionPolicy defaultRetention = new CmdletModel.LongTermRetentionPolicy();

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

@@ -34,6 +34,11 @@ public class MabPsBackupProvider : IPsBackupProvider
         Dictionary<System.Enum, object> ProviderData { get; set; }
         ServiceClientAdapter ServiceClientAdapter { get; set; }
 
+        /// <summary>
+        /// Initializes the provider with the data recieved from the cmdlet layer
+        /// </summary>
+        /// <param name="providerData">Data from the cmdlet layer intended for the provider</param>
+        /// <param name="serviceClientAdapter">Service client adapter for communicating with the backend service</param>
         public void Initialize(Dictionary<System.Enum, object> providerData, ServiceClientAdapter serviceClientAdapter)
         {
             this.ProviderData = providerData;
@@ -85,6 +90,10 @@ public ProtectionPolicyResponse ModifyPolicy()
             throw new NotImplementedException();
         }
 
+        /// <summary>
+        /// Lists containers registered to the recovery services vault according to the provider data
+        /// </summary>
+        /// <returns>List of protection containers</returns>
         public List<Models.ContainerBase> ListProtectionContainers()
         {
             string name = (string)this.ProviderData[ContainerParams.Name];

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

@@ -45,6 +45,12 @@ public PsBackupProviderManager(Dictionary<System.Enum, object> providerDataIn, S
             serviceClientAdapter = serviceClientAdapterIn;
         }
 
+        /// <summary>
+        /// Gets an instance of the provider based on the container type and backup management type (optional)
+        /// </summary>
+        /// <param name="containerType">Type of the container</param>
+        /// <param name="backupManagementType">Type of the backup management type (optional)</param>
+        /// <returns></returns>
         public IPsBackupProvider GetProviderInstance
             (
             ContainerType containerType,