[dsccommunity#568] We don't need ComputerName when not using $SQLAdmi…

…nCredential
bozho · May 26, 2017 · a327850 · a327850
1 parent bccf9a0
commit a327850
Showing 1 changed file with 2 additions and 2 deletions.
diff --git a/DSCResources/MSFT_xSQLServerRSConfig/MSFT_xSQLServerRSConfig.psm1 b/DSCResources/MSFT_xSQLServerRSConfig/MSFT_xSQLServerRSConfig.psm1
@@ -33,10 +33,10 @@ function Get-TargetResource
         $SQLVersion = ((Get-ItemProperty -Path "HKLM:\SOFTWARE\Microsoft\Microsoft SQL Server\$InstanceKey\Setup" -Name "Version").Version).Split(".")[0]
 
         $invokeParameters = @{
-            ComputerName = "localhost"
             ArgumentList = @($SQLVersion,$InstanceName)
         }
         if($SQLAdminCredential -ne $null) { 
+            $invokeParameters.Add("ComputerName", "localhost")
             $invokeParameters.Add("Credential", $SQLAdminCredential)
             $invokeParameters.Add("Authentication", "CredSSP")
         }
@@ -101,10 +101,10 @@ function Set-TargetResource
     {
 
         $invokeParameters = @{
-            ComputerName = "localhost"
             ArgumentList = @($InstanceName,$RSSQLServer,$RSSQLInstanceName)
         }
         if($SQLAdminCredential -ne $null) { 
+            $invokeParameters.Add("ComputerName", "localhost")
             $invokeParameters.Add("Credential", $SQLAdminCredential)
             $invokeParameters.Add("Authentication", "CredSSP")
         }