[MODULES-7553] sqlserver module, documentation: Advanced Example using the hostname fact has issues Created: 2018/07/30 Updated: 2019/05/20 Resolved: 2019/05/20
|Reporter:||Patrick Grant||Assignee:||William Hurt|
|Labels:||Support, maintenance, triage, windows_engineer_triaged|
|Remaining Estimate:||Not Specified|
|Time Spent:||Not Specified|
|Original Estimate:||Not Specified|
|Epic Link:||IAC - Customer Escalations|
|QA Risk Assessment:||Needs Assessment|
Ticket raised on behalf of PE customer using our sqlserver module.
The customer had an issue when they followed the modules advanced example: https://forge.puppet.com/puppetlabs/sqlserver#advanced-example
Like the example, the customer tried to use the hostname fact when assigning a user to a role.
However, this didn’t work and the customer had to specifically use the Uppercase domain shortname rather than the hostname fact as below:
The customer feels that this needs to be documented or there needs to be an engineering effort to account for this.
|Comment by Benjamin Ostrowski [ 2018/08/01 ]|
FYI, it wasn't the hostname fact I was attempting to use, it was the domain fact that gets resolved to the FQDN of the domain that the server is joined to (e.g. mydomain.com). It would be my hope that i wouldn't have explicitly define a variable such as
but instead use the fact that already exists for "domain" to be able to add these logins and roles. Such as...
|Comment by William Hurt [ 2019/05/10 ]|
The documentation should be updated to point out that the `$::domain` facter fact is not suitable for constructing user names.
I have also raised an issue with the vox module windows_env asking for feedback on the idea of adding the NETBIOS domain name to their facts module. This would give users a suitable fact for this purpose without having to hard code strings like this, or attempt to substring them in ways that could be difficult for some users.
|Comment by William Hurt [ 2019/05/16 ]|
My offer to add NETBIOS name as a fact to the vox windows_env module was not accepted. This ticket will now default to a simple documentation change.
|Comment by William Hurt [ 2019/05/20 ]|