Move testing of active node in try block

dsccommunity · Sep 15, 2022 · 80cd226 · 80cd226
1 parent 1190c43
commit 80cd226
Showing 1 changed file with 5 additions and 6 deletions.
diff --git a/source/DSCResources/DSC_SqlAGListener/DSC_SqlAGListener.psm1 b/source/DSCResources/DSC_SqlAGListener/DSC_SqlAGListener.psm1
@@ -24,7 +24,6 @@ $script:localizedData = Get-LocalizedData -DefaultUICulture 'en-US'
 #>
 function Get-TargetResource
 {
-    [System.Diagnostics.CodeAnalysis.SuppressMessageAttribute('SqlServerDsc.AnalyzerRules\Measure-CommandsNeededToLoadSMO', '', Justification='The command Connect-Sql is called when Get-SQLAlwaysOnAvailabilityGroupListener is called')]
     [CmdletBinding()]
     [OutputType([System.Collections.Hashtable])]
     param
@@ -51,13 +50,13 @@ function Get-TargetResource
         $script:localizedData.GetAvailabilityGroupListener -f $Name, $AvailabilityGroup, $InstanceName
     )
 
-    $serverObject = Connect-SQL -ServerName $ServerName -InstanceName $InstanceName
-
-    # Is this node actively hosting the SQL instance?
-    $isActiveNode = Test-ActiveNode -ServerObject $serverObject
-
     try
     {
+        $serverObject = Connect-SQL -ServerName $ServerName -InstanceName $InstanceName
+
+        # Is this node actively hosting the SQL instance?
+        $isActiveNode = Test-ActiveNode -ServerObject $serverObject
+
         $availabilityGroupListener = Get-SQLAlwaysOnAvailabilityGroupListener -Name $Name -AvailabilityGroup $AvailabilityGroup -ServerName $ServerName -InstanceName $InstanceName
 
         if ($null -ne $availabilityGroupListener)