From e38a8557d2bd47b4b87bf4b0db702f6936599285 Mon Sep 17 00:00:00 2001 From: Ziyue Zheng Date: Wed, 22 Jun 2022 14:30:06 +0800 Subject: [PATCH] Fix Az.Sql XML comments (#18576) * Fix Az.Sql XML comments * Fix Az.Sql XML comments * Update DatabaseEditions.cs --- .../Generated/Models/DatabaseEditions.cs | 14 +- ...dInstanceAdvancedDataSecurityCmdletBase.cs | 1 - .../BuildAzureSqlDatabaseDataMaskingRule.cs | 4 +- .../SqlDatabaseDataMaskingRuleCmdletBase.cs | 2 +- .../Data Sync/Model/AzureSqlSyncAgentModel.cs | 1 + .../AzureSqlSyncGroupSchemaColumnModel.cs | 2 +- .../Model/AzureSqlSyncGroupSchemaModel.cs | 2 +- .../AzureSqlSyncGroupSchemaTableModel.cs | 2 +- .../Model/AzureSqlSyncMemberModel.cs | 5 +- .../Services/AzureSqlDataSyncAdapter.cs | 12 +- .../SqlAzureDatabaseActivationAdapter.cs | 3 +- .../SqlAzureDatabaseActivationCommunicator.cs | 3 +- ...AzureSqlDatabaseLongTermRetentionBackup.cs | 2 +- .../Cmdlet/RestoreAzureRMSqlDatabase.cs | 2 +- ...baseBackupShortTermRetentionPolicyModel.cs | 1 + .../Services/AzureSqlDatabaseBackupAdapter.cs | 46 ++---- .../AzureSqlDatabaseBackupCommunicator.cs | 35 ++-- .../Services/AzureSqlDatabaseCommunicator.cs | 3 +- .../AzureSqlManagedInstanceLinkAdapter.cs | 4 +- ...AzureSqlManagedInstanceLinkCommunicator.cs | 3 +- .../AzureSqlElasticJobAgentCmdletBase.cs | 2 +- .../AzureSqlElasticJobCredentialCmdletBase.cs | 2 +- .../GetAzureSqlElasticJobCredential.cs | 2 +- .../RemoveAzureSqlElasticJobCredential.cs | 2 +- .../Job/AzureSqlElasticJobCmdletBase.cs | 2 +- .../AzureSqlElasticJobExecutionCmdletBase.cs | 2 +- .../AzureSqlElasticJobStepCmdletBase.cs | 2 +- .../AzureSqlElasticJobTargetCmdletBase.cs | 3 +- ...AzureSqlElasticJobTargetGroupCmdletBase.cs | 3 +- .../GetAzureSqlElasticJobTargetGroup.cs | 2 +- .../Services/AzureSqlElasticJobAdapter.cs | 50 +++++- .../AzureSqlElasticJobCommunicator.cs | 12 +- .../Services/AzureSqlElasticPoolAdapter.cs | 9 +- .../AzureSqlElasticPoolCommunicator.cs | 3 +- ...reSqlInstanceEndpointCertificateAdapter.cs | 6 +- ...InstanceEndpointCertificateCommunicator.cs | 3 +- .../Model/AzureSqlFailoverGroupModel.cs | 2 +- .../Services/AzureSqlFailoverGroupAdapter.cs | 21 +-- .../AzureSqlFailoverGroupCommunicator.cs | 3 +- .../AzureSqlServerFirewallRuleAdapter.cs | 5 +- .../AzureSqlServerFirewallRuleCommunicator.cs | 3 +- .../GetAzureSqlDatabaseImportExportStatus.cs | 1 - .../Service/ImportExportDatabaseAdapter.cs | 5 +- .../ImportExportDatabaseCommunicator.cs | 3 +- ...reSqlDatabaseIndexRecommendationAdapter.cs | 3 +- ...DatabaseIndexRecommendationCommunicator.cs | 3 +- ...AzureSqlInstanceFailoverGroupCmdletBase.cs | 1 - .../AzureSqlInstanceFailoverGroupAdapter.cs | 15 +- ...ureSqlInstanceFailoverGroupCommunicator.cs | 3 +- .../Cmdlet/SetAzureSqlInstancePool.cs | 2 +- ...anceActiveDirectoryAdministratorAdapter.cs | 5 +- ...ctiveDirectoryAdministratorCommunicator.cs | 3 +- ...ctiveDirectoryOnlyAuthenticationAdapter.cs | 8 +- ...DirectoryOnlyAuthenticationCommunicator.cs | 3 +- ...ureSqlDatabaseLedgerDigestUploadAdapter.cs | 18 +- ...lDatabaseLedgerDigestUploadCommunicator.cs | 3 +- .../Services/AzureSqlCapabilitiesAdapter.cs | 5 +- .../AzureSqlCapabilitiesCommunicator.cs | 3 +- ...baseBackupShortTermRetentionPolicyModel.cs | 2 +- .../AzureSqlManagedDatabaseBackupAdapter.cs | 11 +- ...ureSqlManagedDatabaseBackupCommunicator.cs | 9 +- ...SqlRecoverableManagedDatabaseCmdletBase.cs | 1 - .../AzureSqlManagedDatabaseAdapter.cs | 8 +- .../AzureSqlManagedDatabaseCommunicator.cs | 14 +- ...ureSqlRecoverableManagedDatabaseAdapter.cs | 5 +- ...lRecoverableManagedDatabaseCommunicator.cs | 3 +- .../Cmdlet/NewAzureSqlManagedInstance.cs | 6 +- .../Cmdlet/SetAzureSqlManagedInstance.cs | 6 +- .../AzureSqlManagedInstanceAdapter.cs | 6 +- .../AzureSqlManagedInstanceCommunicator.cs | 3 +- ...SqlManagedInstanceOperationCommunicator.cs | 3 +- ...ServerOutboundFirewallRulesCommunicator.cs | 3 +- src/Sql/Sql/Properties/Resources.Designer.cs | 4 +- ...zureSqlElasticPoolRecommendationAdapter.cs | 3 +- ...qlElasticPoolRecommendationCommunicator.cs | 3 +- .../AzureSqlDatabaseReplicationAdapter.cs | 14 +- ...AzureSqlDatabaseReplicationCommunicator.cs | 3 +- .../Sql/Server/Cmdlet/NewAzureSqlServer.cs | 4 +- .../Sql/Server/Cmdlet/SetAzureSqlServer.cs | 4 +- .../Server/Services/AzureSqlServerAdapter.cs | 8 +- .../Services/AzureSqlServerCommunicator.cs | 3 +- ...rverActiveDirectoryAdministratorAdapter.cs | 5 +- ...ctiveDirectoryAdministratorCommunicator.cs | 3 +- ...ctiveDirectoryOnlyAuthenticationAdapter.cs | 8 +- ...DirectoryOnlyAuthenticationCommunicator.cs | 3 +- .../AzureSqlServerCommunicationLinkAdapter.cs | 7 +- ...eSqlServerCommunicationLinkCommunicator.cs | 3 +- ...verDisasterRecoveryConfigurationAdapter.cs | 3 +- .../AzureSqlServerDnsAliasCommunicator.cs | 155 +++++++++--------- .../AzureSqlServerKeyVaultKeyAdapter.cs | 1 + ...qlInstanceServerTrustCertificateAdapter.cs | 2 + ...tanceServerTrustCertificateCommunicator.cs | 3 +- .../AzureSqlServerServiceObjectiveAdapter.cs | 3 +- .../AzureSqlServiceTierAdvisorAdapter.cs | 3 +- .../AzureSqlServiceTierAdvisorCommunicator.cs | 4 +- ...AddAzureRmSqlManagedInstanceKeyVaultKey.cs | 2 +- ...zureRmSqlManagedInstanceKeyVaultKeyBase.cs | 1 - ...eTransparentDataEncryptionProtectorBase.cs | 1 - ...atabaseTransparentDataEncryptionAdapter.cs | 6 +- ...baseTransparentDataEncryptionArmAdapter.cs | 3 +- ...AzureSqlServerVirtualNetworkRuleAdapter.cs | 5 +- ...SqlServerVirtualNetworkRuleCommunicator.cs | 25 ++- 102 files changed, 348 insertions(+), 394 deletions(-) diff --git a/src/Sql/Sql.LegacySdk/Generated/Models/DatabaseEditions.cs b/src/Sql/Sql.LegacySdk/Generated/Models/DatabaseEditions.cs index 255c94bb71b9..9e1a950bef37 100644 --- a/src/Sql/Sql.LegacySdk/Generated/Models/DatabaseEditions.cs +++ b/src/Sql/Sql.LegacySdk/Generated/Models/DatabaseEditions.cs @@ -35,33 +35,33 @@ public static partial class DatabaseEditions /// and workgroup or departmental applications. /// public const string Web = "Web"; - + /// /// The Business Edition Database is best suited for independent /// software vendors (ISVs), line-of-business (LOB) applications, and /// enterprise applications. /// public const string Business = "Business"; - + /// /// The Basic Edition Database is best suited for small databases with - /// few transactions (<100) per minute and only a couple users. + /// few transactions (<100) per minute and only a couple users. /// public const string Basic = "Basic"; - + /// /// The Standard Edition Database is best suited for a group of users /// makingfewer than 1000 transactions per minute. /// public const string Standard = "Standard"; - + /// /// The Premium Edition Database is for situations requiring a high - /// number of transactionsper second (>100). Supports more users than + /// number of transactionsper second (>100). Supports more users than /// Basic and Standard editions. /// public const string Premium = "Premium"; - + /// /// The Free Edition Database is available for certain offers and /// product bundles. diff --git a/src/Sql/Sql/AdvancedThreatProtection/Cmdlet/SqlManagedInstanceAdvancedDataSecurityCmdletBase.cs b/src/Sql/Sql/AdvancedThreatProtection/Cmdlet/SqlManagedInstanceAdvancedDataSecurityCmdletBase.cs index 5518ef118a4a..c441817d94ac 100644 --- a/src/Sql/Sql/AdvancedThreatProtection/Cmdlet/SqlManagedInstanceAdvancedDataSecurityCmdletBase.cs +++ b/src/Sql/Sql/AdvancedThreatProtection/Cmdlet/SqlManagedInstanceAdvancedDataSecurityCmdletBase.cs @@ -71,7 +71,6 @@ protected override ManagedInstanceAdvancedDataSecurityPolicyModel GetEntity() /// /// Creation and initialization of the ModelAdapter object /// - /// The AzureSubscription in which the current execution is performed /// An initialized and ready to use ModelAdapter object protected override SqlAdvancedDataSecurityAdapter InitModelAdapter() { diff --git a/src/Sql/Sql/Data Masking/Cmdlet/BuildAzureSqlDatabaseDataMaskingRule.cs b/src/Sql/Sql/Data Masking/Cmdlet/BuildAzureSqlDatabaseDataMaskingRule.cs index a2bd130ea9a2..236e31df9851 100644 --- a/src/Sql/Sql/Data Masking/Cmdlet/BuildAzureSqlDatabaseDataMaskingRule.cs +++ b/src/Sql/Sql/Data Masking/Cmdlet/BuildAzureSqlDatabaseDataMaskingRule.cs @@ -71,7 +71,7 @@ public abstract class BuildAzureSqlDatabaseDataMaskingRule : SqlDatabaseDataMask /// /// Updates the given model element with the cmdlet specific operation /// - /// A model object + /// A model object protected override IEnumerable ApplyUserInputToModel(IEnumerable rules) { string errorMessage = ValidateOperation(rules); @@ -215,7 +215,7 @@ private MaskingFunction ModelizeMaskingFunction() /// This method is responsible to call the right API in the communication layer that will eventually send the information in the /// object to the REST endpoint /// - /// The model object with the data to be sent to the REST endpoints + /// The model object with the data to be sent to the REST endpoints protected override IEnumerable PersistChanges(IEnumerable rules) { ModelAdapter.SetDatabaseDataMaskingRule(rules.First(IsModelOfRule)); diff --git a/src/Sql/Sql/Data Masking/Cmdlet/SqlDatabaseDataMaskingRuleCmdletBase.cs b/src/Sql/Sql/Data Masking/Cmdlet/SqlDatabaseDataMaskingRuleCmdletBase.cs index 1d76e26bb740..08fc64e401dc 100644 --- a/src/Sql/Sql/Data Masking/Cmdlet/SqlDatabaseDataMaskingRuleCmdletBase.cs +++ b/src/Sql/Sql/Data Masking/Cmdlet/SqlDatabaseDataMaskingRuleCmdletBase.cs @@ -78,7 +78,7 @@ protected override IEnumerable GetEntity() /// This method is responsible to call the right API in the communication layer that will eventually send the information in the /// object to the REST endpoint /// - /// The model object with the data to be sent to the REST endpoints + /// The model object with the data to be sent to the REST endpoints protected override IEnumerable PersistChanges(IEnumerable rules) { DatabaseDataMaskingRuleModel model = rules.First(); diff --git a/src/Sql/Sql/Data Sync/Model/AzureSqlSyncAgentModel.cs b/src/Sql/Sql/Data Sync/Model/AzureSqlSyncAgentModel.cs index cbca8a5e730f..b9804cc4a290 100644 --- a/src/Sql/Sql/Data Sync/Model/AzureSqlSyncAgentModel.cs +++ b/src/Sql/Sql/Data Sync/Model/AzureSqlSyncAgentModel.cs @@ -81,6 +81,7 @@ public AzureSqlSyncAgentModel() /// Construct AzureSqlSyncAgentModel /// /// resource group + /// server name /// sync agent public AzureSqlSyncAgentModel(string resourceGroup, string serverName, SyncAgent syncAgent) { diff --git a/src/Sql/Sql/Data Sync/Model/AzureSqlSyncGroupSchemaColumnModel.cs b/src/Sql/Sql/Data Sync/Model/AzureSqlSyncGroupSchemaColumnModel.cs index b52fe9a83ebe..c0dc438d18da 100644 --- a/src/Sql/Sql/Data Sync/Model/AzureSqlSyncGroupSchemaColumnModel.cs +++ b/src/Sql/Sql/Data Sync/Model/AzureSqlSyncGroupSchemaColumnModel.cs @@ -23,7 +23,7 @@ namespace Microsoft.Azure.Commands.Sql.DataSync.Model /// public class AzureSqlSyncGroupSchemaColumnModel { - // + /// /// The quoted table name /// public string QuotedName { get; set; } diff --git a/src/Sql/Sql/Data Sync/Model/AzureSqlSyncGroupSchemaModel.cs b/src/Sql/Sql/Data Sync/Model/AzureSqlSyncGroupSchemaModel.cs index 54f048660279..de9446dcfd53 100644 --- a/src/Sql/Sql/Data Sync/Model/AzureSqlSyncGroupSchemaModel.cs +++ b/src/Sql/Sql/Data Sync/Model/AzureSqlSyncGroupSchemaModel.cs @@ -28,7 +28,7 @@ public class AzureSqlSyncGroupSchemaModel /// public IList Tables { get; set; } - // + /// /// Master sync member name /// public string MasterSyncMemberName { get; set; } diff --git a/src/Sql/Sql/Data Sync/Model/AzureSqlSyncGroupSchemaTableModel.cs b/src/Sql/Sql/Data Sync/Model/AzureSqlSyncGroupSchemaTableModel.cs index b16b90cfa8bb..3eac906db941 100644 --- a/src/Sql/Sql/Data Sync/Model/AzureSqlSyncGroupSchemaTableModel.cs +++ b/src/Sql/Sql/Data Sync/Model/AzureSqlSyncGroupSchemaTableModel.cs @@ -28,7 +28,7 @@ public class AzureSqlSyncGroupSchemaTableModel /// public IList Columns { get; set; } - // + /// /// The quoted table name /// public string QuotedName { get; set; } diff --git a/src/Sql/Sql/Data Sync/Model/AzureSqlSyncMemberModel.cs b/src/Sql/Sql/Data Sync/Model/AzureSqlSyncMemberModel.cs index 4375e8726510..e5a82afe5615 100644 --- a/src/Sql/Sql/Data Sync/Model/AzureSqlSyncMemberModel.cs +++ b/src/Sql/Sql/Data Sync/Model/AzureSqlSyncMemberModel.cs @@ -118,10 +118,11 @@ public AzureSqlSyncMemberModel() /// /// Construct AzureSqlSyncMemberModel for Management.Sql.Models.syncMember object /// - /// Resource group name + /// Resource group name /// Server name /// Databse name - /// sync member object + /// The name of the sync group + /// sync member object public AzureSqlSyncMemberModel(string resourceGroup, string serverName, string databaseName, string syncGroupName, SyncMember syncMember) { ResourceGroupName = resourceGroup; diff --git a/src/Sql/Sql/Data Sync/Services/AzureSqlDataSyncAdapter.cs b/src/Sql/Sql/Data Sync/Services/AzureSqlDataSyncAdapter.cs index 2a0499e11b09..05f6c7e821fa 100644 --- a/src/Sql/Sql/Data Sync/Services/AzureSqlDataSyncAdapter.cs +++ b/src/Sql/Sql/Data Sync/Services/AzureSqlDataSyncAdapter.cs @@ -161,6 +161,7 @@ public void InvokeSyncMemberSchemaRefresh(string resourceGroupName, string serve /// Create a sync group /// /// AzureSqlSyncGroupModel object + /// The sync database resource id /// Created AzureSqlSyncGroupModel object internal AzureSqlSyncGroupModel CreateSyncGroup(AzureSqlSyncGroupModel model, string syncDatabaseId) { @@ -295,8 +296,7 @@ public void RemoveSyncMember(string resourceGroupName, string serverName, string /// Create a sync member /// /// AzureSqlSyncMemberModel object - /// The type of member database - /// The password of member database + /// The sync agent resource id /// Created AzureSqlSyncGroupModel object internal AzureSqlSyncMemberModel CreateSyncMember(AzureSqlSyncMemberModel model, string syncAgentId) { @@ -335,8 +335,6 @@ internal AzureSqlSyncMemberModel CreateSyncMember(AzureSqlSyncMemberModel model, /// Update a sync member /// /// AzureSqlSyncMemberModel object - /// The type of member database - /// The password of member database /// Updated AzureSqlSyncGroupModel object internal AzureSqlSyncMemberModel UpdateSyncMember(AzureSqlSyncMemberModel model) { @@ -384,6 +382,7 @@ public AzureSqlSyncFullSchemaModel GetSyncMemberSchema(string resourceGroupName, /// Gets a sync agent by name /// /// The resource group the sync agent is in + /// The name of the server /// The name of the sync agent /// The sync agent object public AzureSqlSyncAgentModel GetSyncAgent(string resourceGroupName, string serverName, string syncAgentName) @@ -412,6 +411,7 @@ internal ICollection ListSyncAgents(string resourceGroup /// Create a sync agent /// /// AzureSqlSyncAgentModel object + /// The sync database resource id /// Created AzureSqlSyncAgentModel object internal AzureSqlSyncAgentModel CreateSyncAgent(AzureSqlSyncAgentModel model, string syncDatabaseId) { @@ -458,6 +458,7 @@ internal AzureSqlSyncAgentKeyModel CreateSyncAgentKey(string resourceGroupName, /// Get all linked databases connected by a specified sync agent /// /// The resource group the sync agent is in + /// The name of the server /// The name of the sync agent internal ICollection ListSyncAgentLinkedDatabases(string resourceGroupName, string serverName, string syncAgentName) { @@ -497,6 +498,7 @@ public static AzureSqlSyncGroupLogModel CreateSyncGroupLogModelFromResponse(Sync /// The resource group the sync member is in /// The name of the server /// The name of the database + /// The name of the sync group /// The sync member object from the response /// The converted model public static AzureSqlSyncMemberModel CreateSyncMemberModelFromResponse(string resourceGroupName, string serverName, string databaseName, string syncGroupName, Management.Sql.Models.SyncMember syncMember) @@ -508,7 +510,7 @@ public static AzureSqlSyncMemberModel CreateSyncMemberModelFromResponse(string r /// Converts the response from the service to a powershell sync agent object /// /// The resource group the agent is in - /// The server name + /// The server name /// The sync agent object from the response /// The converted model public static AzureSqlSyncAgentModel CreateSyncAgentModelFromResponse(string resourceGroupName, string serverName, SyncAgent syncAgent) diff --git a/src/Sql/Sql/Database Activation/Services/SqlAzureDatabaseActivationAdapter.cs b/src/Sql/Sql/Database Activation/Services/SqlAzureDatabaseActivationAdapter.cs index d02c47a6b95e..4088308461be 100644 --- a/src/Sql/Sql/Database Activation/Services/SqlAzureDatabaseActivationAdapter.cs +++ b/src/Sql/Sql/Database Activation/Services/SqlAzureDatabaseActivationAdapter.cs @@ -38,8 +38,7 @@ public class AzureSqlDatabaseActivationAdapter /// /// Constructs a database activation adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlDatabaseActivationAdapter(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/Database Activation/Services/SqlAzureDatabaseActivationCommunicator.cs b/src/Sql/Sql/Database Activation/Services/SqlAzureDatabaseActivationCommunicator.cs index b3a303d8dee4..7c0861689fba 100644 --- a/src/Sql/Sql/Database Activation/Services/SqlAzureDatabaseActivationCommunicator.cs +++ b/src/Sql/Sql/Database Activation/Services/SqlAzureDatabaseActivationCommunicator.cs @@ -43,8 +43,7 @@ public class AzureSqlDatabaseActivationCommunicator /// /// Creates a communicator for Azure Sql Databases /// - /// - /// + /// The current azure context public AzureSqlDatabaseActivationCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/Database Backup/Cmdlet/CopyAzureSqlDatabaseLongTermRetentionBackup.cs b/src/Sql/Sql/Database Backup/Cmdlet/CopyAzureSqlDatabaseLongTermRetentionBackup.cs index c534721cebb7..63aaa3afc2c1 100644 --- a/src/Sql/Sql/Database Backup/Cmdlet/CopyAzureSqlDatabaseLongTermRetentionBackup.cs +++ b/src/Sql/Sql/Database Backup/Cmdlet/CopyAzureSqlDatabaseLongTermRetentionBackup.cs @@ -135,7 +135,7 @@ public class CopyAzureSqlDatabaseLongTermRetentionBackup : AzureSqlDatabaseLongT public string TargetServerFullyQualifiedDomainName { get; set; } /// - /// Gets or sets the resource ID of the target server (target subscription ID & target resource group name can be derived). + /// Gets or sets the resource ID of the target server (target subscription ID & target resource group name can be derived). /// [Parameter(Mandatory = false, HelpMessage = "The name of the target server.")] diff --git a/src/Sql/Sql/Database Backup/Cmdlet/RestoreAzureRMSqlDatabase.cs b/src/Sql/Sql/Database Backup/Cmdlet/RestoreAzureRMSqlDatabase.cs index c06a4707f6c6..05d30cb32e14 100644 --- a/src/Sql/Sql/Database Backup/Cmdlet/RestoreAzureRMSqlDatabase.cs +++ b/src/Sql/Sql/Database Backup/Cmdlet/RestoreAzureRMSqlDatabase.cs @@ -399,7 +399,7 @@ protected override AzureSqlDatabaseModel GetEntity() model.Edition = Edition; } - /// get auth headers for cross-sub and cross-tenant restore operations + // get auth headers for cross-sub and cross-tenant restore operations string targetSubscriptionId = ModelAdapter.Context?.Subscription?.Id; string sourceSubscriptionId = ParseSubscriptionIdFromResourceId(ResourceId); bool isCrossSubscriptionRestore = false; diff --git a/src/Sql/Sql/Database Backup/Model/AzureSqlDatabaseBackupShortTermRetentionPolicyModel.cs b/src/Sql/Sql/Database Backup/Model/AzureSqlDatabaseBackupShortTermRetentionPolicyModel.cs index 27d414bbba9f..3c01671adbb7 100644 --- a/src/Sql/Sql/Database Backup/Model/AzureSqlDatabaseBackupShortTermRetentionPolicyModel.cs +++ b/src/Sql/Sql/Database Backup/Model/AzureSqlDatabaseBackupShortTermRetentionPolicyModel.cs @@ -48,6 +48,7 @@ public class AzureSqlDatabaseBackupShortTermRetentionPolicyModel /// /// /// + /// /// public AzureSqlDatabaseBackupShortTermRetentionPolicyModel(string resourceGroup, string serverName, string databaseName, Management.Sql.Models.BackupShortTermRetentionPolicy policy) { diff --git a/src/Sql/Sql/Database Backup/Services/AzureSqlDatabaseBackupAdapter.cs b/src/Sql/Sql/Database Backup/Services/AzureSqlDatabaseBackupAdapter.cs index f8e80e96b61b..19fa4a5f35e9 100644 --- a/src/Sql/Sql/Database Backup/Services/AzureSqlDatabaseBackupAdapter.cs +++ b/src/Sql/Sql/Database Backup/Services/AzureSqlDatabaseBackupAdapter.cs @@ -47,8 +47,7 @@ public class AzureSqlDatabaseBackupAdapter /// /// Constructs a database backup adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlDatabaseBackupAdapter(IAzureContext context) { Context = context; @@ -85,9 +84,7 @@ internal IEnumerable ListRestorePoints(string /// /// Creates a new the restore point for a given Sql Azure Database. /// - /// The name of the resource group - /// The name of the Azure SQL Server - /// The name of the Azure SQL database + /// /// List of restore points internal IEnumerable NewRestorePoint(IEnumerable entityList) { @@ -113,10 +110,7 @@ internal IEnumerable NewRestorePoint(IEnumera /// /// Removes a given restore point for a given Sql Azure Database. /// - /// The name of the resource group - /// The name of the Azure SQL Server - /// The name of the Azure SQL database - /// The create time of the restore point + /// /// void internal void RemoveRestorePoint(IEnumerable entityList) { @@ -252,7 +246,6 @@ internal AzureSqlServerBackupLongTermRetentionVaultModel GetBackupLongTermRetent /// The name of the resource group /// The name of the Azure SQL Server /// The name of the Azure SQL Database - /// Whether or not the user provided the Current switch to get the current implementation of LTR policy /// A backup LongTermRetention policy internal AzureSqlDatabaseBackupLongTermRetentionPolicyModel GetDatabaseBackupLongTermRetentionPolicy( string resourceGroup, @@ -278,13 +271,7 @@ internal AzureSqlDatabaseBackupLongTermRetentionPolicyModel GetDatabaseBackupLon /// /// Update a Long Term Retention backup. /// - /// The location name. - /// The server name. - /// The database name. - /// The backup name. - /// The resource group name - /// Whether or not to only get the latest backup per database. - /// The state of databases to get backups for: All, Live, Deleted. + /// internal AzureSqlDatabaseLongTermRetentionBackupCopyModel CopyDatabaseLongTermRetentionBackup( AzureSqlDatabaseLongTermRetentionBackupCopyModel model) { @@ -329,14 +316,8 @@ internal AzureSqlDatabaseLongTermRetentionBackupCopyModel CopyDatabaseLongTermRe /// /// Update a Long Term Retention backup. /// - /// The location name. - /// The server name. - /// The database name. - /// The backup name. - /// The resource group name - /// Whether or not to only get the latest backup per database. - /// The state of databases to get backups for: All, Live, De - /// leted. + /// + /// internal AzureSqlDatabaseLongTermRetentionBackupModel UpdateDatabaseLongTermRetentionBackup( AzureSqlDatabaseLongTermRetentionBackupModel model, Management.Sql.Models.UpdateLongTermRetentionBackupParameters updateParameters) @@ -365,6 +346,7 @@ internal AzureSqlDatabaseLongTermRetentionBackupModel UpdateDatabaseLongTermRete /// /// The name of the resource group /// The name of the Azure SQL Server + /// A backup vault /// A backup vault internal AzureSqlServerBackupLongTermRetentionVaultModel SetBackupLongTermRetentionVault( string resourceGroup, @@ -398,6 +380,7 @@ internal AzureSqlServerBackupLongTermRetentionVaultModel SetBackupLongTermRetent /// The name of the resource group /// The name of the Azure SQL Server /// The name of the Azure SQL Database + /// A backup LongTermRetention policy /// A backup LongTermRetention policy internal AzureSqlDatabaseBackupLongTermRetentionPolicyModel SetDatabaseBackupLongTermRetentionPolicy( string resourceGroup, @@ -541,6 +524,7 @@ internal AzureSqlDatabaseGeoBackupPolicyModel GetDatabaseGeoBackupPolicy( /// The name of the resource group /// The name of the Azure SQL Server /// The name of the Azure SQL Database + /// A geo backup policy /// A geo backup policy internal AzureSqlDatabaseGeoBackupPolicyModel SetDatabaseGeoBackupPolicy( string resourceGroup, @@ -620,7 +604,7 @@ internal AzureSqlDatabaseModel RestoreDatabase(string resourceGroup, DateTime re if (dbModel.CreateMode != Management.Sql.Models.CreateMode.Recovery && dbModel.CreateMode != Management.Sql.Models.CreateMode.Restore && dbModel.CreateMode != Management.Sql.Models.CreateMode.PointInTimeRestore) - { + { throw new ArgumentException("Restore mode not supported for cross subscription restore. Supported restore modes for cross subscription are Recovery, Restore, PointInTimeRestore."); } @@ -670,12 +654,13 @@ public string GetServerLocation(string resourceGroupName, string serverName) } /// - /// Create or update a backup LongTermRetention policy for a Azure SQL Database + /// Create or update a backup ShortTermRetention policy for a Azure SQL Database /// /// The name of the resource group /// The name of the Azure SQL Server /// The name of the Azure SQL Database - /// A backup LongTermRetention policy + /// A backup ShortTermRetention policy + /// A backup ShortTermRetention policy internal AzureSqlDatabaseBackupShortTermRetentionPolicyModel SetDatabaseBackupShortTermRetentionPolicy( string resourceGroup, string serverName, @@ -696,13 +681,12 @@ internal AzureSqlDatabaseBackupShortTermRetentionPolicyModel SetDatabaseBackupSh } /// - /// Get a backup LongTermRetention policy for a Azure SQL Database + /// Get a backup ShortTermRetention policy for a Azure SQL Database /// /// The name of the resource group /// The name of the Azure SQL Server /// The name of the Azure SQL Database - /// Whether or not the user provided the Current switch to get the current implementation of LTR policy - /// A backup LongTermRetention policy + /// A backup ShortTermRetention policy internal AzureSqlDatabaseBackupShortTermRetentionPolicyModel GetDatabaseBackupShortTermRetentionPolicy( string resourceGroup, string serverName, diff --git a/src/Sql/Sql/Database Backup/Services/AzureSqlDatabaseBackupCommunicator.cs b/src/Sql/Sql/Database Backup/Services/AzureSqlDatabaseBackupCommunicator.cs index 2794782f8995..6d0668ad771b 100644 --- a/src/Sql/Sql/Database Backup/Services/AzureSqlDatabaseBackupCommunicator.cs +++ b/src/Sql/Sql/Database Backup/Services/AzureSqlDatabaseBackupCommunicator.cs @@ -50,8 +50,7 @@ public class AzureSqlDatabaseBackupCommunicator /// /// Creates a communicator for Azure Sql Database backup REST endpoints. /// - /// Azure profile - /// Associated subscription + /// Azure context public AzureSqlDatabaseBackupCommunicator(IAzureContext context) { Context = context; @@ -65,7 +64,7 @@ public AzureSqlDatabaseBackupCommunicator(IAzureContext context) /// /// Lists the restore points for a given Sql Azure Database. /// - /// The name of the resource group + /// The name of the resource group /// The name of the Azure SQL Server /// The name of the Azure SQL database /// List of restore points @@ -77,9 +76,10 @@ public AzureSqlDatabaseBackupCommunicator(IAzureContext context) /// /// Creates a new restore point for a given Sql Azure Database. /// - /// The name of the resource group + /// The name of the resource group /// The name of the Azure SQL Server /// The name of the Azure SQL database + /// The definition for creating the restore point of this database /// A restore point public Management.Sql.Models.RestorePoint NewRestorePoint(string resourceGroupName, string serverName, string databaseName, Management.Sql.Models.CreateDatabaseRestorePointDefinition restoreDefinition) { @@ -89,7 +89,7 @@ public Management.Sql.Models.RestorePoint NewRestorePoint(string resourceGroupNa /// /// Removes a given restore point for a given Sql Azure Database. /// - /// The name of the resource group + /// The name of the resource group /// The name of the Azure SQL Server /// The name of the Azure SQL database /// The name of the restore point @@ -102,7 +102,7 @@ public void RemoveRestorePoint(string resourceGroupName, string serverName, stri /// /// Lists the geo backups for a given Sql Azure Server /// - /// The name of the resource group + /// The name of the resource group /// The name of the Azure SQL Server /// List of restore points public IList ListGeoBackups(string resourceGroupName, string serverName) @@ -113,7 +113,7 @@ public void RemoveRestorePoint(string resourceGroupName, string serverName, stri /// /// Lists the restorable deleted databases for a given Sql Azure Server /// - /// The name of the resource group + /// The name of the resource group /// The name of the Azure SQL Server /// List of restore points public IList ListDeletedDatabaseBackups(string resourceGroupName, string serverName) @@ -124,7 +124,7 @@ public void RemoveRestorePoint(string resourceGroupName, string serverName, stri /// /// Get a geo backup for a given Sql Azure Database /// - /// The name of the resource group + /// The name of the resource group /// The name of the Azure SQL Server /// The name of the Azure SQL database /// List of restore points @@ -136,7 +136,7 @@ public Management.Sql.LegacySdk.Models.GeoBackup GetGeoBackup(string resourceGro /// /// Get a restorable deleted database for a given Sql Azure Database /// - /// The name of the resource group + /// The name of the resource group /// The name of the Azure SQL Server /// The name of the Azure SQL database /// List of restore points @@ -148,8 +148,9 @@ public Management.Sql.LegacySdk.Models.DeletedDatabaseBackup GetDeletedDatabaseB /// /// Get a backup LongTermRetention vault for a given Azure SQL Server /// - /// The name of the resource group + /// The name of the resource group /// The name of the Azure SQL Server + /// The name of the Azure SQL Database backup LongTermRetention vault. /// A backup vault public Management.Sql.LegacySdk.Models.BackupLongTermRetentionVault GetBackupLongTermRetentionVault( string resourceGroupName, @@ -165,9 +166,10 @@ public Management.Sql.LegacySdk.Models.BackupLongTermRetentionVault GetBackupLon /// /// Get a backup LongTermRetention policy for a Azure SQL Database /// - /// The name of the resource group + /// The name of the resource group /// The name of the Azure SQL Server /// The name of the Azure SQL Database + /// The name of the Azure SQL Database backup LongTermRetention policy. /// A backup LongTermRetention policy public Management.Sql.LegacySdk.Models.DatabaseBackupLongTermRetentionPolicy GetDatabaseBackupLongTermRetentionPolicy( string resourceGroupName, @@ -331,6 +333,7 @@ public Management.Sql.Models.LongTermRetentionBackup GetDatabaseLongTermRetentio /// The database name. /// The backup name. /// The resource group name + /// The parameters needed for long term retention copy request public Management.Sql.Models.LongTermRetentionBackupOperationResult CopyDatabaseLongTermRetentionBackup( string locationName, string serverName, @@ -357,6 +360,7 @@ public Management.Sql.Models.LongTermRetentionBackupOperationResult CopyDatabase /// The database name. /// The backup name. /// The resource group name + /// The requested backup resource state public Management.Sql.Models.LongTermRetentionBackupOperationResult UpdateDatabaseLongTermRetentionBackup( string locationName, string serverName, @@ -404,9 +408,10 @@ public void RemoveDatabaseLongTermRetentionBackup( /// /// Get a geo backup policy for a Azure SQL Database /// - /// The name of the resource group + /// The name of the resource group /// The name of the Azure SQL Server /// The name of the Azure SQL Database + /// The name of the geo backup policy /// A geo backup policy public Management.Sql.LegacySdk.Models.GeoBackupPolicy GetDatabaseGeoBackupPolicy( string resourceGroupName, @@ -442,7 +447,7 @@ public Management.Sql.LegacySdk.Models.GeoBackupPolicy SetDatabaseGeoBackupPolic /// /// Restore a given Sql Azure Database /// - /// The name of the resource group + /// The name of the resource group /// The name of the Azure SQL Server /// The name of the Azure SQL database /// Parameters describing the database restore request @@ -455,7 +460,7 @@ public Management.Sql.LegacySdk.Models.Database LegacyRestoreDatabase(string res /// /// Restore a given Sql Azure Database /// - /// The name of the resource group + /// The name of the resource group /// The name of the Azure SQL Server /// The name of the Azure SQL database /// Sql Database Model with required parameters @@ -502,7 +507,7 @@ public Management.Sql.Models.BackupShortTermRetentionPolicy GetDatabaseBackupSho /// The resource group name. /// The server name. /// The database name. - /// The Long Term Retention policy to apply. + /// The Short Term Retention policy to apply. /// A backup ShortTermRetention policy public Management.Sql.Models.BackupShortTermRetentionPolicy SetDatabaseBackupShortTermRetentionPolicy( string resourceGroup, diff --git a/src/Sql/Sql/Database/Services/AzureSqlDatabaseCommunicator.cs b/src/Sql/Sql/Database/Services/AzureSqlDatabaseCommunicator.cs index 899cfc8809a6..a5c9d351bc55 100644 --- a/src/Sql/Sql/Database/Services/AzureSqlDatabaseCommunicator.cs +++ b/src/Sql/Sql/Database/Services/AzureSqlDatabaseCommunicator.cs @@ -52,8 +52,7 @@ public class AzureSqlDatabaseCommunicator /// /// Creates a communicator for Azure Sql Databases /// - /// - /// + /// The current azure context public AzureSqlDatabaseCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/DistributedAvailabilityGroup/Services/AzureSqlManagedInstanceLinkAdapter.cs b/src/Sql/Sql/DistributedAvailabilityGroup/Services/AzureSqlManagedInstanceLinkAdapter.cs index bafa1d76818d..1d24b01025d5 100644 --- a/src/Sql/Sql/DistributedAvailabilityGroup/Services/AzureSqlManagedInstanceLinkAdapter.cs +++ b/src/Sql/Sql/DistributedAvailabilityGroup/Services/AzureSqlManagedInstanceLinkAdapter.cs @@ -117,7 +117,9 @@ public void RemoveManagedInstanceLink(string resourceGroupName, string instanceN /// /// Convert a Management.Sql.Models.DistributedAvailabilityGroup to AzureSqlManagedInstanceLinkModel /// - /// The management client distributed availiability group response to convert + /// Resource group used by the managed instance + /// Name of the managed instance + /// The management client distributed availiability group response to convert /// The converted managed instance link model private static AzureSqlManagedInstanceLinkModel CreateManagedInstanceLinkModelFromResponse(string resourceGroupName, string instanceName, Management.Sql.Models.DistributedAvailabilityGroup managedInstanceLink) { diff --git a/src/Sql/Sql/DistributedAvailabilityGroup/Services/AzureSqlManagedInstanceLinkCommunicator.cs b/src/Sql/Sql/DistributedAvailabilityGroup/Services/AzureSqlManagedInstanceLinkCommunicator.cs index 9b0f479d0590..d9cb8202aca3 100644 --- a/src/Sql/Sql/DistributedAvailabilityGroup/Services/AzureSqlManagedInstanceLinkCommunicator.cs +++ b/src/Sql/Sql/DistributedAvailabilityGroup/Services/AzureSqlManagedInstanceLinkCommunicator.cs @@ -43,8 +43,7 @@ public class AzureSqlManagedInstanceLinkCommunicator /// /// Creates a communicator for Azure Sql Instance Server Trust Certificate /// - /// - /// + /// The current azure context public AzureSqlManagedInstanceLinkCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/Elastic Jobs/Cmdlet/Agent/AzureSqlElasticJobAgentCmdletBase.cs b/src/Sql/Sql/Elastic Jobs/Cmdlet/Agent/AzureSqlElasticJobAgentCmdletBase.cs index 42751b0a21b4..19cfe7cdcd57 100644 --- a/src/Sql/Sql/Elastic Jobs/Cmdlet/Agent/AzureSqlElasticJobAgentCmdletBase.cs +++ b/src/Sql/Sql/Elastic Jobs/Cmdlet/Agent/AzureSqlElasticJobAgentCmdletBase.cs @@ -47,7 +47,7 @@ protected override AzureSqlElasticJobAdapter InitModelAdapter() /// We clear these properties so that during piping scenarios we can ensure we initialize the minimum properties /// for either getting, creating, updating, or removing the correct resource /// Resource group name, server name, agent name, and name are cleared - /// so that during the next iteration in list, they will be initialized properly during + /// so that during the next iteration in list, they will be initialized properly during /// protected void ClearProperties() { diff --git a/src/Sql/Sql/Elastic Jobs/Cmdlet/Credential/AzureSqlElasticJobCredentialCmdletBase.cs b/src/Sql/Sql/Elastic Jobs/Cmdlet/Credential/AzureSqlElasticJobCredentialCmdletBase.cs index eadc19746e70..122aa54f75d6 100644 --- a/src/Sql/Sql/Elastic Jobs/Cmdlet/Credential/AzureSqlElasticJobCredentialCmdletBase.cs +++ b/src/Sql/Sql/Elastic Jobs/Cmdlet/Credential/AzureSqlElasticJobCredentialCmdletBase.cs @@ -42,7 +42,7 @@ protected override AzureSqlElasticJobAdapter InitModelAdapter() /// We clear these properties so that during piping scenarios we can ensure we initialize the minimum properties /// for either getting, creating, updating, or removing the correct resource /// Resource group name, server name, agent name, credential name, and name are cleared - /// so that during the next iteration in list, they will be initialized properly during + /// so that during the next iteration in list, they will be initialized properly during /// protected void ClearProperties() { diff --git a/src/Sql/Sql/Elastic Jobs/Cmdlet/Credential/GetAzureSqlElasticJobCredential.cs b/src/Sql/Sql/Elastic Jobs/Cmdlet/Credential/GetAzureSqlElasticJobCredential.cs index df04dabd93a4..d7eb2c38098a 100644 --- a/src/Sql/Sql/Elastic Jobs/Cmdlet/Credential/GetAzureSqlElasticJobCredential.cs +++ b/src/Sql/Sql/Elastic Jobs/Cmdlet/Credential/GetAzureSqlElasticJobCredential.cs @@ -99,7 +99,7 @@ public override void ExecuteCmdlet() /// /// Check to see if the credential already exists for the agent. /// - /// Throws exception if the credential doesn't exist. + /// Throws exception if the credential doesn't exist. protected override IEnumerable GetEntity() { try diff --git a/src/Sql/Sql/Elastic Jobs/Cmdlet/Credential/RemoveAzureSqlElasticJobCredential.cs b/src/Sql/Sql/Elastic Jobs/Cmdlet/Credential/RemoveAzureSqlElasticJobCredential.cs index 456c4544207c..53ded95fdb19 100644 --- a/src/Sql/Sql/Elastic Jobs/Cmdlet/Credential/RemoveAzureSqlElasticJobCredential.cs +++ b/src/Sql/Sql/Elastic Jobs/Cmdlet/Credential/RemoveAzureSqlElasticJobCredential.cs @@ -114,7 +114,7 @@ public override void ExecuteCmdlet() /// /// Check to see if the credential already exists for the agent. /// - /// Throws exception if the credential doesn't exist. + /// Throws exception if the credential doesn't exist. protected override IEnumerable GetEntity() { try diff --git a/src/Sql/Sql/Elastic Jobs/Cmdlet/Job/AzureSqlElasticJobCmdletBase.cs b/src/Sql/Sql/Elastic Jobs/Cmdlet/Job/AzureSqlElasticJobCmdletBase.cs index 8047f34eded7..7bbf0863ab8c 100644 --- a/src/Sql/Sql/Elastic Jobs/Cmdlet/Job/AzureSqlElasticJobCmdletBase.cs +++ b/src/Sql/Sql/Elastic Jobs/Cmdlet/Job/AzureSqlElasticJobCmdletBase.cs @@ -42,7 +42,7 @@ protected override AzureSqlElasticJobAdapter InitModelAdapter() /// We clear these properties so that during piping scenarios we can ensure we initialize the minimum properties /// for either getting, creating, updating, or removing the correct resource /// Resource group name, server name, agent name, job name, and name are cleared - /// so that during the next iteration in list, they will be initialized properly during + /// so that during the next iteration in list, they will be initialized properly during /// protected void ClearProperties() { diff --git a/src/Sql/Sql/Elastic Jobs/Cmdlet/JobExecution/AzureSqlElasticJobExecutionCmdletBase.cs b/src/Sql/Sql/Elastic Jobs/Cmdlet/JobExecution/AzureSqlElasticJobExecutionCmdletBase.cs index 74682d04dfae..dc69397dd013 100644 --- a/src/Sql/Sql/Elastic Jobs/Cmdlet/JobExecution/AzureSqlElasticJobExecutionCmdletBase.cs +++ b/src/Sql/Sql/Elastic Jobs/Cmdlet/JobExecution/AzureSqlElasticJobExecutionCmdletBase.cs @@ -42,7 +42,7 @@ protected override AzureSqlElasticJobAdapter InitModelAdapter() /// We clear these properties so that during piping scenarios we can ensure we initialize the minimum properties /// for either getting, starting, stopping the current job execution. /// Resource group name, server name, agent name, job, job execution id, and step name are cleared - /// so that during the next iteration in list, they will be initialized properly during + /// so that during the next iteration in list, they will be initialized properly during /// protected void ClearProperties() { diff --git a/src/Sql/Sql/Elastic Jobs/Cmdlet/JobStep/AzureSqlElasticJobStepCmdletBase.cs b/src/Sql/Sql/Elastic Jobs/Cmdlet/JobStep/AzureSqlElasticJobStepCmdletBase.cs index 5aeb7e8457a3..6dc847e44f62 100644 --- a/src/Sql/Sql/Elastic Jobs/Cmdlet/JobStep/AzureSqlElasticJobStepCmdletBase.cs +++ b/src/Sql/Sql/Elastic Jobs/Cmdlet/JobStep/AzureSqlElasticJobStepCmdletBase.cs @@ -42,7 +42,7 @@ protected override AzureSqlElasticJobAdapter InitModelAdapter() /// We clear these properties so that during piping scenarios we can ensure we initialize the minimum properties /// for either getting, starting, stopping the current job execution. /// Resource group name, server name, agent name, job, job execution id, and step name are cleared - /// so that during the next iteration in list, they will be initialized properly during + /// so that during the next iteration in list, they will be initialized properly during /// /// Note: We dont' clear credential name and target group name in case during Set-AzureRmSqlElasticJobStep scenario, client wishes to /// update multiple steps with same credential or target group. This is why we only clear the minimum parameters necessary to find the diff --git a/src/Sql/Sql/Elastic Jobs/Cmdlet/Target/AzureSqlElasticJobTargetCmdletBase.cs b/src/Sql/Sql/Elastic Jobs/Cmdlet/Target/AzureSqlElasticJobTargetCmdletBase.cs index 890453b91ddb..6ff9ed6b276b 100644 --- a/src/Sql/Sql/Elastic Jobs/Cmdlet/Target/AzureSqlElasticJobTargetCmdletBase.cs +++ b/src/Sql/Sql/Elastic Jobs/Cmdlet/Target/AzureSqlElasticJobTargetCmdletBase.cs @@ -407,7 +407,8 @@ public string GetTargetType() /// We clear these properties so that during piping scenarios we can ensure we initialize the minimum properties /// for either getting, starting, stopping the current job execution. /// Resource group name, server name, agent name, target group name, and name are cleared - /// so that during the next iteration in list, they will be initialized properly during + /// so that during the next iteration in list, they will be initialized properly during + /// protected void ClearProperties() { this.ResourceGroupName = null; diff --git a/src/Sql/Sql/Elastic Jobs/Cmdlet/TargetGroup/AzureSqlElasticJobTargetGroupCmdletBase.cs b/src/Sql/Sql/Elastic Jobs/Cmdlet/TargetGroup/AzureSqlElasticJobTargetGroupCmdletBase.cs index 3ae5a324c935..55c31966409d 100644 --- a/src/Sql/Sql/Elastic Jobs/Cmdlet/TargetGroup/AzureSqlElasticJobTargetGroupCmdletBase.cs +++ b/src/Sql/Sql/Elastic Jobs/Cmdlet/TargetGroup/AzureSqlElasticJobTargetGroupCmdletBase.cs @@ -42,7 +42,8 @@ protected override AzureSqlElasticJobAdapter InitModelAdapter() /// We clear these properties so that during piping scenarios we can ensure we initialize the minimum properties /// for either getting, starting, stopping the current job execution. /// Resource group name, server name, agent name, target group name, and name are cleared - /// so that during the next iteration in list, they will be initialized properly during + /// so that during the next iteration in list, they will be initialized properly during + /// protected void ClearProperties() { this.ResourceGroupName = null; diff --git a/src/Sql/Sql/Elastic Jobs/Cmdlet/TargetGroup/GetAzureSqlElasticJobTargetGroup.cs b/src/Sql/Sql/Elastic Jobs/Cmdlet/TargetGroup/GetAzureSqlElasticJobTargetGroup.cs index acc230324938..f84349d63fcb 100644 --- a/src/Sql/Sql/Elastic Jobs/Cmdlet/TargetGroup/GetAzureSqlElasticJobTargetGroup.cs +++ b/src/Sql/Sql/Elastic Jobs/Cmdlet/TargetGroup/GetAzureSqlElasticJobTargetGroup.cs @@ -113,7 +113,7 @@ public override void ExecuteCmdlet() /// /// Check to see if the target group exists /// - /// Throws exception if the target group doesn't exist. + /// Throws exception if the target group doesn't exist. protected override IEnumerable GetEntity() { try diff --git a/src/Sql/Sql/Elastic Jobs/Services/AzureSqlElasticJobAdapter.cs b/src/Sql/Sql/Elastic Jobs/Services/AzureSqlElasticJobAdapter.cs index dee2e42c148d..bfefb7122e5e 100644 --- a/src/Sql/Sql/Elastic Jobs/Services/AzureSqlElasticJobAdapter.cs +++ b/src/Sql/Sql/Elastic Jobs/Services/AzureSqlElasticJobAdapter.cs @@ -166,7 +166,6 @@ private static AzureSqlElasticJobAgentModel CreateAgentModelFromResponse(string /// /// The resource group the server is in /// The name of the server - /// The client identifier. /// /// /// These 2 operations (get location, throw if not supported) are combined in order to minimize round trips. @@ -405,10 +404,6 @@ private AzureSqlElasticJobTargetGroupModel CreateTargetGroupModelFromResponse(st /// /// Upserts a job /// - /// The resource group name - /// The server name - /// The agent name - /// The job name /// The job parameters /// public AzureSqlElasticJobModel UpsertJob(AzureSqlElasticJobModel model) @@ -704,6 +699,8 @@ public AzureSqlElasticJobExecutionModel BeginCreateJobExecution(AzureSqlElasticJ /// The resource group /// The server the agent is in /// The agent name + /// The job name + /// The job execution id /// The converted agent model public AzureSqlElasticJobExecutionModel GetJobExecution(string resourceGroupName, string serverName, string agentName, string jobName, Guid jobExecutionId) { @@ -714,8 +711,7 @@ public AzureSqlElasticJobExecutionModel GetJobExecution(string resourceGroupName /// /// Cancels a job execution /// - /// The resource group name - /// The server the agents are in + /// /// The converted agent model(s) public void CancelJobExecution(AzureSqlElasticJobExecutionModel model) { @@ -728,6 +724,13 @@ public void CancelJobExecution(AzureSqlElasticJobExecutionModel model) /// The resource group name /// The server the agent is in /// The agent name + /// The create time min + /// The create time max + /// The end time min + /// The end time max + /// The is active flag + /// The skip count + /// The top count public List ListByAgent( string resourceGroupName, string serverName, @@ -760,6 +763,14 @@ public List ListByAgent( /// The resource group name /// The server the agent is in /// The agent name + /// The job name + /// The create time min + /// The create time max + /// The end time min + /// The end time max + /// The is active flag + /// The skip count + /// The top count public List ListByJob( string resourceGroupName, string serverName, @@ -821,7 +832,13 @@ public AzureSqlElasticJobStepExecutionModel GetJobStepExecution( /// /// /// - /// + /// + /// + /// + /// + /// + /// + /// /// public List ListJobExecutionSteps( string resourceGroupName, @@ -887,6 +904,13 @@ public AzureSqlElasticJobTargetExecutionModel GetJobTargetExecution( /// /// /// + /// + /// + /// + /// + /// + /// + /// /// public List ListJobTargetExecutionsByStep( string resourceGroupName, @@ -924,7 +948,13 @@ public List ListJobTargetExecutionsBySte /// /// /// - /// + /// + /// + /// + /// + /// + /// + /// /// public List ListJobTargetExecutions( string resourceGroupName, @@ -958,6 +988,8 @@ public List ListJobTargetExecutions( /// /// The resource group the server is in /// The server the agent is in + /// The agent name + /// The job name /// The management client server response to convert /// The converted agent model private static AzureSqlElasticJobExecutionModel CreateJobExecutionModelFromResponse( diff --git a/src/Sql/Sql/Elastic Jobs/Services/AzureSqlElasticJobCommunicator.cs b/src/Sql/Sql/Elastic Jobs/Services/AzureSqlElasticJobCommunicator.cs index f7c30f394a11..5a56604840be 100644 --- a/src/Sql/Sql/Elastic Jobs/Services/AzureSqlElasticJobCommunicator.cs +++ b/src/Sql/Sql/Elastic Jobs/Services/AzureSqlElasticJobCommunicator.cs @@ -279,10 +279,11 @@ public JobExecution CreateJobExecution( /// /// Gets the associated Job Credential associated to the Azure SQL Database Agent /// - /// - /// - /// - /// + /// The resource group name + /// The server name + /// The agent name + /// The job name + /// The job execution id /// The agent belonging to specified server public JobExecution GetJobExecution( string resourceGroupName, @@ -401,7 +402,8 @@ public List ListJobExecutionsByJob( /// The resource group name /// The server name /// The agent name - /// The credential name + /// The job name + /// The job execution id public void CancelJobExecution( string resourceGroupName, string serverName, diff --git a/src/Sql/Sql/Elastic Pools/Services/AzureSqlElasticPoolAdapter.cs b/src/Sql/Sql/Elastic Pools/Services/AzureSqlElasticPoolAdapter.cs index e7289a739cfe..d332a429f476 100644 --- a/src/Sql/Sql/Elastic Pools/Services/AzureSqlElasticPoolAdapter.cs +++ b/src/Sql/Sql/Elastic Pools/Services/AzureSqlElasticPoolAdapter.cs @@ -47,8 +47,7 @@ public class AzureSqlElasticPoolAdapter /// /// Constructs a database adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlElasticPoolAdapter(IAzureContext context) { Context = context; @@ -87,8 +86,6 @@ internal ICollection ListElasticPools(string resourceG /// /// Creates an Azure Sql Database ElasticPool. /// - /// The name of the resource group - /// The name of the Azure Sql Database Server /// The input parameters for the create/update operation /// The upserted Azure Sql Database ElasticPool internal AzureSqlElasticPoolModel CreateElasticPool(AzureSqlElasticPoolModel model) @@ -122,8 +119,6 @@ internal AzureSqlElasticPoolModel CreateElasticPool(AzureSqlElasticPoolModel mod /// /// Updates an Azure Sql Database ElasticPool using Patch. /// - /// The name of the resource group - /// The name of the Azure Sql Database Server /// The input parameters for the create/update operation /// The upserted Azure Sql Database ElasticPool internal AzureSqlElasticPoolModel UpsertElasticPool(AzureSqlElasticPoolModel model) @@ -387,7 +382,7 @@ public string GetServerLocation(string resourceGroupName, string serverName) /// /// Converts the response from the service to a powershell database object /// - /// The resource group the server is in + /// The resource group the server is in /// The name of the Azure Sql Database Server /// The service response /// The converted model diff --git a/src/Sql/Sql/Elastic Pools/Services/AzureSqlElasticPoolCommunicator.cs b/src/Sql/Sql/Elastic Pools/Services/AzureSqlElasticPoolCommunicator.cs index c031ff99afde..176c28ed517b 100644 --- a/src/Sql/Sql/Elastic Pools/Services/AzureSqlElasticPoolCommunicator.cs +++ b/src/Sql/Sql/Elastic Pools/Services/AzureSqlElasticPoolCommunicator.cs @@ -54,8 +54,7 @@ public class AzureSqlElasticPoolCommunicator /// /// Creates a communicator for Azure Sql Elastic Pool /// - /// - /// + /// The current azure context public AzureSqlElasticPoolCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/EndpointCertificate/Services/AzureSqlInstanceEndpointCertificateAdapter.cs b/src/Sql/Sql/EndpointCertificate/Services/AzureSqlInstanceEndpointCertificateAdapter.cs index 54f6ea71189b..6f606b932023 100644 --- a/src/Sql/Sql/EndpointCertificate/Services/AzureSqlInstanceEndpointCertificateAdapter.cs +++ b/src/Sql/Sql/EndpointCertificate/Services/AzureSqlInstanceEndpointCertificateAdapter.cs @@ -60,8 +60,8 @@ public AzureSqlInstanceEndpointCertificateModel GetEndpointCertificate(string re /// /// Gets a list of all endpoint certificates in managed instance /// - /// - /// + /// Name of the resource group + /// Name of the managed instance /// A list of all the endpoint certificates public List ListEndpointCertificates(string resourceGroupName, string instanceName) { @@ -73,6 +73,8 @@ public List ListEndpointCertificates(s /// /// Convert a Management.Sql.Models.EndpointCertificate to AzureSqlInstanceEndpointCertificateModel /// + /// Name of the resource group + /// Name of the managed instance /// The management client endpoint certificate response to convert /// The converted endpoint certificate model private static AzureSqlInstanceEndpointCertificateModel CreateEndpointCertificateModelFromResponse(string resourceGroupName, string instanceName, Management.Sql.Models.EndpointCertificate endpointCertificate) diff --git a/src/Sql/Sql/EndpointCertificate/Services/AzureSqlInstanceEndpointCertificateCommunicator.cs b/src/Sql/Sql/EndpointCertificate/Services/AzureSqlInstanceEndpointCertificateCommunicator.cs index a837967e09e3..b63e35345427 100644 --- a/src/Sql/Sql/EndpointCertificate/Services/AzureSqlInstanceEndpointCertificateCommunicator.cs +++ b/src/Sql/Sql/EndpointCertificate/Services/AzureSqlInstanceEndpointCertificateCommunicator.cs @@ -43,8 +43,7 @@ class AzureSqlInstanceEndpointCertificateCommunicator /// /// Creates a communicator for endpoint certificate /// - /// - /// + /// The current azure context public AzureSqlInstanceEndpointCertificateCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/Failover Group/Model/AzureSqlFailoverGroupModel.cs b/src/Sql/Sql/Failover Group/Model/AzureSqlFailoverGroupModel.cs index 8791081b7f3e..71c3f8b92f06 100644 --- a/src/Sql/Sql/Failover Group/Model/AzureSqlFailoverGroupModel.cs +++ b/src/Sql/Sql/Failover Group/Model/AzureSqlFailoverGroupModel.cs @@ -136,7 +136,7 @@ public AzureSqlFailoverGroupModel() /// /// Resource group /// Server name - /// Advisor name + /// The name of the Azure Sql Database FailoverGroup /// Recommended Action object public AzureSqlFailoverGroupModel(string resourceGroupName, string serverName, string failoverGroupName, Management.Sql.LegacySdk.Models.FailoverGroup failoverGroup) { diff --git a/src/Sql/Sql/Failover Group/Services/AzureSqlFailoverGroupAdapter.cs b/src/Sql/Sql/Failover Group/Services/AzureSqlFailoverGroupAdapter.cs index 6b1469cc9b6d..92fd723fbdb5 100644 --- a/src/Sql/Sql/Failover Group/Services/AzureSqlFailoverGroupAdapter.cs +++ b/src/Sql/Sql/Failover Group/Services/AzureSqlFailoverGroupAdapter.cs @@ -48,8 +48,7 @@ public class AzureSqlFailoverGroupAdapter /// /// Constructs a database adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlFailoverGroupAdapter(IAzureContext context) { _subscription = context?.Subscription; @@ -90,8 +89,6 @@ internal ICollection ListFailoverGroups(string resou /// /// Creates or updates an Azure Sql Database FailoverGroup. /// - /// The name of the resource group - /// The name of the Azure Sql Database Server /// The input parameters for the create/update operation /// The upserted Azure Sql Database FailoverGroup internal AzureSqlFailoverGroupModel UpsertFailoverGroup(AzureSqlFailoverGroupModel model) @@ -132,8 +129,6 @@ internal AzureSqlFailoverGroupModel UpsertFailoverGroup(AzureSqlFailoverGroupMod /// /// Patch updates an Azure Sql Database FailoverGroup. /// - /// The name of the resource group - /// The name of the Azure Sql Database Server /// The input parameters for the create/update operation /// The upserted Azure Sql Database FailoverGroup internal AzureSqlFailoverGroupModel PatchUpdateFailoverGroup(AzureSqlFailoverGroupModel model) @@ -166,7 +161,7 @@ internal AzureSqlFailoverGroupModel PatchUpdateFailoverGroup(AzureSqlFailoverGro /// /// The resource group the server is in /// The name of the Azure Sql Database Server - /// The name of the Azure SQL Database Failover Group to delete + /// The name of the Azure SQL Database Failover Group to delete public void RemoveFailoverGroup(string resourceGroupName, string serverName, string failoverGroupName) { Communicator.Remove(resourceGroupName, serverName, failoverGroupName); @@ -191,9 +186,9 @@ internal ICollection ListDatabasesOnServer(string resourc /// /// Patch updates an Azure Sql Database FailoverGroup. /// - /// The name of the resource group - /// The name of the Azure Sql Database Server + /// The name of the resource group /// The name of the Azure Sql Database Server + /// The name of the Azure Sql Database FailoverGroup /// The input parameters for the create/update operation /// The updated Azure Sql Database FailoverGroup internal AzureSqlFailoverGroupModel AddOrRemoveDatabaseToFailoverGroup(string resourceGroupName, string serverName, string failoverGroupName, AzureSqlFailoverGroupModel model) @@ -215,9 +210,7 @@ internal AzureSqlFailoverGroupModel AddOrRemoveDatabaseToFailoverGroup(string re /// /// The name of the Resource Group containing the primary database /// The name of the Azure SQL Server containing the primary database - /// The name of primary database - /// The name of the Resource Group containing the secondary database - /// The name of the Azure SQL Server containing the secondary database + /// The name of the Azure Sql Database FailoverGroup /// Whether the failover operation will allow data loss /// The Azure SQL Database ReplicationLink object internal AzureSqlFailoverGroupModel Failover(string resourceGroupName, string serverName, string failoverGroupName, bool allowDataLoss) @@ -250,9 +243,7 @@ public string GetServerLocation(string resourceGroupName, string serverName) /// /// Converts the response from the service to a powershell database object /// - /// The resource group the server is in - /// The name of the Azure Sql Database Server - /// The service response + /// Recommended Action object /// The converted model private AzureSqlFailoverGroupModel CreateFailoverGroupModelFromResponse(Management.Sql.LegacySdk.Models.FailoverGroup failoverGroup) { diff --git a/src/Sql/Sql/Failover Group/Services/AzureSqlFailoverGroupCommunicator.cs b/src/Sql/Sql/Failover Group/Services/AzureSqlFailoverGroupCommunicator.cs index ec34f6b53a65..f979d93eba3b 100644 --- a/src/Sql/Sql/Failover Group/Services/AzureSqlFailoverGroupCommunicator.cs +++ b/src/Sql/Sql/Failover Group/Services/AzureSqlFailoverGroupCommunicator.cs @@ -43,8 +43,7 @@ public class AzureSqlFailoverGroupCommunicator /// /// Creates a communicator for Azure SQL Database Failover Group /// - /// - /// + /// The current azure context public AzureSqlFailoverGroupCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/FirewallRule/Services/AzureSqlServerFirewallRuleAdapter.cs b/src/Sql/Sql/FirewallRule/Services/AzureSqlServerFirewallRuleAdapter.cs index 6ced381eb903..1e068b8da85a 100644 --- a/src/Sql/Sql/FirewallRule/Services/AzureSqlServerFirewallRuleAdapter.cs +++ b/src/Sql/Sql/FirewallRule/Services/AzureSqlServerFirewallRuleAdapter.cs @@ -41,8 +41,7 @@ public class AzureSqlServerFirewallRuleAdapter /// /// Constructs a firewall rule adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlServerFirewallRuleAdapter(IAzureContext context) { Context = context; @@ -80,8 +79,6 @@ public List ListFirewallRules(string resourceGr /// /// Upserts a Firewall Rule /// - /// The name of the resource group - /// The name of ther server /// The firewall rule to create /// The updated server model public AzureSqlServerFirewallRuleModel UpsertFirewallRule(AzureSqlServerFirewallRuleModel model) diff --git a/src/Sql/Sql/FirewallRule/Services/AzureSqlServerFirewallRuleCommunicator.cs b/src/Sql/Sql/FirewallRule/Services/AzureSqlServerFirewallRuleCommunicator.cs index b922c9b5097a..796ad46d3ef8 100644 --- a/src/Sql/Sql/FirewallRule/Services/AzureSqlServerFirewallRuleCommunicator.cs +++ b/src/Sql/Sql/FirewallRule/Services/AzureSqlServerFirewallRuleCommunicator.cs @@ -45,8 +45,7 @@ public class AzureSqlServerFirewallRuleCommunicator /// /// Creates a communicator for Azure Sql Databases FirewallRules /// - /// - /// + /// The current azure context public AzureSqlServerFirewallRuleCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/ImportExport/Cmdlet/GetAzureSqlDatabaseImportExportStatus.cs b/src/Sql/Sql/ImportExport/Cmdlet/GetAzureSqlDatabaseImportExportStatus.cs index 5d66c630c795..fe1ec842031e 100644 --- a/src/Sql/Sql/ImportExport/Cmdlet/GetAzureSqlDatabaseImportExportStatus.cs +++ b/src/Sql/Sql/ImportExport/Cmdlet/GetAzureSqlDatabaseImportExportStatus.cs @@ -62,7 +62,6 @@ protected AzureSqlDatabaseImportExportStatusModel GetEntity() /// /// Intializes the model adapter /// - /// The subscription the cmdlets are operation under /// The server adapter protected ImportExportDatabaseAdapter InitModelAdapter() { diff --git a/src/Sql/Sql/ImportExport/Service/ImportExportDatabaseAdapter.cs b/src/Sql/Sql/ImportExport/Service/ImportExportDatabaseAdapter.cs index af2323970764..5d263aa47976 100644 --- a/src/Sql/Sql/ImportExport/Service/ImportExportDatabaseAdapter.cs +++ b/src/Sql/Sql/ImportExport/Service/ImportExportDatabaseAdapter.cs @@ -42,8 +42,7 @@ public class ImportExportDatabaseAdapter /// /// Constructs a firewall rule adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public ImportExportDatabaseAdapter(IAzureContext context) { Context = context; @@ -169,6 +168,8 @@ public AzureSqlDatabaseImportExportStatusModel GetStatus(string operationStatusL /// Creates the response model given server response /// /// Server Response + /// + /// /// Response Model private AzureSqlDatabaseImportExportBaseModel CreateImportExportResponse(ImportExportOperationResult response, AzureSqlDatabaseImportExportBaseModel originalModel, Uri statusLink) { diff --git a/src/Sql/Sql/ImportExport/Service/ImportExportDatabaseCommunicator.cs b/src/Sql/Sql/ImportExport/Service/ImportExportDatabaseCommunicator.cs index ea5b2beb9d7f..b29d295b3207 100644 --- a/src/Sql/Sql/ImportExport/Service/ImportExportDatabaseCommunicator.cs +++ b/src/Sql/Sql/ImportExport/Service/ImportExportDatabaseCommunicator.cs @@ -59,8 +59,7 @@ public class ImportExportDatabaseCommunicator /// /// Creates a communicator for Azure Sql Databases /// - /// - /// + /// The current azure context public ImportExportDatabaseCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/Index Recommendations/Service/AzureSqlDatabaseIndexRecommendationAdapter.cs b/src/Sql/Sql/Index Recommendations/Service/AzureSqlDatabaseIndexRecommendationAdapter.cs index ab542b43bf62..1dac78c952af 100644 --- a/src/Sql/Sql/Index Recommendations/Service/AzureSqlDatabaseIndexRecommendationAdapter.cs +++ b/src/Sql/Sql/Index Recommendations/Service/AzureSqlDatabaseIndexRecommendationAdapter.cs @@ -43,8 +43,7 @@ public class AzureSqlDatabaseIndexRecommendationAdapter /// /// Constructs adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlDatabaseIndexRecommendationAdapter(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/Index Recommendations/Service/AzureSqlDatabaseIndexRecommendationCommunicator.cs b/src/Sql/Sql/Index Recommendations/Service/AzureSqlDatabaseIndexRecommendationCommunicator.cs index 21c881fb7cdb..9fadad3397ed 100644 --- a/src/Sql/Sql/Index Recommendations/Service/AzureSqlDatabaseIndexRecommendationCommunicator.cs +++ b/src/Sql/Sql/Index Recommendations/Service/AzureSqlDatabaseIndexRecommendationCommunicator.cs @@ -51,8 +51,7 @@ public class AzureSqlDatabaseIndexRecommendationCommunicator /// /// Creates a communicator for Azure Sql Databases /// - /// - /// + /// The current azure context public AzureSqlDatabaseIndexRecommendationCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/Instance Failover Group/Cmdlet/AzureSqlInstanceFailoverGroupCmdletBase.cs b/src/Sql/Sql/Instance Failover Group/Cmdlet/AzureSqlInstanceFailoverGroupCmdletBase.cs index b854c701fb48..4f994bc27620 100644 --- a/src/Sql/Sql/Instance Failover Group/Cmdlet/AzureSqlInstanceFailoverGroupCmdletBase.cs +++ b/src/Sql/Sql/Instance Failover Group/Cmdlet/AzureSqlInstanceFailoverGroupCmdletBase.cs @@ -48,7 +48,6 @@ public abstract class AzureSqlInstanceFailoverGroupCmdletBase : AzureSqlCmdletBa /// /// Initializes the Azure Sql Instance Failover Group Adapter /// - /// /// protected override AzureSqlInstanceFailoverGroupAdapter InitModelAdapter() { diff --git a/src/Sql/Sql/Instance Failover Group/Services/AzureSqlInstanceFailoverGroupAdapter.cs b/src/Sql/Sql/Instance Failover Group/Services/AzureSqlInstanceFailoverGroupAdapter.cs index eac0424a3b7a..2a5d92d4aa0c 100644 --- a/src/Sql/Sql/Instance Failover Group/Services/AzureSqlInstanceFailoverGroupAdapter.cs +++ b/src/Sql/Sql/Instance Failover Group/Services/AzureSqlInstanceFailoverGroupAdapter.cs @@ -45,8 +45,7 @@ public class AzureSqlInstanceFailoverGroupAdapter /// /// Constructs a database adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlInstanceFailoverGroupAdapter(IAzureContext context) { _subscription = context.Subscription; @@ -87,8 +86,6 @@ internal ICollection ListInstanceFailoverGro /// /// Creates or updates an Azure Sql Database InstanceFailoverGroup. /// - /// The name of the resource group - /// The name of the local region /// The input parameters for the create/update operation /// The upserted Azure Sql Database InstanceFailoverGroup internal AzureSqlInstanceFailoverGroupModel UpsertInstanceFailoverGroup(AzureSqlInstanceFailoverGroupModel model) @@ -137,18 +134,18 @@ internal AzureSqlInstanceFailoverGroupModel UpsertInstanceFailoverGroup(AzureSql /// /// The resource group the server is in /// The name of the local region - /// The name of the Azure SQL Database Failover Group to delete + /// The name of the Azure SQL Database Failover Group to delete public void RemoveInstanceFailoverGroup(string resourceGroupName, string location, string failoverGroupName) { Communicator.Remove(resourceGroupName, location, failoverGroupName); } - + /// /// Finds and removes the Secondary Link by the secondary resource group and Azure SQL Server /// /// The name of the Resource Group containing the primary database /// The name of the partner location - /// The name of the Resource Group containing the secondary database + /// The name of the Azure Sql Database FailoverGroup /// Whether the failover operation will allow data loss /// The Azure SQL Database ReplicationLink object internal AzureSqlInstanceFailoverGroupModel Failover(string resourceGroupName, string location, string failoverGroupName, bool allowDataLoss) @@ -168,9 +165,7 @@ internal AzureSqlInstanceFailoverGroupModel Failover(string resourceGroupName, s /// /// Converts the response from the service to a powershell database object /// - /// The resource group the server is in - /// The name of the local region - /// The service response + /// Recommended Action object /// The converted model private AzureSqlInstanceFailoverGroupModel CreateInstanceFailoverGroupModelFromResponse(Management.Sql.Models.InstanceFailoverGroup failoverGroup) { diff --git a/src/Sql/Sql/Instance Failover Group/Services/AzureSqlInstanceFailoverGroupCommunicator.cs b/src/Sql/Sql/Instance Failover Group/Services/AzureSqlInstanceFailoverGroupCommunicator.cs index 011669361e46..dff3bf0c952d 100644 --- a/src/Sql/Sql/Instance Failover Group/Services/AzureSqlInstanceFailoverGroupCommunicator.cs +++ b/src/Sql/Sql/Instance Failover Group/Services/AzureSqlInstanceFailoverGroupCommunicator.cs @@ -46,8 +46,7 @@ public class AzureSqlInstanceFailoverGroupCommunicator /// /// Creates a communicator for Instance Failover Group /// - /// - /// + /// The current azure context public AzureSqlInstanceFailoverGroupCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/Instance Pools/Cmdlet/SetAzureSqlInstancePool.cs b/src/Sql/Sql/Instance Pools/Cmdlet/SetAzureSqlInstancePool.cs index f6628f9d0d15..50d97136e3d9 100644 --- a/src/Sql/Sql/Instance Pools/Cmdlet/SetAzureSqlInstancePool.cs +++ b/src/Sql/Sql/Instance Pools/Cmdlet/SetAzureSqlInstancePool.cs @@ -155,7 +155,7 @@ protected override IEnumerable GetEntity() /// /// Generates the updated model from user input. /// - /// The existing instance pool model + /// The existing instance pool model /// The generated model from user input protected override IEnumerable ApplyUserInputToModel(IEnumerable model) { diff --git a/src/Sql/Sql/InstanceActiveDirectoryAdministrator/Services/AzureSqlInstanceActiveDirectoryAdministratorAdapter.cs b/src/Sql/Sql/InstanceActiveDirectoryAdministrator/Services/AzureSqlInstanceActiveDirectoryAdministratorAdapter.cs index df154ee10714..409f1cdfe0c5 100644 --- a/src/Sql/Sql/InstanceActiveDirectoryAdministrator/Services/AzureSqlInstanceActiveDirectoryAdministratorAdapter.cs +++ b/src/Sql/Sql/InstanceActiveDirectoryAdministrator/Services/AzureSqlInstanceActiveDirectoryAdministratorAdapter.cs @@ -66,8 +66,7 @@ public MicrosoftGraphClient MicrosoftGraphClient /// /// Constructs a Azure SQL Instance Active Directory administrator adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlInstanceActiveDirectoryAdministratorAdapter(IAzureContext context) { Context = context; @@ -134,7 +133,7 @@ public void RemoveInstanceActiveDirectoryAdministrator(string resourceGroupName, /// /// Converts the response from the service to a powershell database object /// - /// The resource group the managed instance is in + /// The resource group the managed instance is in /// The name of the Azure Sql InstanceActiveDirectoryAdministrator Managed Instance /// The service response /// The converted model diff --git a/src/Sql/Sql/InstanceActiveDirectoryAdministrator/Services/AzureSqlInstanceActiveDirectoryAdministratorCommunicator.cs b/src/Sql/Sql/InstanceActiveDirectoryAdministrator/Services/AzureSqlInstanceActiveDirectoryAdministratorCommunicator.cs index 191d6b4a8c64..37937372bc70 100644 --- a/src/Sql/Sql/InstanceActiveDirectoryAdministrator/Services/AzureSqlInstanceActiveDirectoryAdministratorCommunicator.cs +++ b/src/Sql/Sql/InstanceActiveDirectoryAdministrator/Services/AzureSqlInstanceActiveDirectoryAdministratorCommunicator.cs @@ -47,8 +47,7 @@ public class AzureSqlInstanceActiveDirectoryAdministratorCommunicator /// /// Creates a communicator for Azure SQL Instance Active Directory administrator /// - /// - /// + /// The current azure context public AzureSqlInstanceActiveDirectoryAdministratorCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/InstanceActiveDirectoryOnlyAuthentication/Services/AzureSqlInstanceActiveDirectoryOnlyAuthenticationAdapter.cs b/src/Sql/Sql/InstanceActiveDirectoryOnlyAuthentication/Services/AzureSqlInstanceActiveDirectoryOnlyAuthenticationAdapter.cs index e662ab6f232b..5512382a7c13 100644 --- a/src/Sql/Sql/InstanceActiveDirectoryOnlyAuthentication/Services/AzureSqlInstanceActiveDirectoryOnlyAuthenticationAdapter.cs +++ b/src/Sql/Sql/InstanceActiveDirectoryOnlyAuthentication/Services/AzureSqlInstanceActiveDirectoryOnlyAuthenticationAdapter.cs @@ -39,8 +39,7 @@ public class AzureSqlInstanceActiveDirectoryOnlyAuthenticationAdapter /// /// Constructs a Azure SQL Managed Instance Active Directory only authentication administrator adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlInstanceActiveDirectoryOnlyAuthenticationAdapter(IAzureContext context) { Context = context; @@ -80,6 +79,7 @@ internal ICollection Lis /// /// The name of the resource group /// The name of the Azure Sql Managed Instance + /// /// The upserted Azure SQL Managed Insance AD Only Authentication internal AzureSqlInstanceActiveDirectoryOnlyAuthenticationModel UpsertAzureADOnlyAuthenticaion(string resourceGroup, string InstanceName, AzureSqlInstanceActiveDirectoryOnlyAuthenticationModel model) { @@ -91,9 +91,9 @@ internal AzureSqlInstanceActiveDirectoryOnlyAuthenticationModel UpsertAzureADOnl /// /// Converts the response from the service to a powershell database object /// - /// The resource group the instance is in + /// The resource group the instance is in /// The name of the Azure Sql Managed Instance - /// The service response + /// /// The converted model public static AzureSqlInstanceActiveDirectoryOnlyAuthenticationModel CreateInstanceActiveDirectoryOnlyAuthenticationModelFromResponse(string resourceGroup, string InstanceName, Management.Sql.Models.ManagedInstanceAzureADOnlyAuthentication serverAzureADOnlyAuthentication) { diff --git a/src/Sql/Sql/InstanceActiveDirectoryOnlyAuthentication/Services/AzureSqlInstanceActiveDirectoryOnlyAuthenticationCommunicator.cs b/src/Sql/Sql/InstanceActiveDirectoryOnlyAuthentication/Services/AzureSqlInstanceActiveDirectoryOnlyAuthenticationCommunicator.cs index b4cb91cd752a..a7e213655c63 100644 --- a/src/Sql/Sql/InstanceActiveDirectoryOnlyAuthentication/Services/AzureSqlInstanceActiveDirectoryOnlyAuthenticationCommunicator.cs +++ b/src/Sql/Sql/InstanceActiveDirectoryOnlyAuthentication/Services/AzureSqlInstanceActiveDirectoryOnlyAuthenticationCommunicator.cs @@ -53,8 +53,7 @@ public class AzureSqlInstanceActiveDirectoryOnlyAuthenticationCommunicator /// /// Creates a communicator for Azure SQL Managed Instance Active Directory administrator /// - /// - /// + /// The current azure context public AzureSqlInstanceActiveDirectoryOnlyAuthenticationCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/LedgerDigestUploads/Services/AzureSqlDatabaseLedgerDigestUploadAdapter.cs b/src/Sql/Sql/LedgerDigestUploads/Services/AzureSqlDatabaseLedgerDigestUploadAdapter.cs index 75fe28b4dfdd..a46b31de2be5 100644 --- a/src/Sql/Sql/LedgerDigestUploads/Services/AzureSqlDatabaseLedgerDigestUploadAdapter.cs +++ b/src/Sql/Sql/LedgerDigestUploads/Services/AzureSqlDatabaseLedgerDigestUploadAdapter.cs @@ -40,8 +40,7 @@ public class AzureSqlDatabaseLedgerDigestUploadAdapter /// /// Constructs a database backup adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlDatabaseLedgerDigestUploadAdapter(IAzureContext context) { Context = context; @@ -62,8 +61,8 @@ internal AzureSqlDatabaseLedgerDigestUploadModel GetLedgerDigestUpload( string databaseName) { Management.Sql.Models.LedgerDigestUploads configuration = Communicator.GetLedgerDigestUpload( - resourceGroup, - serverName, + resourceGroup, + serverName, databaseName); return new AzureSqlDatabaseLedgerDigestUploadModel(resourceGroup, serverName, databaseName, configuration); @@ -72,9 +71,6 @@ internal AzureSqlDatabaseLedgerDigestUploadModel GetLedgerDigestUpload( /// /// Create or update a ledger digest upload configuration for a Azure SQL Database /// - /// The name of the resource group - /// The name of the Azure SQL Server - /// The name of the Azure SQL Database /// AzureSqlDatabaseLedgerDigestUploadModel model /// A ledger digest upload internal AzureSqlDatabaseLedgerDigestUploadModel SetLedgerDigestUpload(AzureSqlDatabaseLedgerDigestUploadModel model) @@ -85,8 +81,8 @@ internal AzureSqlDatabaseLedgerDigestUploadModel SetLedgerDigestUpload(AzureSqlD }; Communicator.SetLedgerDigestUpload( - model.ResourceGroupName, - model.ServerName, + model.ResourceGroupName, + model.ServerName, model.DatabaseName, config); @@ -96,9 +92,7 @@ internal AzureSqlDatabaseLedgerDigestUploadModel SetLedgerDigestUpload(AzureSqlD /// /// Disables ledger digest upload configuration for a Azure SQL Database /// - /// The name of the resource group - /// The name of the Azure SQL Server - /// The name of the Azure SQL Database + /// AzureSqlDatabaseLedgerDigestUploadModel model /// A ledger digest upload internal AzureSqlDatabaseLedgerDigestUploadModel DisableLedgerDigestUpload(AzureSqlDatabaseLedgerDigestUploadModel model) { diff --git a/src/Sql/Sql/LedgerDigestUploads/Services/AzureSqlDatabaseLedgerDigestUploadCommunicator.cs b/src/Sql/Sql/LedgerDigestUploads/Services/AzureSqlDatabaseLedgerDigestUploadCommunicator.cs index 05323daf8864..02fc38ff74f0 100644 --- a/src/Sql/Sql/LedgerDigestUploads/Services/AzureSqlDatabaseLedgerDigestUploadCommunicator.cs +++ b/src/Sql/Sql/LedgerDigestUploads/Services/AzureSqlDatabaseLedgerDigestUploadCommunicator.cs @@ -48,8 +48,7 @@ public class AzureSqlDatabaseLedgerDigestUploadCommunicator /// /// Creates a communicator for Azure Sql Database ledger digest upload REST endpoints. /// - /// Azure profile - /// Associated subscription + /// Azure context public AzureSqlDatabaseLedgerDigestUploadCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/Location Capabilities/Services/AzureSqlCapabilitiesAdapter.cs b/src/Sql/Sql/Location Capabilities/Services/AzureSqlCapabilitiesAdapter.cs index 2c6bc6ef0ba9..f6ced93196d0 100644 --- a/src/Sql/Sql/Location Capabilities/Services/AzureSqlCapabilitiesAdapter.cs +++ b/src/Sql/Sql/Location Capabilities/Services/AzureSqlCapabilitiesAdapter.cs @@ -35,8 +35,7 @@ public class AzureSqlCapabilitiesAdapter /// /// Constructs a firewall rule adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlCapabilitiesAdapter(IAzureContext context) { Context = context; @@ -121,7 +120,7 @@ private ServiceObjectiveCapabilityModel CreateSupportedSLOModel(Management.Sql.M /// /// Converts from an API object to a PowerShell object /// - /// The object to transform + /// The object to transform /// The converted database max size capability model private MaxSizeRangeCapabilityModel CreateSupportedMaxSizeModel(Management.Sql.Models.MaxSizeRangeCapability m) { diff --git a/src/Sql/Sql/Location Capabilities/Services/AzureSqlCapabilitiesCommunicator.cs b/src/Sql/Sql/Location Capabilities/Services/AzureSqlCapabilitiesCommunicator.cs index 5c220ae78242..284dc8ac2951 100644 --- a/src/Sql/Sql/Location Capabilities/Services/AzureSqlCapabilitiesCommunicator.cs +++ b/src/Sql/Sql/Location Capabilities/Services/AzureSqlCapabilitiesCommunicator.cs @@ -43,8 +43,7 @@ public class AzureSqlCapabilitiesCommunicator /// /// Creates a communicator for Azure Sql Databases FirewallRules /// - /// - /// + /// The current azure context public AzureSqlCapabilitiesCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/ManagedDatabase Backup/Model/AzureSqlManagedDatabaseBackupShortTermRetentionPolicyModel.cs b/src/Sql/Sql/ManagedDatabase Backup/Model/AzureSqlManagedDatabaseBackupShortTermRetentionPolicyModel.cs index 1814c655d0a3..d686919cb2f7 100644 --- a/src/Sql/Sql/ManagedDatabase Backup/Model/AzureSqlManagedDatabaseBackupShortTermRetentionPolicyModel.cs +++ b/src/Sql/Sql/ManagedDatabase Backup/Model/AzureSqlManagedDatabaseBackupShortTermRetentionPolicyModel.cs @@ -50,7 +50,7 @@ public class AzureSqlManagedDatabaseBackupShortTermRetentionPolicyModel /// /// Resource group /// Managed Instance name - /// Managed Instance name + /// Managed Database name /// Managed Database object /// Deletion date of the database, if it is deleted public AzureSqlManagedDatabaseBackupShortTermRetentionPolicyModel(string resourceGroup, string managedInstanceName, string managedDatabaseName, ManagedBackupShortTermRetentionPolicy managedBackupRetentionPolicy, DateTime? deletionDate = null) diff --git a/src/Sql/Sql/ManagedDatabase Backup/Services/AzureSqlManagedDatabaseBackupAdapter.cs b/src/Sql/Sql/ManagedDatabase Backup/Services/AzureSqlManagedDatabaseBackupAdapter.cs index 8394e2f6d2ea..1f8fb193222e 100644 --- a/src/Sql/Sql/ManagedDatabase Backup/Services/AzureSqlManagedDatabaseBackupAdapter.cs +++ b/src/Sql/Sql/ManagedDatabase Backup/Services/AzureSqlManagedDatabaseBackupAdapter.cs @@ -49,8 +49,7 @@ public class AzureSqlManagedDatabaseBackupAdapter /// /// Constructs a managed database adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlManagedDatabaseBackupAdapter(IAzureContext context) { Context = context; @@ -91,6 +90,7 @@ internal ICollection /// The name of the resource group /// The name of the Azure Sql Database Managed Instance /// The name of the Azure Sql Managed Database + /// Deletion date for deleted databases /// The Azure Sql Database object internal AzureSqlManagedDatabaseBackupShortTermRetentionPolicyModel ManagedBackupShortTermRetentionPoliciesDropped(string resourceGroupName, string managedInstanceName, string databaseName, DateTime deletionDate) { @@ -103,6 +103,8 @@ internal AzureSqlManagedDatabaseBackupShortTermRetentionPolicyModel ManagedBacku /// /// The name of the resource group /// The name of the Azure Sql Database Managed Instance + /// The name of the Azure Sql Managed Database + /// Deletion date for deleted databases /// A list of database objects internal ICollection ListManagedBackupShortTermRetentionPoliciesDropped(string resourceGroupName, string managedInstanceName, string databaseName, DateTime deletionDate) { @@ -149,7 +151,8 @@ internal AzureSqlManagedDatabaseBackupShortTermRetentionPolicyModel UpsertDelete /// /// The resource group the managed instance is in /// The name of the Azure Sql Database Managed Instance - /// The service response + /// The name of the Azure Sql Managed Database + /// /// Deletion date for deleted databases /// The converted model public static AzureSqlManagedDatabaseBackupShortTermRetentionPolicyModel CreateManagedDatabaseBackupShortTermRetentionPolicyModelFromResponse(string resourceGroup, string managedInstanceName, string managedDatabaseName, ManagedBackupShortTermRetentionPolicy managedBackupShortTermRetentionPolicy, DateTime? deletionDate = null) @@ -209,7 +212,6 @@ internal AzureSqlDeletedManagedDatabaseBackupModel GetDeletedDatabaseBackup(stri /// The name of the resource group /// The name of the Managed Instance /// The name of the Managed Database - /// Whether or not the user provided the Current switch to get the current implementation of LTR policy /// A backup LongTermRetention policy internal AzureSqlManagedDatabaseBackupLongTermRetentionPolicyModel GetManagedDatabaseLongTermRetentionPolicy( string resourceGroup, @@ -238,6 +240,7 @@ internal AzureSqlManagedDatabaseBackupLongTermRetentionPolicyModel GetManagedDat /// The name of the resource group /// The name of the Azure SQL Server /// The name of the Azure SQL Database + /// /// A backup LongTermRetention policy internal AzureSqlManagedDatabaseBackupLongTermRetentionPolicyModel SetManagedDatabaseBackupLongTermRetentionPolicy( string resourceGroup, diff --git a/src/Sql/Sql/ManagedDatabase Backup/Services/AzureSqlManagedDatabaseBackupCommunicator.cs b/src/Sql/Sql/ManagedDatabase Backup/Services/AzureSqlManagedDatabaseBackupCommunicator.cs index 5dadd53226b5..8c74058b5116 100644 --- a/src/Sql/Sql/ManagedDatabase Backup/Services/AzureSqlManagedDatabaseBackupCommunicator.cs +++ b/src/Sql/Sql/ManagedDatabase Backup/Services/AzureSqlManagedDatabaseBackupCommunicator.cs @@ -48,8 +48,7 @@ public class AzureSqlManagedDatabaseBackupCommunicator /// /// Creates a communicator for Azure Sql Managed Databases /// - /// - /// + /// The current azure context public AzureSqlManagedDatabaseBackupCommunicator(IAzureContext context) { Context = context; @@ -124,7 +123,7 @@ private Management.Sql.SqlManagementClient GetCurrentSqlClient() /// /// Lists the restorable deleted databases for a given Sql Azure Server /// - /// The name of the resource group + /// The name of the resource group /// The name of the Azure SQL Server /// List of restore points public IList ListDeletedDatabaseBackups(string resourceGroupName, string serverName) @@ -136,7 +135,7 @@ public IList ListDeletedDatabaseBackups(string /// /// Get a restorable deleted database for a given Sql Azure Database /// - /// The name of the resource group + /// The name of the resource group /// The name of the Managed Instance /// The name of the Managed database /// List of restore points @@ -148,7 +147,7 @@ public RestorableDroppedManagedDatabase GetDeletedDatabaseBackup(string resource /// /// Get a backup LongTermRetention policy for a Azure SQL Database /// - /// The name of the resource group + /// The name of the resource group /// The name of the Managed Instance /// The name of the Managed Database /// A backup LongTermRetention policy diff --git a/src/Sql/Sql/ManagedDatabase/Cmdlet/AzureSqlRecoverableManagedDatabaseCmdletBase.cs b/src/Sql/Sql/ManagedDatabase/Cmdlet/AzureSqlRecoverableManagedDatabaseCmdletBase.cs index 03123355667f..42ec91bf6716 100644 --- a/src/Sql/Sql/ManagedDatabase/Cmdlet/AzureSqlRecoverableManagedDatabaseCmdletBase.cs +++ b/src/Sql/Sql/ManagedDatabase/Cmdlet/AzureSqlRecoverableManagedDatabaseCmdletBase.cs @@ -34,7 +34,6 @@ public abstract class AzureSqlRecoverableManagedDatabaseCmdletBase : Azu /// /// Initializes the adapter /// - /// /// protected override AzureSqlRecoverableManagedDatabaseAdapter InitModelAdapter() { diff --git a/src/Sql/Sql/ManagedDatabase/Services/AzureSqlManagedDatabaseAdapter.cs b/src/Sql/Sql/ManagedDatabase/Services/AzureSqlManagedDatabaseAdapter.cs index 421b8e1bb52b..dcae866085ac 100644 --- a/src/Sql/Sql/ManagedDatabase/Services/AzureSqlManagedDatabaseAdapter.cs +++ b/src/Sql/Sql/ManagedDatabase/Services/AzureSqlManagedDatabaseAdapter.cs @@ -47,8 +47,7 @@ public class AzureSqlManagedDatabaseAdapter /// /// Constructs a managed database adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlManagedDatabaseAdapter(IAzureContext context) { Context = context; @@ -229,7 +228,8 @@ public void CompleteManagedDatabaseLogReplay(AzureSqlManagedDatabaseModel parame parameters.LastBackupName); } - private bool isLRSRestore(ManagedDatabaseRestoreDetailsResult restoreDetails) { + private bool isLRSRestore(ManagedDatabaseRestoreDetailsResult restoreDetails) + { var restoreType = restoreDetails.GetType().GetProperty("ManagedDatabaseRestoreDetailsResultType"); // 1) no property => old api => it's log replay as the API only works for such dbs // 2) property is there => new api => property value will tell us if its LRS @@ -259,7 +259,7 @@ public void StopManagedDatabaseLogReplay(AzureSqlManagedDatabaseModel parameters /// /// The resource group the managed instance is in /// The name of the Azure Sql Database Managed Instance - /// The service response + /// The service response /// The converted model public static AzureSqlManagedDatabaseModel CreateManagedDatabaseModelFromResponse(string resourceGroup, string managedInstanceName, Management.Sql.Models.ManagedDatabase managedDatabase) { diff --git a/src/Sql/Sql/ManagedDatabase/Services/AzureSqlManagedDatabaseCommunicator.cs b/src/Sql/Sql/ManagedDatabase/Services/AzureSqlManagedDatabaseCommunicator.cs index 627b3b9800ec..386a26a72e0b 100644 --- a/src/Sql/Sql/ManagedDatabase/Services/AzureSqlManagedDatabaseCommunicator.cs +++ b/src/Sql/Sql/ManagedDatabase/Services/AzureSqlManagedDatabaseCommunicator.cs @@ -43,8 +43,7 @@ public class AzureSqlManagedDatabaseCommunicator /// /// Creates a communicator for Azure Sql Managed Databases /// - /// - /// + /// The current azure context public AzureSqlManagedDatabaseCommunicator(IAzureContext context) { Context = context; @@ -89,9 +88,9 @@ public void Remove(string resourceGroupName, string managedInstanceName, string /// /// Restore a given Sql Azure Managed Database /// - /// The name of the resource group + /// The name of the resource group /// The name of the Azure SQL Managed Instance - /// The name of the Azure SQL Managed database + /// The name of the Azure SQL Managed database /// Model describing the managed database restore request /// Restored database object public Management.Sql.Models.ManagedDatabase RestoreDatabase(string resourceGroupName, string managedInstanceName, string managedDatabaseName, Management.Sql.Models.ManagedDatabase model) @@ -102,10 +101,11 @@ public Management.Sql.Models.ManagedDatabase RestoreDatabase(string resourceGrou /// /// Restore a given Sql Azure Managed Database /// - /// The name of the resource group + /// The name of the resource group /// The name of the Azure SQL Managed Instance - /// The name of the Azure SQL Managed database - /// Parameters describing the managed database restore request + /// The name of the Azure SQL Managed database + /// The resource ID of the Azure SQL Managed database + /// Model describing the managed database restore request /// Restored database object public Management.Sql.Models.ManagedDatabase RecoverDatabase(string resourceGroupName, string managedInstanceName, string managedDatabaseName, string resourceId, AzureSqlRecoverableManagedDatabaseModel model) { diff --git a/src/Sql/Sql/ManagedDatabase/Services/AzureSqlRecoverableManagedDatabaseAdapter.cs b/src/Sql/Sql/ManagedDatabase/Services/AzureSqlRecoverableManagedDatabaseAdapter.cs index 685846713235..bcfe4afa80eb 100644 --- a/src/Sql/Sql/ManagedDatabase/Services/AzureSqlRecoverableManagedDatabaseAdapter.cs +++ b/src/Sql/Sql/ManagedDatabase/Services/AzureSqlRecoverableManagedDatabaseAdapter.cs @@ -48,8 +48,7 @@ public class AzureSqlRecoverableManagedDatabaseAdapter /// /// Constructs a recoverable managed database adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlRecoverableManagedDatabaseAdapter(IAzureContext context) { Context = context; @@ -88,7 +87,7 @@ internal ICollection ListRecoverableMan /// /// The resource group the managed instance is in /// The name of the Azure Sql Database Managed Instance - /// The service response + /// The service response /// The converted model public static AzureSqlRecoverableManagedDatabaseModel CreateRecoverableManagedDatabaseModelFromResponse(string resourceGroup, string managedInstanceName, Management.Sql.Models.RecoverableManagedDatabase managedDatabase) { diff --git a/src/Sql/Sql/ManagedDatabase/Services/AzureSqlRecoverableManagedDatabaseCommunicator.cs b/src/Sql/Sql/ManagedDatabase/Services/AzureSqlRecoverableManagedDatabaseCommunicator.cs index ffe4cbec852f..6766bcbfddd7 100644 --- a/src/Sql/Sql/ManagedDatabase/Services/AzureSqlRecoverableManagedDatabaseCommunicator.cs +++ b/src/Sql/Sql/ManagedDatabase/Services/AzureSqlRecoverableManagedDatabaseCommunicator.cs @@ -47,8 +47,7 @@ public class AzureSqlRecoverableManagedDatabaseCommunicator /// /// Creates a communicator for Recoverable Azure Sql Managed Databases /// - /// - /// + /// The current azure context public AzureSqlRecoverableManagedDatabaseCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/ManagedInstance/Cmdlet/NewAzureSqlManagedInstance.cs b/src/Sql/Sql/ManagedInstance/Cmdlet/NewAzureSqlManagedInstance.cs index 3f1321de485b..a6555187457f 100644 --- a/src/Sql/Sql/ManagedInstance/Cmdlet/NewAzureSqlManagedInstance.cs +++ b/src/Sql/Sql/ManagedInstance/Cmdlet/NewAzureSqlManagedInstance.cs @@ -343,14 +343,14 @@ public class NewAzureSqlManagedInstance : ManagedInstanceCmdletBase HelpMessage = "The Key Vault URI for encryption")] public string KeyId { get; set; } - // + /// /// List of user assigned identities. /// [Parameter(Mandatory = false, HelpMessage = "List of user assigned identities")] public List UserAssignedIdentityId { get; set; } - // + /// /// Type of identity to be assigned to the server.. /// [Parameter(Mandatory = false, @@ -397,7 +397,7 @@ public class NewAzureSqlManagedInstance : ManagedInstanceCmdletBase [Parameter(Mandatory = false, HelpMessage = "Use zone redundant storage")] public SwitchParameter ZoneRedundant { get; set; } - // + /// /// Gets or sets service principal type /// [Parameter(Mandatory = false, diff --git a/src/Sql/Sql/ManagedInstance/Cmdlet/SetAzureSqlManagedInstance.cs b/src/Sql/Sql/ManagedInstance/Cmdlet/SetAzureSqlManagedInstance.cs index 772daeb8cb9d..61a17f095451 100644 --- a/src/Sql/Sql/ManagedInstance/Cmdlet/SetAzureSqlManagedInstance.cs +++ b/src/Sql/Sql/ManagedInstance/Cmdlet/SetAzureSqlManagedInstance.cs @@ -231,14 +231,14 @@ public class SetAzureSqlManagedInstance : ManagedInstanceCmdletBase HelpMessage = "The Maintenance configuration id for the Sql Azure Managed Instance.")] public string MaintenanceConfigurationId { get; set; } - // + /// /// List of user assigned identities. /// [Parameter(Mandatory = false, HelpMessage = "List of user assigned identities")] public List UserAssignedIdentityId { get; set; } - // + /// /// List of user assigned identities. /// [Parameter(Mandatory = false, @@ -266,7 +266,7 @@ public class SetAzureSqlManagedInstance : ManagedInstanceCmdletBase [Parameter(Mandatory = false, HelpMessage = "Use zone redundant storage")] public SwitchParameter ZoneRedundant { get; set; } - // + /// /// Gets or sets service principal type /// [Parameter(Mandatory = false, diff --git a/src/Sql/Sql/ManagedInstance/Services/AzureSqlManagedInstanceAdapter.cs b/src/Sql/Sql/ManagedInstance/Services/AzureSqlManagedInstanceAdapter.cs index a2ab87d0bd3f..a05a09437dfd 100644 --- a/src/Sql/Sql/ManagedInstance/Services/AzureSqlManagedInstanceAdapter.cs +++ b/src/Sql/Sql/ManagedInstance/Services/AzureSqlManagedInstanceAdapter.cs @@ -90,6 +90,7 @@ public AzureSqlManagedInstanceAdapter(IAzureContext context) /// /// The name of the resource group /// The name of the managed instance + /// The child resources to include in the response. /// The managed instance public AzureSqlManagedInstanceModel GetManagedInstance(string resourceGroupName, string managedInstanceName, string expand = null) { @@ -122,6 +123,7 @@ public List ListManagedInstancesByInstancePool(str /// Gets a list of all the managed instances in a resource group /// /// The name of the resource group + /// The child resources to include in the response. /// A list of all the managed instances public List ListManagedInstancesByResourceGroup(string resourceGroupName, string expand = null) { @@ -220,7 +222,6 @@ public void RemoveManagedInstance(string resourceGroupName, string managedInstan /// /// Convert a Management.Sql.LegacySdk.Models.ManagedInstance to AzureSqlDatabaseManagedInstanceModel /// - /// The resource group the managed instance is in /// The management client managed instance response to convert /// The converted managed instance model private static AzureSqlManagedInstanceModel CreateManagedInstanceModelFromResponse(Management.Sql.Models.ManagedInstance resp) @@ -299,8 +300,7 @@ public static string GetInstanceSkuPrefix(string tier) /// /// Verifies that the Azure Active Directory user or group exists, and will get the object id if it is not set. /// - /// Azure Active Directory user or group display name - /// Azure Active Directory user or group object id + /// Azure Active Directory user or group object /// protected ManagedInstanceExternalAdministrator GetActiveDirectoryInformation(ManagedInstanceExternalAdministrator input) { diff --git a/src/Sql/Sql/ManagedInstance/Services/AzureSqlManagedInstanceCommunicator.cs b/src/Sql/Sql/ManagedInstance/Services/AzureSqlManagedInstanceCommunicator.cs index 367fabb7e1d0..11b140106170 100644 --- a/src/Sql/Sql/ManagedInstance/Services/AzureSqlManagedInstanceCommunicator.cs +++ b/src/Sql/Sql/ManagedInstance/Services/AzureSqlManagedInstanceCommunicator.cs @@ -48,8 +48,7 @@ public class AzureSqlManagedInstanceCommunicator /// /// Creates a communicator for Managed instance /// - /// - /// + /// The current azure context public AzureSqlManagedInstanceCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/ManagedInstanceOperation/Services/AzureSqlManagedInstanceOperationCommunicator.cs b/src/Sql/Sql/ManagedInstanceOperation/Services/AzureSqlManagedInstanceOperationCommunicator.cs index cb54dc3d8f23..5234d651011c 100644 --- a/src/Sql/Sql/ManagedInstanceOperation/Services/AzureSqlManagedInstanceOperationCommunicator.cs +++ b/src/Sql/Sql/ManagedInstanceOperation/Services/AzureSqlManagedInstanceOperationCommunicator.cs @@ -47,8 +47,7 @@ public class AzureSqlManagedInstanceOperationCommunicator /// /// Creates a communicator for Managed instance /// - /// - /// + /// The current azure context public AzureSqlManagedInstanceOperationCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/OutboundFirewallRules/Services/AzureSqlServerOutboundFirewallRulesCommunicator.cs b/src/Sql/Sql/OutboundFirewallRules/Services/AzureSqlServerOutboundFirewallRulesCommunicator.cs index 897886b10ad2..a27b30eb0c6a 100644 --- a/src/Sql/Sql/OutboundFirewallRules/Services/AzureSqlServerOutboundFirewallRulesCommunicator.cs +++ b/src/Sql/Sql/OutboundFirewallRules/Services/AzureSqlServerOutboundFirewallRulesCommunicator.cs @@ -43,8 +43,7 @@ public class AzureSqlServerOutboundFirewallRulesCommunicator /// /// Creates a communicator for Azure Sql Databases OutboundFirewallRules /// - /// - /// + /// The current azure context public AzureSqlServerOutboundFirewallRulesCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/Properties/Resources.Designer.cs b/src/Sql/Sql/Properties/Resources.Designer.cs index 7ad59c4a9fe6..758704e1a7a2 100644 --- a/src/Sql/Sql/Properties/Resources.Designer.cs +++ b/src/Sql/Sql/Properties/Resources.Designer.cs @@ -692,8 +692,8 @@ internal static string InvalidKeyId { /// /// Looks up a localized string similar to Invalid format of the resource identifier. A long term retention backup resource Id should follow one of the following formats: - ////subscriptions/<subscriptionId>/providers/Microsoft.Sql/locations/<location>/longTermRetentionServers/<serverName>/longTermRetentionDatabases/<databaseName>/longTermRetentionBackups/<backupName> - ////subscriptions/<subscriptionId>/resourceGroups/<resourceGroupName>/providers/Microsoft.Sql/locations/<location>/longTermRetentionServers/<serverName>/longTermRetentionDatabases/<databaseN [rest of string was truncated]";. + /// subscriptions/<subscriptionId>/providers/Microsoft.Sql/locations/<location>/longTermRetentionServers/<serverName>/longTermRetentionDatabases/<databaseName>/longTermRetentionBackups/<backupName> + /// subscriptions/<subscriptionId>/resourceGroups/<resourceGroupName>/providers/Microsoft.Sql/locations/<location>/longTermRetentionServers/<serverName>/longTermRetentionDatabases/<databaseN [rest of string was truncated]";. /// internal static string InvalidLongTermRetentionBackupResourceIdFormat { get { diff --git a/src/Sql/Sql/RecommendedElasticPools/Services/AzureSqlElasticPoolRecommendationAdapter.cs b/src/Sql/Sql/RecommendedElasticPools/Services/AzureSqlElasticPoolRecommendationAdapter.cs index bdc835100a6d..1006428a285a 100644 --- a/src/Sql/Sql/RecommendedElasticPools/Services/AzureSqlElasticPoolRecommendationAdapter.cs +++ b/src/Sql/Sql/RecommendedElasticPools/Services/AzureSqlElasticPoolRecommendationAdapter.cs @@ -44,8 +44,7 @@ public class AzureSqlElasticPoolRecommendationAdapter /// /// Constructs a recommended elastic pool adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlElasticPoolRecommendationAdapter(IAzureContext context) { _subscription = context.Subscription; diff --git a/src/Sql/Sql/RecommendedElasticPools/Services/AzureSqlElasticPoolRecommendationCommunicator.cs b/src/Sql/Sql/RecommendedElasticPools/Services/AzureSqlElasticPoolRecommendationCommunicator.cs index 139cd806df56..730157b2ddcf 100644 --- a/src/Sql/Sql/RecommendedElasticPools/Services/AzureSqlElasticPoolRecommendationCommunicator.cs +++ b/src/Sql/Sql/RecommendedElasticPools/Services/AzureSqlElasticPoolRecommendationCommunicator.cs @@ -44,8 +44,7 @@ public class AzureSqlElasticPoolRecommendationCommunicator /// /// Creates a communicator for Azure Sql Recommended Elastic Pool /// - /// - /// + /// The current azure context public AzureSqlElasticPoolRecommendationCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/Replication/Services/AzureSqlDatabaseReplicationAdapter.cs b/src/Sql/Sql/Replication/Services/AzureSqlDatabaseReplicationAdapter.cs index 37a020657104..0a779c5af6d4 100644 --- a/src/Sql/Sql/Replication/Services/AzureSqlDatabaseReplicationAdapter.cs +++ b/src/Sql/Sql/Replication/Services/AzureSqlDatabaseReplicationAdapter.cs @@ -54,8 +54,7 @@ public class AzureSqlDatabaseReplicationAdapter /// /// Constructs a database adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlDatabaseReplicationAdapter(IAzureContext context) { Context = context; @@ -165,8 +164,6 @@ internal AzureSqlDatabaseCopyModel CopyDatabaseWithNewSdk(string copyResourceGro /// The source's resource group name /// The source's Azure SQL Server name /// The source database name - /// The copy's target elastic pool - /// The copy's nondefault service level objective /// The database create response /// A powershell DatabaseCopy object private AzureSqlDatabaseCopyModel CreateDatabaseCopyModelFromDatabaseCreateOrUpdateResponse(string copyResourceGroupName, string copyServerName, string copyDatabaseName, @@ -196,7 +193,7 @@ private AzureSqlDatabaseCopyModel CreateDatabaseCopyModelFromDatabaseCreateOrUpd /// The source's resource group name /// The source's Azure SQL Server name /// The source database name - /// The database create response + /// The database create response /// A powershell DatabaseCopy object private AzureSqlDatabaseCopyModel CreateDatabaseCopyModelFromResponse(string copyResourceGroup, string copyServerName, string resourceGroupName, string serverName, string databaseName, Management.Sql.Models.Database database) @@ -337,8 +334,7 @@ private void CheckPartnerResourceGroupValid(string partnerResourceGroupName) /// The name of the Azure SQL Server containing the primary database /// The name of primary database /// The name of the Resource Group containing the secondary database - /// The linkId of the replication link to the secondary - /// The replication link response + /// The replication link response /// The Azure SQL Database ReplicationLink object private AzureReplicationLinkModel CreateReplicationLinkModelFromReplicationLinkResponse(string resourceGroupName, string serverName, string databaseName, string partnerResourceGroupName, Management.Sql.LegacySdk.Models.ReplicationLink resp) @@ -376,8 +372,7 @@ private AzureReplicationLinkModel CreateReplicationLinkModelFromReplicationLinkR /// The name of the Azure SQL Server containing the primary database /// The name of primary database /// The name of the Resource Group containing the secondary database - /// The linkId of the replication link to the secondary - /// The replication link response + /// The replication link response /// The Azure SQL Database ReplicationLink object private AzureReplicationLinkModel CreateReplicationLinkModelFromResponse(string resourceGroupName, string serverName, string databaseName, string partnerResourceGroupName, Management.Sql.Models.ReplicationLink resp) { @@ -447,7 +442,6 @@ internal void RemoveLink(string resourceGroupName, string serverName, string dat /// The name of the Azure SQL Server containing the primary database /// The name of primary database /// The name of the Resource Group containing the secondary database - /// The name of the Azure SQL Server containing the secondary database /// Whether the failover operation will allow data loss /// The Azure SQL Database ReplicationLink object internal AzureReplicationLinkModel FailoverLink(string resourceGroupName, string serverName, string databaseName, string partnerResourceGroupName, bool allowDataLoss) diff --git a/src/Sql/Sql/Replication/Services/AzureSqlDatabaseReplicationCommunicator.cs b/src/Sql/Sql/Replication/Services/AzureSqlDatabaseReplicationCommunicator.cs index a7bd13bbd9ef..9f0cf2270461 100644 --- a/src/Sql/Sql/Replication/Services/AzureSqlDatabaseReplicationCommunicator.cs +++ b/src/Sql/Sql/Replication/Services/AzureSqlDatabaseReplicationCommunicator.cs @@ -46,8 +46,7 @@ public class AzureSqlDatabaseReplicationCommunicator /// /// Creates a communicator for Azure SQL Databases /// - /// - /// + /// The current azure context public AzureSqlDatabaseReplicationCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/Server/Cmdlet/NewAzureSqlServer.cs b/src/Sql/Sql/Server/Cmdlet/NewAzureSqlServer.cs index 3adb7dcd3071..80b5ff28d3b8 100644 --- a/src/Sql/Sql/Server/Cmdlet/NewAzureSqlServer.cs +++ b/src/Sql/Sql/Server/Cmdlet/NewAzureSqlServer.cs @@ -108,14 +108,14 @@ public class NewAzureSqlServer : AzureSqlServerCmdletBase HelpMessage = "The Key Vault URI for encryption")] public string KeyId { get; set; } - // + /// /// List of user assigned identities. /// [Parameter(Mandatory = false, HelpMessage = "List of user assigned identities")] public List UserAssignedIdentityId { get; set; } - // + /// /// Type of identity to be assigned to the server.. /// [Parameter(Mandatory = false, diff --git a/src/Sql/Sql/Server/Cmdlet/SetAzureSqlServer.cs b/src/Sql/Sql/Server/Cmdlet/SetAzureSqlServer.cs index eb8ad0360ca9..a749c74bd0e8 100644 --- a/src/Sql/Sql/Server/Cmdlet/SetAzureSqlServer.cs +++ b/src/Sql/Sql/Server/Cmdlet/SetAzureSqlServer.cs @@ -110,14 +110,14 @@ public class SetAzureSqlServer : AzureSqlServerCmdletBase HelpMessage = "Specifies the Federated client ID of the server when using Cross-Tenant CMK, Do not set this value if you do not intent to use Cross-Tenant CMK")] public Guid? FederatedClientId { get; set; } - // + /// /// List of user assigned identities. /// [Parameter(Mandatory = false, HelpMessage = "List of user assigned identities")] public List UserAssignedIdentityId { get; set; } - // + /// /// Type of identity to be assigned to the server.. /// [Parameter(Mandatory = false, diff --git a/src/Sql/Sql/Server/Services/AzureSqlServerAdapter.cs b/src/Sql/Sql/Server/Services/AzureSqlServerAdapter.cs index 57c8df141aac..ffc5a7fcd29a 100644 --- a/src/Sql/Sql/Server/Services/AzureSqlServerAdapter.cs +++ b/src/Sql/Sql/Server/Services/AzureSqlServerAdapter.cs @@ -89,6 +89,7 @@ public AzureSqlServerAdapter(IAzureContext context) /// /// The name of the resource group /// The name of the server + /// The child resources to include in the response. /// The server public AzureSqlServerModel GetServer(string resourceGroupName, string serverName, string expand = null) { @@ -99,7 +100,7 @@ public AzureSqlServerModel GetServer(string resourceGroupName, string serverName /// /// Gets a list of all the servers in a subscription /// - /// The name of the resource group + /// The child resources to include in the response. /// A list of all the servers public List ListServers(string expand = null) { @@ -114,6 +115,7 @@ public List ListServers(string expand = null) /// Gets a list of all the servers in a resource group /// /// The name of the resource group + /// The child resources to include in the response. /// A list of all the servers public List ListServersByResourceGroup(string resourceGroupName, string expand = null) { @@ -164,7 +166,6 @@ public void RemoveServer(string resourceGroupName, string serverName) /// /// Convert a Management.Sql.LegacySdk.Models.Server to AzureSqlDatabaseServerModel /// - /// The resource group the server is in /// The management client server response to convert /// The converted server model private static AzureSqlServerModel CreateServerModelFromResponse(Management.Sql.Models.Server resp) @@ -226,8 +227,7 @@ internal static string Decrypt(SecureString secureString) /// /// Verifies that the Azure Active Directory user or group exists, and will get the object id if it is not set. /// - /// Azure Active Directory user or group display name - /// Azure Active Directory user or group object id + /// Azure Active Directory user or group object /// protected ServerExternalAdministrator GetActiveDirectoryInformation(ServerExternalAdministrator input) { diff --git a/src/Sql/Sql/Server/Services/AzureSqlServerCommunicator.cs b/src/Sql/Sql/Server/Services/AzureSqlServerCommunicator.cs index 2768cc9279ad..22783ba64801 100644 --- a/src/Sql/Sql/Server/Services/AzureSqlServerCommunicator.cs +++ b/src/Sql/Sql/Server/Services/AzureSqlServerCommunicator.cs @@ -43,8 +43,7 @@ public class AzureSqlServerCommunicator /// /// Creates a communicator for Azure Sql Databases /// - /// - /// + /// The current azure context public AzureSqlServerCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/ServerActiveDirectoryAdministrator/Services/AzureSqlServerActiveDirectoryAdministratorAdapter.cs b/src/Sql/Sql/ServerActiveDirectoryAdministrator/Services/AzureSqlServerActiveDirectoryAdministratorAdapter.cs index b3c2b83ec7e3..7981e301dbf6 100644 --- a/src/Sql/Sql/ServerActiveDirectoryAdministrator/Services/AzureSqlServerActiveDirectoryAdministratorAdapter.cs +++ b/src/Sql/Sql/ServerActiveDirectoryAdministrator/Services/AzureSqlServerActiveDirectoryAdministratorAdapter.cs @@ -67,8 +67,7 @@ public MicrosoftGraphClient MicrosoftGraphClient /// /// Constructs a Azure SQL Server Active Directory administrator adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlServerActiveDirectoryAdministratorAdapter(IAzureContext context) { Context = context; @@ -130,7 +129,7 @@ public void RemoveServerActiveDirectoryAdministrator(string resourceGroupName, s /// /// Converts the response from the service to a powershell database object /// - /// The resource group the server is in + /// The resource group the server is in /// The name of the Azure Sql ServerActiveDirectoryAdministrator Server /// The service response /// The converted model diff --git a/src/Sql/Sql/ServerActiveDirectoryAdministrator/Services/AzureSqlServerActiveDirectoryAdministratorCommunicator.cs b/src/Sql/Sql/ServerActiveDirectoryAdministrator/Services/AzureSqlServerActiveDirectoryAdministratorCommunicator.cs index 6a14dff5b7fc..512f5ccbac1c 100644 --- a/src/Sql/Sql/ServerActiveDirectoryAdministrator/Services/AzureSqlServerActiveDirectoryAdministratorCommunicator.cs +++ b/src/Sql/Sql/ServerActiveDirectoryAdministrator/Services/AzureSqlServerActiveDirectoryAdministratorCommunicator.cs @@ -53,8 +53,7 @@ public class AzureSqlServerActiveDirectoryAdministratorCommunicator /// /// Creates a communicator for Azure SQL Server Active Directory administrator /// - /// - /// + /// The current azure context public AzureSqlServerActiveDirectoryAdministratorCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/ServerActiveDirectoryOnlyAuthentication/Services/AzureSqlServerActiveDirectoryOnlyAuthenticationAdapter.cs b/src/Sql/Sql/ServerActiveDirectoryOnlyAuthentication/Services/AzureSqlServerActiveDirectoryOnlyAuthenticationAdapter.cs index e02f410cdd26..7bc0ffaa3741 100644 --- a/src/Sql/Sql/ServerActiveDirectoryOnlyAuthentication/Services/AzureSqlServerActiveDirectoryOnlyAuthenticationAdapter.cs +++ b/src/Sql/Sql/ServerActiveDirectoryOnlyAuthentication/Services/AzureSqlServerActiveDirectoryOnlyAuthenticationAdapter.cs @@ -39,8 +39,7 @@ public class AzureSqlServerActiveDirectoryOnlyAuthenticationAdapter /// /// Constructs a Azure SQL Server Active Directory administrator adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlServerActiveDirectoryOnlyAuthenticationAdapter(IAzureContext context) { Context = context; @@ -80,6 +79,7 @@ internal ICollection ListS /// /// The name of the resource group /// The name of the Azure Sql ServerActiveDirectoryAdministrator Server + /// /// The upserted Azure SQL Server Active Directory administrator internal AzureSqlServerActiveDirectoryOnlyAuthenticationModel UpsertAzureADOnlyAuthenticaion(string resourceGroup, string serverName, AzureSqlServerActiveDirectoryOnlyAuthenticationModel model) { @@ -91,9 +91,9 @@ internal AzureSqlServerActiveDirectoryOnlyAuthenticationModel UpsertAzureADOnlyA /// /// Converts the response from the service to a powershell database object /// - /// The resource group the server is in + /// The resource group the server is in /// The name of the Azure Sql ServerActiveDirectoryAdministrator Server - /// The service response + /// /// The converted model public static AzureSqlServerActiveDirectoryOnlyAuthenticationModel CreateServerActiveDirectoryOnlyAuthenticationModelFromResponse(string resourceGroup, string serverName, Management.Sql.Models.ServerAzureADOnlyAuthentication serverAzureADOnlyAuthentication) { diff --git a/src/Sql/Sql/ServerActiveDirectoryOnlyAuthentication/Services/AzureSqlServerActiveDirectoryOnlyAuthenticationCommunicator.cs b/src/Sql/Sql/ServerActiveDirectoryOnlyAuthentication/Services/AzureSqlServerActiveDirectoryOnlyAuthenticationCommunicator.cs index 5466700cc8af..de31eb82d361 100644 --- a/src/Sql/Sql/ServerActiveDirectoryOnlyAuthentication/Services/AzureSqlServerActiveDirectoryOnlyAuthenticationCommunicator.cs +++ b/src/Sql/Sql/ServerActiveDirectoryOnlyAuthentication/Services/AzureSqlServerActiveDirectoryOnlyAuthenticationCommunicator.cs @@ -53,8 +53,7 @@ public class AzureSqlServerActiveDirectoryOnlyAuthenticationCommunicator /// /// Creates a communicator for Azure SQL Server Active Directory administrator /// - /// - /// + /// The current azure context public AzureSqlServerActiveDirectoryOnlyAuthenticationCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/ServerCommunicationLink/Services/AzureSqlServerCommunicationLinkAdapter.cs b/src/Sql/Sql/ServerCommunicationLink/Services/AzureSqlServerCommunicationLinkAdapter.cs index a54ecabdb039..b5200d50a4d0 100644 --- a/src/Sql/Sql/ServerCommunicationLink/Services/AzureSqlServerCommunicationLinkAdapter.cs +++ b/src/Sql/Sql/ServerCommunicationLink/Services/AzureSqlServerCommunicationLinkAdapter.cs @@ -41,8 +41,7 @@ public class AzureSqlServerCommunicationLinkAdapter /// /// Constructs an adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlServerCommunicationLinkAdapter(IAzureContext context) { Context = context; @@ -81,8 +80,6 @@ internal ICollection ListServerCommunicati /// /// Creates or updates a Azure Sql Server communication link. /// - /// The name of the resource group - /// The name of the Azure Sql Database Server /// The input parameters for the create/update operation /// The upserted Azure Sql server communication link internal AzureSqlServerCommunicationLinkModel UpsertServerCommunicationLink(AzureSqlServerCommunicationLinkModel model) @@ -126,7 +123,7 @@ public string GetServerLocation(string resourceGroupName, string serverName) /// /// Converts the response from the service to a powershell database object /// - /// The resource group the server is in + /// The resource group the server is in /// The name of the Azure Sql Database Server /// The service response /// The converted model diff --git a/src/Sql/Sql/ServerCommunicationLink/Services/AzureSqlServerCommunicationLinkCommunicator.cs b/src/Sql/Sql/ServerCommunicationLink/Services/AzureSqlServerCommunicationLinkCommunicator.cs index abd601658525..73ad1aa58eb0 100644 --- a/src/Sql/Sql/ServerCommunicationLink/Services/AzureSqlServerCommunicationLinkCommunicator.cs +++ b/src/Sql/Sql/ServerCommunicationLink/Services/AzureSqlServerCommunicationLinkCommunicator.cs @@ -43,8 +43,7 @@ public class AzureSqlServerCommunicationLinkCommunicator /// /// Creates a communicator for Azure Sql Elastic Pool /// - /// - /// + /// The current azure context public AzureSqlServerCommunicationLinkCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/ServerDisasterRecoveryConfiguration/Services/AzureSqlServerDisasterRecoveryConfigurationAdapter.cs b/src/Sql/Sql/ServerDisasterRecoveryConfiguration/Services/AzureSqlServerDisasterRecoveryConfigurationAdapter.cs index 10039c31df54..068084c4a20e 100644 --- a/src/Sql/Sql/ServerDisasterRecoveryConfiguration/Services/AzureSqlServerDisasterRecoveryConfigurationAdapter.cs +++ b/src/Sql/Sql/ServerDisasterRecoveryConfiguration/Services/AzureSqlServerDisasterRecoveryConfigurationAdapter.cs @@ -42,8 +42,7 @@ public class AzureSqlServerDisasterRecoveryConfigurationAdapter /// /// Constructs a Server Disaster Recovery Configuration adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlServerDisasterRecoveryConfigurationAdapter(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/ServerDnsAlias/Services/AzureSqlServerDnsAliasCommunicator.cs b/src/Sql/Sql/ServerDnsAlias/Services/AzureSqlServerDnsAliasCommunicator.cs index 68459863bba4..55a6e0d3bc05 100644 --- a/src/Sql/Sql/ServerDnsAlias/Services/AzureSqlServerDnsAliasCommunicator.cs +++ b/src/Sql/Sql/ServerDnsAlias/Services/AzureSqlServerDnsAliasCommunicator.cs @@ -26,95 +26,94 @@ namespace Microsoft.Azure.Commands.Sql.ServerDnsAlias.Services { - /// - /// This class is responsible for all the REST communication with the audit REST endpoints - /// - public class AzureSqlServerDnsAliasCommunicator - { - /// - /// The Sql client to be used by this end points communicator - /// - private static SqlManagementClient SqlClient { get; set; } + /// + /// This class is responsible for all the REST communication with the audit REST endpoints + /// + public class AzureSqlServerDnsAliasCommunicator + { + /// + /// The Sql client to be used by this end points communicator + /// + private static SqlManagementClient SqlClient { get; set; } - /// - /// Gets or set the Azure subscription - /// - private static IAzureSubscription Subscription { get; set; } + /// + /// Gets or set the Azure subscription + /// + private static IAzureSubscription Subscription { get; set; } - /// - /// Gets or sets the Azure profile - /// - public IAzureContext Context { get; set; } + /// + /// Gets or sets the Azure profile + /// + public IAzureContext Context { get; set; } /// /// Creates a communicator for Azure Sql Server DNS Aliases /// - /// - /// + /// The current azure context public AzureSqlServerDnsAliasCommunicator(IAzureContext context) - { - Context = context; - if (context?.Subscription != Subscription) - { - Subscription = context?.Subscription; - SqlClient = null; - } - } + { + Context = context; + if (context?.Subscription != Subscription) + { + Subscription = context?.Subscription; + SqlClient = null; + } + } - /// - /// Get the Azure Sql Server Dns Alias - /// - public Management.Sql.Models.ServerDnsAlias Get(string resourceGroupName, string serverName, string serverDnsAliasName) - { - return GetCurrentSqlClient().ServerDnsAliases.Get(resourceGroupName, serverName, serverDnsAliasName); - } + /// + /// Get the Azure Sql Server Dns Alias + /// + public Management.Sql.Models.ServerDnsAlias Get(string resourceGroupName, string serverName, string serverDnsAliasName) + { + return GetCurrentSqlClient().ServerDnsAliases.Get(resourceGroupName, serverName, serverDnsAliasName); + } - /// - /// Lists the Azure Sql Server Dns Aliases - /// - public IList List(string resourceGroupName, string serverName) - { - return GetCurrentSqlClient().ServerDnsAliases.ListByServer(resourceGroupName, serverName).ToList(); - } + /// + /// Lists the Azure Sql Server Dns Aliases + /// + public IList List(string resourceGroupName, string serverName) + { + return GetCurrentSqlClient().ServerDnsAliases.ListByServer(resourceGroupName, serverName).ToList(); + } - /// - /// Creates or updates an Azure Sql Server Dns Alias - /// - public Management.Sql.Models.ServerDnsAlias CreateOrUpdate(string resourceGroupName, string serverName, string serverDnsAliasName) - { - return GetCurrentSqlClient().ServerDnsAliases.CreateOrUpdate(resourceGroupName, serverName, serverDnsAliasName); - } + /// + /// Creates or updates an Azure Sql Server Dns Alias + /// + public Management.Sql.Models.ServerDnsAlias CreateOrUpdate(string resourceGroupName, string serverName, string serverDnsAliasName) + { + return GetCurrentSqlClient().ServerDnsAliases.CreateOrUpdate(resourceGroupName, serverName, serverDnsAliasName); + } - /// - /// Deletes an Azure Sql Server Dns Alias - /// - public void Remove(string resourceGroupName, string serverName, string serverDnsAliasName) - { - GetCurrentSqlClient().ServerDnsAliases.Delete(resourceGroupName, serverName, serverDnsAliasName); - } + /// + /// Deletes an Azure Sql Server Dns Alias + /// + public void Remove(string resourceGroupName, string serverName, string serverDnsAliasName) + { + GetCurrentSqlClient().ServerDnsAliases.Delete(resourceGroupName, serverName, serverDnsAliasName); + } - /// - /// Acquires an Azure Sql Server Dns Alias from one server and assign it to another - /// - public void Acquire(string resourceGroupName, string serverName, string serverDnsAliasName, ServerDnsAliasAcquisition parameters) - { - GetCurrentSqlClient().ServerDnsAliases.Acquire(resourceGroupName, serverName, serverDnsAliasName, parameters); - } + /// + /// Acquires an Azure Sql Server Dns Alias from one server and assign it to another + /// + public void Acquire(string resourceGroupName, string serverName, string serverDnsAliasName, ServerDnsAliasAcquisition parameters) + { + GetCurrentSqlClient().ServerDnsAliases.Acquire(resourceGroupName, serverName, serverDnsAliasName, parameters); + } - /// - /// Retrieve the SQL Management client for the currently selected subscription, adding the session and request - /// id tracing headers for the current cmdlet invocation. - /// - /// The SQL Management client for the currently selected subscription. - private SqlManagementClient GetCurrentSqlClient() - { - // Get the SQL management client for the current subscription - // - if (SqlClient == null) - { - SqlClient = AzureSession.Instance.ClientFactory.CreateArmClient(Context, AzureEnvironment.Endpoint.ResourceManager); - } - return SqlClient; - } - } + /// + /// Retrieve the SQL Management client for the currently selected subscription, adding the session and request + /// id tracing headers for the current cmdlet invocation. + /// + /// The SQL Management client for the currently selected subscription. + private SqlManagementClient GetCurrentSqlClient() + { + // Get the SQL management client for the current subscription + // + if (SqlClient == null) + { + SqlClient = AzureSession.Instance.ClientFactory.CreateArmClient(Context, AzureEnvironment.Endpoint.ResourceManager); + } + return SqlClient; + } + } } diff --git a/src/Sql/Sql/ServerKeyVaultKey/Services/AzureSqlServerKeyVaultKeyAdapter.cs b/src/Sql/Sql/ServerKeyVaultKey/Services/AzureSqlServerKeyVaultKeyAdapter.cs index 34e3e423e3ef..4d1bac0f121d 100644 --- a/src/Sql/Sql/ServerKeyVaultKey/Services/AzureSqlServerKeyVaultKeyAdapter.cs +++ b/src/Sql/Sql/ServerKeyVaultKey/Services/AzureSqlServerKeyVaultKeyAdapter.cs @@ -114,6 +114,7 @@ public void Delete(string resourceGroupName, string serverName, string keyId) /// /// The resource group the server is in /// The name of the server + /// Server Key Vault Key name /// The management client server key response to convert /// The converted server key vault key model private static AzureSqlServerKeyVaultKeyModel CreateServerKeyModelFromResponse(string resourceGroup, string serverName, string keyName, Microsoft.Azure.Management.Sql.LegacySdk.Models.ServerKey resp) diff --git a/src/Sql/Sql/ServerTrustCertificate/Services/AzureSqlInstanceServerTrustCertificateAdapter.cs b/src/Sql/Sql/ServerTrustCertificate/Services/AzureSqlInstanceServerTrustCertificateAdapter.cs index e7dd9385ec1e..0e9802211bdb 100644 --- a/src/Sql/Sql/ServerTrustCertificate/Services/AzureSqlInstanceServerTrustCertificateAdapter.cs +++ b/src/Sql/Sql/ServerTrustCertificate/Services/AzureSqlInstanceServerTrustCertificateAdapter.cs @@ -99,6 +99,8 @@ public void RemoveServerTrustCertificate(string resourceGroupName, string instan /// /// Convert a Management.Sql.Models.ServerTrustCertificate to AzureSqlInstanceServerTrustCertificateModel /// + /// The resource group the managed instance is in + /// The name of the managed instance /// The management client server trust certificate response to convert /// The converted server trust certificate model private static AzureSqlInstanceServerTrustCertificateModel CreateServerTrustCertificateModelFromResponse(string resourceGroupName, string instanceName, Management.Sql.Models.ServerTrustCertificate serverTrustCertificate) diff --git a/src/Sql/Sql/ServerTrustCertificate/Services/AzureSqlInstanceServerTrustCertificateCommunicator.cs b/src/Sql/Sql/ServerTrustCertificate/Services/AzureSqlInstanceServerTrustCertificateCommunicator.cs index 23b3dc93f72d..4abfdcd26ac1 100644 --- a/src/Sql/Sql/ServerTrustCertificate/Services/AzureSqlInstanceServerTrustCertificateCommunicator.cs +++ b/src/Sql/Sql/ServerTrustCertificate/Services/AzureSqlInstanceServerTrustCertificateCommunicator.cs @@ -43,8 +43,7 @@ public class AzureSqlInstanceServerTrustCertificateCommunicator /// /// Creates a communicator for Azure Sql Instance Server Trust Certificate /// - /// - /// + /// The current azure context public AzureSqlInstanceServerTrustCertificateCommunicator(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/ServiceObjective/Service/AzureSqlServerServiceObjectiveAdapter.cs b/src/Sql/Sql/ServiceObjective/Service/AzureSqlServerServiceObjectiveAdapter.cs index 3b4cb5cb9fd6..9a7ccbf3bec5 100644 --- a/src/Sql/Sql/ServiceObjective/Service/AzureSqlServerServiceObjectiveAdapter.cs +++ b/src/Sql/Sql/ServiceObjective/Service/AzureSqlServerServiceObjectiveAdapter.cs @@ -47,8 +47,7 @@ public class AzureSqlServerServiceObjectiveAdapter /// /// Constructs a ServiceObjective adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlServerServiceObjectiveAdapter(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/ServiceTierAdvisor/Services/AzureSqlServiceTierAdvisorAdapter.cs b/src/Sql/Sql/ServiceTierAdvisor/Services/AzureSqlServiceTierAdvisorAdapter.cs index 2d62726d3c42..b01dbac2fae9 100644 --- a/src/Sql/Sql/ServiceTierAdvisor/Services/AzureSqlServiceTierAdvisorAdapter.cs +++ b/src/Sql/Sql/ServiceTierAdvisor/Services/AzureSqlServiceTierAdvisorAdapter.cs @@ -44,8 +44,7 @@ public class AzureSqlServiceTierAdvisorAdapter /// /// Constructs a service tier advisor adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlServiceTierAdvisorAdapter(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/ServiceTierAdvisor/Services/AzureSqlServiceTierAdvisorCommunicator.cs b/src/Sql/Sql/ServiceTierAdvisor/Services/AzureSqlServiceTierAdvisorCommunicator.cs index bf203bb691f5..21ec17464f25 100644 --- a/src/Sql/Sql/ServiceTierAdvisor/Services/AzureSqlServiceTierAdvisorCommunicator.cs +++ b/src/Sql/Sql/ServiceTierAdvisor/Services/AzureSqlServiceTierAdvisorCommunicator.cs @@ -44,8 +44,7 @@ public class AzureSqlServiceTierAdvisorCommunicator /// /// Creates a communicator for Azure Sql Service Tier Advisor /// - /// - /// + /// The current azure context public AzureSqlServiceTierAdvisorCommunicator(IAzureContext context) { Context = context; @@ -86,6 +85,7 @@ public Management.Sql.LegacySdk.Models.Database GetDatabaseExpanded(string resou /// /// The name of the resource group /// The name of Azure Sql server + /// Expand string /// List of recommended elastic pools public IList GetRecommendedElasticPoolsExpanded(string resourceGroupName, string serverName, string expand) { diff --git a/src/Sql/Sql/TransparentDataEncryption/Cmdlet/AddAzureRmSqlManagedInstanceKeyVaultKey.cs b/src/Sql/Sql/TransparentDataEncryption/Cmdlet/AddAzureRmSqlManagedInstanceKeyVaultKey.cs index 013486bf956b..a05cf270ba36 100644 --- a/src/Sql/Sql/TransparentDataEncryption/Cmdlet/AddAzureRmSqlManagedInstanceKeyVaultKey.cs +++ b/src/Sql/Sql/TransparentDataEncryption/Cmdlet/AddAzureRmSqlManagedInstanceKeyVaultKey.cs @@ -39,7 +39,7 @@ public class AddAzureRmSqlManagedInstanceKeyVaultKey : AzureRmSqlManagedInstance /// /// Constructs the model to send to the update API /// - /// Model to send to the update API + /// Model to send to the update API /// The model to send to the update protected override IEnumerable ApplyUserInputToModel(IEnumerable model) { diff --git a/src/Sql/Sql/TransparentDataEncryption/Cmdlet/AzureRmSqlManagedInstanceKeyVaultKeyBase.cs b/src/Sql/Sql/TransparentDataEncryption/Cmdlet/AzureRmSqlManagedInstanceKeyVaultKeyBase.cs index 09e070addb9a..4d41e0c7e4ce 100644 --- a/src/Sql/Sql/TransparentDataEncryption/Cmdlet/AzureRmSqlManagedInstanceKeyVaultKeyBase.cs +++ b/src/Sql/Sql/TransparentDataEncryption/Cmdlet/AzureRmSqlManagedInstanceKeyVaultKeyBase.cs @@ -136,7 +136,6 @@ public override void ExecuteCmdlet() /// /// Intializes the model adapter /// - /// The subscription the cmdlets are operation under /// The server adapter protected override AzureSqlDatabaseTransparentDataEncryptionArmAdapter InitModelAdapter() { diff --git a/src/Sql/Sql/TransparentDataEncryption/Cmdlet/AzureSqlRmManagedInstanceTransparentDataEncryptionProtectorBase.cs b/src/Sql/Sql/TransparentDataEncryption/Cmdlet/AzureSqlRmManagedInstanceTransparentDataEncryptionProtectorBase.cs index 724c4b21c14a..b07c77bdd355 100644 --- a/src/Sql/Sql/TransparentDataEncryption/Cmdlet/AzureSqlRmManagedInstanceTransparentDataEncryptionProtectorBase.cs +++ b/src/Sql/Sql/TransparentDataEncryption/Cmdlet/AzureSqlRmManagedInstanceTransparentDataEncryptionProtectorBase.cs @@ -108,7 +108,6 @@ public override void ExecuteCmdlet() /// /// Intializes the model adapter /// - /// The subscription the cmdlets are operation under /// The server adapter protected override AzureSqlDatabaseTransparentDataEncryptionArmAdapter InitModelAdapter() { diff --git a/src/Sql/Sql/TransparentDataEncryption/Services/AzureSqlDatabaseTransparentDataEncryptionAdapter.cs b/src/Sql/Sql/TransparentDataEncryption/Services/AzureSqlDatabaseTransparentDataEncryptionAdapter.cs index 22ef129cc26a..123f5fcebaf2 100644 --- a/src/Sql/Sql/TransparentDataEncryption/Services/AzureSqlDatabaseTransparentDataEncryptionAdapter.cs +++ b/src/Sql/Sql/TransparentDataEncryption/Services/AzureSqlDatabaseTransparentDataEncryptionAdapter.cs @@ -44,8 +44,7 @@ public class AzureSqlDatabaseTransparentDataEncryptionAdapter /// /// Constructs a Transparent Data Encryption adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlDatabaseTransparentDataEncryptionAdapter(IAzureContext context) { Context = context; @@ -68,8 +67,6 @@ public AzureSqlDatabaseTransparentDataEncryptionModel GetTransparentDataEncrypti /// /// Upserts a Transparent Data Encryption /// - /// The name of the resource group - /// The name of ther server /// The Transparent Data Encryption to create /// The updated server model public AzureSqlDatabaseTransparentDataEncryptionModel UpsertTransparentDataEncryption(AzureSqlDatabaseTransparentDataEncryptionModel model) @@ -123,6 +120,7 @@ public AzureSqlServerTransparentDataEncryptionProtectorModel CreateOrUpdateEncry /// /// The resource group the server is in /// The name of the server + /// The name of the database /// The management client server response to convert /// The converted server model private static AzureSqlDatabaseTransparentDataEncryptionModel CreateTransparentDataEncryptionModelFromResponse(string resourceGroup, string serverName, string databaseName, Management.Sql.Models.LogicalDatabaseTransparentDataEncryption resp) diff --git a/src/Sql/Sql/TransparentDataEncryption/Services/AzureSqlDatabaseTransparentDataEncryptionArmAdapter.cs b/src/Sql/Sql/TransparentDataEncryption/Services/AzureSqlDatabaseTransparentDataEncryptionArmAdapter.cs index 81edb6e90fcf..cbd4fdb4626b 100644 --- a/src/Sql/Sql/TransparentDataEncryption/Services/AzureSqlDatabaseTransparentDataEncryptionArmAdapter.cs +++ b/src/Sql/Sql/TransparentDataEncryption/Services/AzureSqlDatabaseTransparentDataEncryptionArmAdapter.cs @@ -48,8 +48,7 @@ public class AzureSqlDatabaseTransparentDataEncryptionArmAdapter /// /// Constructs a Transparent Data Encryption adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlDatabaseTransparentDataEncryptionArmAdapter(IAzureContext context) { Context = context; diff --git a/src/Sql/Sql/VirtualNetworkRule/Services/AzureSqlServerVirtualNetworkRuleAdapter.cs b/src/Sql/Sql/VirtualNetworkRule/Services/AzureSqlServerVirtualNetworkRuleAdapter.cs index 4ffe573f9258..1bf2edcb24aa 100644 --- a/src/Sql/Sql/VirtualNetworkRule/Services/AzureSqlServerVirtualNetworkRuleAdapter.cs +++ b/src/Sql/Sql/VirtualNetworkRule/Services/AzureSqlServerVirtualNetworkRuleAdapter.cs @@ -38,8 +38,7 @@ public class AzureSqlServerVirtualNetworkRuleAdapter /// /// Constructs a virtual network rule adapter /// - /// The current azure profile - /// The current azure subscription + /// The current azure context public AzureSqlServerVirtualNetworkRuleAdapter(IAzureContext context) { Context = context; @@ -77,8 +76,6 @@ public List ListVirtualNetworkRules(strin /// /// Upserts a virtual network rule /// - /// The name of the resource group - /// The name of ther server /// The virtual network rule to create /// The updated virtual network rule model public AzureSqlServerVirtualNetworkRuleModel UpsertVirtualNetworkRule(AzureSqlServerVirtualNetworkRuleModel model) diff --git a/src/Sql/Sql/VirtualNetworkRule/Services/AzureSqlServerVirtualNetworkRuleCommunicator.cs b/src/Sql/Sql/VirtualNetworkRule/Services/AzureSqlServerVirtualNetworkRuleCommunicator.cs index 2ec0ff72e16d..8545d8b35604 100644 --- a/src/Sql/Sql/VirtualNetworkRule/Services/AzureSqlServerVirtualNetworkRuleCommunicator.cs +++ b/src/Sql/Sql/VirtualNetworkRule/Services/AzureSqlServerVirtualNetworkRuleCommunicator.cs @@ -43,8 +43,7 @@ public class AzureSqlServerVirtualNetworkRuleCommunicator /// /// Creates a communicator for Azure Sql Server Virtual Network Rules /// - /// - /// + /// The current azure context public AzureSqlServerVirtualNetworkRuleCommunicator(IAzureContext context) { Context = context; @@ -71,27 +70,27 @@ public Management.Sql.Models.VirtualNetworkRule Get(string resourceGroupName, st return GetCurrentSqlClient().VirtualNetworkRules.ListByServer(resourceGroupName, serverName).ToList(); } - ///// - ///// Creates or updates an Azure Sql Database Server FirewallRule - ///// + /// + /// Creates or updates an Azure Sql Database Server FirewallRule + /// public Management.Sql.Models.VirtualNetworkRule CreateOrUpdate(string resourceGroupName, string serverName, string vnetFirewallRuleName, Management.Sql.Models.VirtualNetworkRule parameters) { return GetCurrentSqlClient().VirtualNetworkRules.CreateOrUpdate(resourceGroupName, serverName, vnetFirewallRuleName, parameters); } - ///// - ///// Deletes an Azure Sql Database Server FirewallRule - ///// + /// + /// Deletes an Azure Sql Database Server FirewallRule + /// public void Remove(string resourceGroupName, string serverName, string vnetFirewallRuleName) { GetCurrentSqlClient().VirtualNetworkRules.Delete(resourceGroupName, serverName, vnetFirewallRuleName); } - ///// - ///// Retrieve the SQL Management client for the currently selected subscription, adding the session and request - ///// id tracing headers for the current cmdlet invocation. - ///// - ///// The SQL Management client for the currently selected subscription. + /// + /// Retrieve the SQL Management client for the currently selected subscription, adding the session and request + /// id tracing headers for the current cmdlet invocation. + /// + /// The SQL Management client for the currently selected subscription. private SqlManagementClient GetCurrentSqlClient() { // Get the SQL management client for the current subscription