Skip to content

Commit a4d9d86

Browse files
authored
Add extra conditional for failover groups going from multi to single secondary (Azure#23613)
* add conditional to persist changes to account for multi -> single scenario * add changelog update
1 parent 75bf2ea commit a4d9d86

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

src/Sql/Sql/ChangeLog.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -21,6 +21,7 @@
2121

2222
## Version 4.12.0
2323
* Added new parameters `MaintenanceConfigurationId`, `DnsZone` to `AzSqlInstancePool` cmdlets
24+
* Fixed `Set-AzSqlDatabaseFailoverGroup` when going from multi-secondary to single secondary
2425

2526
## Version 4.11.0
2627
* Added new parameters to `New-AzSqlDatabaseFailoverGroup`, `Set-AzSqlDatabaseFailoverGroup`

src/Sql/Sql/Failover Group/Cmdlet/SetAzureSqlFailoverGroup.cs

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -172,7 +172,7 @@ protected override IEnumerable<AzureSqlFailoverGroupModel> ApplyUserInputToModel
172172
protected override IEnumerable<AzureSqlFailoverGroupModel> PersistChanges(IEnumerable<AzureSqlFailoverGroupModel> entity)
173173
{
174174
AzureSqlFailoverGroupModel model = entity.First();
175-
bool useV2 = (model.PartnerServers != null && model.PartnerServers.Count > 1);
175+
bool useV2 = (model.PartnerServers != null && model.PartnerServers.Count > 1) || (MyInvocation.BoundParameters.ContainsKey("PartnerServerList"));
176176
return new List<AzureSqlFailoverGroupModel>() {
177177
ModelAdapter.PatchUpdateFailoverGroup(entity.First(), useV2)
178178
};

0 commit comments

Comments
 (0)