Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

SqlSetup: Missing Configuration Feature/Options : Network Configurations #1161

Closed
mdaniou opened this issue Jun 30, 2018 · 8 comments · Fixed by #1521
Closed

SqlSetup: Missing Configuration Feature/Options : Network Configurations #1161

mdaniou opened this issue Jun 30, 2018 · 8 comments · Fixed by #1521
Labels
enhancement The issue is an enhancement request.

Comments

@mdaniou
Copy link
Contributor

mdaniou commented Jun 30, 2018

Details of the scenario you tried and the problem that is occurring

This issue is created to bring some improvements to the SqlSetup DSC Configuration. The idea is to report all the missing options in the resource to be allow the installation of all the features and to use all the options available in the SQL Server installer.

Options to add to the DSC Resource :

/NPENABLED : Specifies the state of the Named Pipes protocol for the SQL Server service. Supported values:
/TCPENABLED : Specifies the state of the TCP protocol for the SQL Server service. Supported values:

The DSC configuration that is using the resource (as detailed as possible)

SqlSetup

Version of the operating system and PowerShell the target node is running

Windows Server 2012/2012R2/2016

SQL Server edition and version the target node is running

SQL Server 2012/2014/2016/2017

What SQL Server PowerShell modules, and which version, are present on the target node.

Most recent version

Version of the DSC module you're using, or write 'dev' if you're using current dev branch

Most recent version

@johlju johlju added enhancement The issue is an enhancement request. help wanted The issue is up for grabs for anyone in the community. labels Jun 30, 2018
@JoeAlanis
Copy link

I solved the ability to Enable Named Pipes for the Server Protocol using the code snip below. I don't know if there is a proper way to load the assembly, but it seems to work in this scenario. I hope this can benefit someone who looking to include this functionality via DSC.

Script 'EnableNamedPipes'
{
GetScript = {
[System.Reflection.Assembly]::LoadWithPartialName("Microsoft.SqlServer.SqlWmiManagement") | Out-Null
$wmi = New-Object ('Microsoft.SqlServer.Management.Smo.Wmi.ManagedComputer')$env:COMPUTERNAME
$np = $wmi.ServerInstances.ServerProtocols | ?{$.displayName -eq "Named Pipes"}
$result = $np.IsEnabled
return @{ 'Result' = "$result" }
}
TestScript = {
[System.Reflection.Assembly]::LoadWithPartialName("Microsoft.SqlServer.SqlWmiManagement") | Out-Null
$wmi = New-Object ('Microsoft.SqlServer.Management.Smo.Wmi.ManagedComputer')$env:COMPUTERNAME
$np = $wmi.ServerInstances.ServerProtocols | ?{$
.displayName -eq "Named Pipes"}
$result = $np.IsEnabled
return $result;
}
SetScript = {
[System.Reflection.Assembly]::LoadWithPartialName("Microsoft.SqlServer.SqlWmiManagement") | Out-Null
$wmi = New-Object ('Microsoft.SqlServer.Management.Smo.Wmi.ManagedComputer')$env:COMPUTERNAME
$wmi.ServerInstances.ServerProtocols | Select DisplayName, State, IsEnabled
$np = $wmi.ServerInstances.ServerProtocols | ?{$_.displayName -eq "Named Pipes"}
$np.IsEnabled = $True
$np.Alter()
}
DependsOn = '[SqlSetup]InstallDefaultInstance'

    }

@johlju
Copy link
Member

johlju commented Sep 6, 2018

Thanks for the snippet, that would simplify for someone to add this, but I think it should be implemented in SqlServerNetwork (issue #14), since SqlSetup will just add the required arguments to SqlSetup. It will be up to SqlServerNetwork to make sure the settings is kept in desired state.

A suggestion is that it should use Import-Module (either SQLPS or SqlServer) to load the correct assemblies into the session instead of using LoadWithPartialName, which will load the first dll it encounters in the GAC regardless of version of SQL Server. If you only have one instance on the target node, then that is probably not a problem. But if you for example is using SqlServer module together with other resource modules, that Script resource would end up loading the wrong assembly making the SqlServer module not working.

@NReilingh
Copy link
Contributor

Would enabling the setting of the "Force Encryption" flag be in scope for this issue?

@johlju
Copy link
Member

johlju commented Sep 18, 2018

@NReilingh Not sure what force encryption flag you a referencing, but If SQL Server setup.exe can enable it, then yes it would be the scope for the SqlSetup resource, otherwise no - then it should be part of another resource.

@SQLHorizons
Copy link
Contributor

Hi @johlju, Enforced Encryption and Hide Instance are properties in the DB instances ietwork properties and found via SQL Server Configuration Manager, see sample code:-

##  apply surface area configuration control 2.12
$WmiObject = @{
    ComputerName = $SQLServer.NetName
    Namespace    = "root\Microsoft\SqlServer\ComputerManagement$($SQLServer.VersionMajor)"
    Class        = "ServerSettingsGeneralFlag"
    Filter       = "FlagName = 'HideInstance'"
}
$HideInstance = Get-WmiObject @WmiObject

if ($HideInstance.FlagValue -eq $false) {
    Write-Verbose "Setting HideInstance to True."
    $HideInstance.SetValue($true) | Out-Null
    Write-Verbose "HideInstance set to True"
}

##  apply surface area configuration control 8.2
$WmiObject = @{
    ComputerName = $SQLServer.NetName
    Namespace    = "root\Microsoft\SqlServer\ComputerManagement$($SQLServer.VersionMajor)"
    Class        = "ServerSettingsGeneralFlag"
    Filter       = "FlagName = 'ForceEncryption'"
}
$ForceEncryption = Get-WmiObject @WmiObject

if ($ForceEncryption.FlagValue -eq $false) {
    Write-Verbose "Setting ForceEncryption to True."
    $ForceEncryption.SetValue($true) | Out-Null
    Write-Verbose "ForceEncryption set to True"
}

Note $SQLServer is the SMO object

However this would only be half a solution as some client may wish to apply the certificate with this option.

@johlju
Copy link
Member

johlju commented Sep 24, 2018

Thanks for the information @SQLHorizons!

The new resource https://github.com/PowerShell/SqlServerDsc#sqlserversecureconnection introduce ForceEncryption and Thumbprint, although it is using the registry to set this, so that resource could be changed to use the CimInstance instead. The HideInstance is not supported in any resource yet, please submit a new issue suggesting where it could fit, since this is not related to this issue.

@johlju
Copy link
Member

johlju commented May 2, 2020

I solved the ability to Enable Named Pipes for the Server Protocol using the code snip below. I don't know if there is a proper way to load the assembly, but it seems to work in this scenario. I hope this can benefit someone who looking to include this functionality via DSC.

The new resource SqlServerProtocol will solve this (currently in PR). But does not fix this issue to make SqlSetup support this.

@johlju
Copy link
Member

johlju commented May 3, 2020

The properties ForceEncryption and HideInstanceare now tracked in issue #1519 and issue #1520.

@johlju johlju added in progress The issue is being actively worked on by someone. and removed help wanted The issue is up for grabs for anyone in the community. labels May 3, 2020
SqlSetup: Missing Configuration Feature/Options automation moved this from Backlog to Done May 6, 2020
johlju added a commit that referenced this issue May 6, 2020
- SqlSetup
  - Added the properties `NpEnabled` and `TcpEnabled` (issue #1161).
@johlju johlju removed the in progress The issue is being actively worked on by someone. label May 6, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement The issue is an enhancement request.
Development

Successfully merging a pull request may close this issue.

5 participants