Skip to content

Commit 7b20194

Browse files
SharanSinghlobashh
andauthored
[Az.Sql] Add support for cross-subscription Azure SQL Database Failover Group creation (#19027)
* Added PartnerSubscriptionId to Azure Sql Failover Group models * Converted PartnerSubscriptionId to string * Added Scenario Test for testing PartnerSubscriptionId parameter * Ran Update-Markdownhelp command and added new example in New-AzSqlDatabaseFailoverGroup.md * Added new feature to ChangeLog.md * Reverted unintended chagnes in src/Synapse codepath * Fixed the test command name and added output to markdown help file * Added ValidateNotNullOrEmpty tag to PartnerSubscriptionId * Removed extra line from ChangeLog.md Co-authored-by: Sharan Singh <[email protected]>
1 parent 65e4448 commit 7b20194

File tree

8 files changed

+6073
-4
lines changed

8 files changed

+6073
-4
lines changed

src/Sql/Sql.Test/ScenarioTests/FailoverGroupTests.cs

Lines changed: 7 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -82,6 +82,13 @@ public void TestCreateFailoverGroup_Overflow()
8282
TestRunner.RunTestScript("Test-CreateFailoverGroup-Overflow");
8383
}
8484

85+
[Fact]
86+
[Trait(Category.AcceptanceType, Category.CheckIn)]
87+
public void TestCreateFailoverGroup_CrossSubscription()
88+
{
89+
TestRunner.RunTestScript("Test-CreateFailoverGroup-CrossSubscription");
90+
}
91+
8592
[Fact]
8693
[Trait(Category.AcceptanceType, Category.CheckIn)]
8794
public void TestSetFailoverGroup_Named()

src/Sql/Sql.Test/ScenarioTests/FailoverGroupTests.ps1

Lines changed: 14 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -50,6 +50,7 @@ function Validate-FailoverGroup($server, $partnerServer, $name, $role, $failover
5050
Assert-NotNull $fg.Id "`$fg.Id ($message)"
5151
Assert-NotNull $fg.PartnerServers "`$fg.PartnerServers ($message)"
5252
Assert-AreEqual $name $fg.FailoverGroupName "`$fg.FailoverGroupName ($message)"
53+
# Assert-AreEqual $true $partnerServer.ResourceId.Contains($fg.PartnerSubscriptionId) "`$fg.PartnerSubscriptionId ($message)"
5354
Assert-AreEqual $server.ResourceGroupName $fg.ResourceGroupName "`$fg.ResourceGroupName ($message)"
5455
Assert-AreEqual $partnerServer.ResourceGroupName $fg.PartnerResourceGroupName "`$fg.PartnerResourceGroupName ($message)"
5556
Assert-AreEqual $server.ServerName $fg.ServerName "`$fg.ServerName ($message)"
@@ -236,6 +237,19 @@ function Test-CreateFailoverGroup-Overflow()
236237
}
237238
}
238239

240+
function Test-CreateFailoverGroup-CrossSubscription()
241+
{
242+
Handle-FailoverGroupTest {
243+
Param($server, $partnerServer)
244+
245+
$fgName = Get-FailoverGroupName
246+
$partnerSubscriptionId = $partnerServer.ResourceId.Split('/')[2]
247+
$fg = $server | New-AzSqlDatabaseFailoverGroup -FailoverGroupName $fgName -PartnerSubscriptionId $partnerSubscriptionId -PartnerResourceGroupName $partnerServer.ResourceGroupName -PartnerServerName $partnerServer.ServerName -FailoverPolicy Manual
248+
Validate-FailoverGroup $server $partnerServer $fgName Primary Manual $null Disabled @() $fg
249+
Validate-FailoverGroupWithGet $fg
250+
}
251+
}
252+
239253
function Test-SetFailoverGroup-Named()
240254
{
241255
Handle-FailoverGroupTestWithFailoverGroup {

0 commit comments

Comments
 (0)