Skip to content

(FM-5389) Fix sql_features when installing SNAC_SDK #219

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

Merged
merged 3 commits into from
Jun 21, 2017
Merged

(FM-5389) Fix sql_features when installing SNAC_SDK #219

merged 3 commits into from
Jun 21, 2017

Conversation

glennsarti
Copy link
Contributor

Previously the sql_features was not idempotent when installing the SQL Native
Client SDK (SNAC_SDK). This Due to the SQL Native Client not being unique
across SQL Server versions (e.g. SQL 2016 (v13) installs Native Client with a
version that matches for SQL 2012 (v11)). Due to MODULES-5060 it is noted that
the SQL Server Module can not be used to manage different SQL Server versions
on the same host. Knowing this it is safe to collate all of the SQL Server
shared features, regardless of version, and use that as the list of installed
features and is safely idempotent.

This commit also adds a README section to known issues about this issue.

Previously the sql_features was not idempotent when installing the SQL Native
Client SDK (SNAC_SDK).  This Due to the SQL Native Client not being unique
across SQL Server versions (e.g. SQL 2016 (v13) installs Native Client with a
version that matches for SQL 2012 (v11)).  Due to MODULES-5060 it is noted that
the SQL Server Module can not be used to manage different SQL Server versions
on the same host.  Knowing this it is safe to collate all of the SQL Server
shared features, regardless of version, and use that as the list of installed
features and is safely idempotent.

This commit also adds a README section to known issues about this issue.
The SNAC_SDK feature can not be managed by the module as it is not able to be
uninstalled using the SQL Server installation media (particularly on SQL Server
2016).  This commit adds a note about this limitation in the README.
@glennsarti
Copy link
Contributor Author

adhoc acceptance tests have passed. This is ready for merge.

@glennsarti glennsarti changed the title {WIP}(FM-5389) Fix sql_features when installing SNAC_SDK (FM-5389) Fix sql_features when installing SNAC_SDK Jun 20, 2017
@jpogran jpogran merged commit 108d109 into puppetlabs:master Jun 21, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants