Skip to content

Commit f1ff493

Browse files
[**URGENT**] Fix for Set-AzSqlServer and Set-AzSqlInstance cmdlets (#15176)
* Fix for Set cmdlets * comma * update changelog.md * Update ChangeLog.md Co-authored-by: Dingmeng Xue <[email protected]>
1 parent 7b38bed commit f1ff493

File tree

3 files changed

+5
-2
lines changed

3 files changed

+5
-2
lines changed

src/Sql/Sql/ChangeLog.md

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -20,10 +20,11 @@
2020
## Upcoming Release
2121
* Added option to support short version of maintenance configuration id for Managed Instance in `New-AzSqlInstance` and `Set-AzSqlInstance` cmdlets
2222
* Added HighAvailabilityReplicaCount to `New-AzSqlDatabaseSecondary`
23-
* Add External Administrator and AAD Only Properties to AzSqlServer and AzSqlInstance
23+
* Added External Administrator and AAD Only Properties to AzSqlServer and AzSqlInstance
2424
- Added option to specify `-ExternalAdminName`, `-ExternalAdminSid`, `-EnableActiveDirectoryOnlyAuthentication` in `New-AzSqlInstance` and `Set-AzSqlInstance` cmdlets
2525
- Added option to expand external administrators information using `-ExpandActiveDirectoryAdministrator` in `Get-AzSqlServer` and `Get-AzSqlInstance` cmdlets
2626
* Fixed `Set-AzSqlDatabase` to no longer default ReadScale to Disabled when not specified
27+
* Fixed `Set-AzSqlServer` and `Set-AzSqlInstance` for partial PUT with only identity and null properties
2728

2829
## Version 3.1.0
2930
* Updated `Set-AzSqlDatabaseVulnerabilityAssessmentRuleBaseline` documentation to include example of define array of array with one inner array.

src/Sql/Sql/ManagedInstance/Cmdlet/SetAzureSqlManagedInstance.cs

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -290,7 +290,8 @@ protected override IEnumerable<AzureSqlManagedInstanceModel> ApplyUserInputToMod
290290
Identity = model.FirstOrDefault().Identity ?? ResourceIdentityHelper.GetIdentityObjectFromType(this.AssignIdentity.IsPresent),
291291
InstancePoolName = this.InstancePoolName,
292292
MinimalTlsVersion = this.MinimalTlsVersion,
293-
MaintenanceConfigurationId = this.MaintenanceConfigurationId
293+
MaintenanceConfigurationId = this.MaintenanceConfigurationId,
294+
AdministratorLogin = model.FirstOrDefault().AdministratorLogin
294295
});
295296
return updateData;
296297
}

src/Sql/Sql/Server/Cmdlet/SetAzureSqlServer.cs

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -123,6 +123,7 @@ public class SetAzureSqlServer : AzureSqlServerCmdletBase
123123
Identity = model.FirstOrDefault().Identity ?? ResourceIdentityHelper.GetIdentityObjectFromType(this.AssignIdentity.IsPresent),
124124
PublicNetworkAccess = this.PublicNetworkAccess,
125125
MinimalTlsVersion = this.MinimalTlsVersion,
126+
SqlAdministratorLogin = model.FirstOrDefault().SqlAdministratorLogin
126127
});
127128
return updateData;
128129
}

0 commit comments

Comments
 (0)