Updates for Standard Edition

[rhealitycheck]

Description

Instance name is modified because SQLEXPRESS is used as an instance name for SQL Server. In Standard or Enterprise installs, SQL Server will default to MSSQLSERVER. Any instance name declared by the user will have MSSQL$ appeneded to the front as the service name.
Agent name needs to be declared because if you use the SQL Agent, you need to restart both services as the Agent is dependent on the SQL Service
If you have declared an agent account it will restart both the agent service and the sql service. If not only restart the sql service
Also added return codes because the install periodically returns 3010 which means a reboot is pending and is not an actual error, the other return codes were taken from the chef output.
SQL Server requires a reboot to complete your install

Issues Resolved

N/A

Check List

• All tests pass. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/TESTING.MD
  -- tested with test kitchen and also locally
• New functionality includes testing.
  -- same testing as before. the out of the box cookbook doesn't work for standard or enterprise if you use instance names other than MSSQLSERVER which is the default.
• [n/a] New functionality has been documented in the README if applicable
• The CLA has been signed. See https://github.com/chef-cookbooks/community_cookbook_documentation/blob/master/CONTRIBUTING.MD

[Annih]

Duplicate definition of the service

[Annih]

Hi @smurawski
I saw that you are preparing the new 2.6.0 release including this patch. I think it should be clearer in the changelog or the version number that the cookbook is now rebooting the node.

BTW you should also update the README stating that chef12+ is now required, because this patch leverages the reboot lwrp.

[smurawski]

@Annih Thanks! Good catch. I'm getting some readme updates together for that.