diff --git a/src/Compute/Compute.Helpers/Storage/Models/BlobInventoryPolicy.cs b/src/Compute/Compute.Helpers/Storage/Models/BlobInventoryPolicy.cs
index a4225df4adaa..997938da2f40 100644
--- a/src/Compute/Compute.Helpers/Storage/Models/BlobInventoryPolicy.cs
+++ b/src/Compute/Compute.Helpers/Storage/Models/BlobInventoryPolicy.cs
@@ -43,6 +43,7 @@ public BlobInventoryPolicy()
         /// "Microsoft.Storage/storageAccounts"</param>
         /// <param name="lastModifiedTime">Returns the last modified date and
         /// time of the blob inventory policy.</param>
+        /// <param name="systemData"></param>
         public BlobInventoryPolicy(BlobInventoryPolicySchema policy, string id = default(string), string name = default(string), string type = default(string), System.DateTime? lastModifiedTime = default(System.DateTime?), SystemData systemData = default(SystemData))
             : base(id, name, type)
         {
diff --git a/src/Compute/Compute/Common/AzureContextAdapterExtensions.cs b/src/Compute/Compute/Common/AzureContextAdapterExtensions.cs
index b4fad370212d..ca885ed56607 100644
--- a/src/Compute/Compute/Common/AzureContextAdapterExtensions.cs
+++ b/src/Compute/Compute/Common/AzureContextAdapterExtensions.cs
@@ -29,6 +29,7 @@ public static class AzureContextAdapterExtensions
         /// Get the current storage account.
         /// </summary>
         /// <param name="context">The current Azure context.</param>
+        /// <param name="provider"></param>
         /// <returns>The current storage account, or null, if no current storage account is set.</returns>
         public static CloudStorageAccount GetCurrentStorageAccount(this IAzureContext context, IStorageServiceProvider provider)
         {
@@ -92,6 +93,7 @@ public static IStorageContext GetStorageContext(this IStorageService service)
         /// </summary>
         /// <param name="provider">The storage service provider to retrieve storage service details</param>
         /// <param name="accountName">The storage accoutn name</param>
+        /// <param name="resourceGroupName"></param>
         /// <returns>A CloudStorageAccount client for storage data plane tasks</returns>
         public static CloudStorageAccount GetCloudStorageAccount(this IStorageServiceProvider provider, string accountName, string resourceGroupName = null)
         {
diff --git a/src/Compute/Compute/Common/DiagnosticsHelper.cs b/src/Compute/Compute/Common/DiagnosticsHelper.cs
index 10cae96bc633..acabd01afc48 100644
--- a/src/Compute/Compute/Common/DiagnosticsHelper.cs
+++ b/src/Compute/Compute/Common/DiagnosticsHelper.cs
@@ -661,6 +661,8 @@ private static string GetEndpointFromStorageContext(IStorageContext context)
         /// </summary>
         /// <param name="publicConfigPath">Public configuration file path</param>
         /// <param name="privateConfigPath">Private configuration file path, can be empty</param>
+        /// <param name="resourceId"></param>
+        /// <param name="cmdlet"></param>
         /// <param name="storageClient">Storage client</param>
         /// <returns>A tuple with public configuration as first element and private configuration as second element</returns>
         public static Tuple<Hashtable, Hashtable> GetConfigurationsFromFiles(string publicConfigPath, string privateConfigPath, string resourceId, Cmdlet cmdlet, IStorageManagementClient storageClient)
diff --git a/src/Compute/Compute/Extension/AzureVMBackup/AzureVMBackupExtensionUtil.cs b/src/Compute/Compute/Extension/AzureVMBackup/AzureVMBackupExtensionUtil.cs
index 35763c093e1b..ca741a033cf8 100644
--- a/src/Compute/Compute/Extension/AzureVMBackup/AzureVMBackupExtensionUtil.cs
+++ b/src/Compute/Compute/Extension/AzureVMBackup/AzureVMBackupExtensionUtil.cs
@@ -73,10 +73,10 @@ private string GetBase64Encoding(object obj)
         /// <summary>
         /// find the snapshot with the tags
         /// </summary>
+        /// <param name="azContext"></param>
         /// <param name="blobUris"></param>
-        /// <param name="snapshotTag"></param>
-        /// <param name="taskId"></param>
         /// <param name="storageCredentialsFactory"></param>
+        /// <param name="snapshotQuery"></param>
         /// <returns></returns>
         public List<CloudPageBlob> FindSnapshot(IAzureContext azContext, List<string> blobUris, List<StorageCredentialsFactory> storageCredentialsFactory, Dictionary<string, string> snapshotQuery)
         {
@@ -178,13 +178,8 @@ public AzureVMBackupBlobSasUris GenerateBlobSasUris(List<string> blobUris, IAzur
         /// <summary>
         /// remove the vmbackups with the metadata  key "vmbackuptag" and value snapshotTag, snapshotTag is the parameter passed in.
         /// </summary>
-        /// <param name="resourceGroupName"></param>
-        /// <param name="vmName"></param>
-        /// <param name="virtualMachineExtensionType"></param>
-        /// <param name="location"></param>
-        /// <param name="virtualMachineResponse"></param>
-        /// <param name="profile"></param>
-        /// <param name="VirtualMachineExtensionClient"></param>
+        /// <param name="vmConfig"></param>
+        /// <param name="virtualMachineExtensionBaseCmdlet"></param>
         /// <param name="snapshotTag"></param>
         public void RemoveSnapshot(AzureVMBackupConfig vmConfig, string snapshotTag, VirtualMachineExtensionBaseCmdlet virtualMachineExtensionBaseCmdlet)
         {
@@ -216,11 +211,7 @@ public void RemoveSnapshot(AzureVMBackupConfig vmConfig, string snapshotTag, Vir
         /// <summary>
         /// we only support the Linux box now, if it's a windows, one AzureVMBackupException would be thrown.
         /// </summary>
-        /// <param name="resourceGroupName"></param>
-        /// <param name="vmName"></param>
-        /// <param name="virtualMachineExtensionType"></param>
-        /// <param name="location"></param>
-        /// <param name="virtualMachineResponse"></param>
+        /// <param name="vmConfig"></param>
         /// <param name="snapshotTag"></param>
         /// <param name="virtualMachineExtensionBaseCmdlet"></param>
         public void CreateSnapshotForDisks(AzureVMBackupConfig vmConfig, string snapshotTag, VirtualMachineExtensionBaseCmdlet virtualMachineExtensionBaseCmdlet)
diff --git a/src/Compute/Compute/StorageServices/VirtualHardDiskSettingData.cs b/src/Compute/Compute/StorageServices/VirtualHardDiskSettingData.cs
index 349259890f6f..7f54cb47b5bf 100644
--- a/src/Compute/Compute/StorageServices/VirtualHardDiskSettingData.cs
+++ b/src/Compute/Compute/StorageServices/VirtualHardDiskSettingData.cs
@@ -83,6 +83,7 @@ public class VirtualHardDiskSettingData
         /// </param>
         /// <param name="blockSize">The block size of the virtual hard disk.</param>
         /// <param name="logicalSectorSize">The logical sector size of the virtual hard disk.
+        /// </param>
         /// <param name="physicalSectorSize">The physical sector size of the virtual hard disk.
         /// </param>
         public