-
Notifications
You must be signed in to change notification settings - Fork 225
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
Install fails when using a Domain Service account that begins with $ #1055
Comments
Just double-checking here.... |
Correct. changed the markdown to code and it is now shown correctly. |
I see this in the verbose log in your issue description. Does it get outputted like this, or have you done some obfuscation so you added stars '*' in the string?
I think it should have looked like below for it to work. If it looks like the one above, then maybe we need to escape $ characters for this to work (just a hunch).
|
@johlju I think somehow ** must have been introduced by the markdown here at github after I pasted it. Here is screenshot from the errors: I got this behavior in two different environments....I appreciate if someone here try to replicate this issue in a lab... |
The username seem to be sent correctly to the setup process. Is it that SQL Server does not like a username having '$' character in it. 🤔 |
@johlju that is a good troubleshooting step to rule out certain scenarios. |
Hmm... 🤔 When you run manually thru the GUI can you see what the Summary.txt file look like for this argument? You find it in the following path "C:\Program Files\Microsoft SQL Server\140\Setup Bootstrap\Log\Summary.txt". Change to the correct major version number depending on SQL Server version. Could you run the arguments from the screenshot above from the command line? Just |
I had time to run this in the lab today. I could reproduce the problem. Confirming that this is a bug. The problem is that the password argument is not returned by the helper function This regex seems wrong since the '$' sign is at the end of the account name, not after the backslash.
The previous behavior was to check at the end of the string, but that check could give false positives also. See the change that was made at the diff d2636b3#diff-2031fd07fa2054ae7de513597a3acb0eL1374 I think we should change the regex to
|
- Now accounts containing '$' will be able to be used for installing SQL Server. Although, if the account ends with '$' it is considered a Managed Service Account (issue dsccommunity#1055.
- Now accounts containing '$' will be able to be used for installing SQL Server. Although, if the account ends with '$' it is considered a Managed Service Account (issue dsccommunity#1055).
- Changes to SqlSetup - Now accounts containing '$' will be able to be used for installing SQL Server. Although, if the account ends with '$' it is considered a Managed Service Account (issue #1055).
@johlju Thank you so much for figuring this out. I back ported the fix in older xSQLServer module I am using (version 9.0.0.0 )and tested it today and the fix worked as expected. |
Details of the scenario you tried and the problem that is occurring:
SQL Server install fails when the service account used is a domain account that starts with $ sign. Ex. $SQLSvcAcct.
Here is the error I got when running it from ISE:
I have checked setup summary.txt and here is what I got:
and here is portion of the Config file generated by setup.exe
If I changed the service account from
contoso\$SQLSvcAcct
tocontoso\SQLSvcAcct
installation will be successful.Does anyone run to this issue before? Any idea what is going on?
The DSC configuration that is using the resource (as detailed as possible):
I can't share at this time but I have tried it from simple DSC configuration to moderately complex one
Version of the Operating System, SQL Server and PowerShell the DSC Target Node is running:
Windows 2012 R2, PowerShell 5.1, SQL Server 2016
What module (SqlServer or SQLPS) and which version of the module the DSC Target Node is running:
Tested on both on SQLServer and SQLPS module and got the same issue
Version of the DSC module you're using, or 'dev' if you're using current dev branch:
xSQLServer module V 9.0.0.0
The text was updated successfully, but these errors were encountered: