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

(FM-5389) Add missing shared feature SNAC_SDK #218

Merged
merged 1 commit into from Jun 16, 2017
Merged

(FM-5389) Add missing shared feature SNAC_SDK #218

merged 1 commit into from Jun 16, 2017

Conversation

glennsarti
Copy link
Contributor

Previously in commit 84f75b7 the feature list was extended however it missed
the SQL Native Client SDK. This commit adds the SNAC_SDK as a detectable and
installable feature. This commit also updates the README.

Previously in commit 84f75b7 the feature list was extended however it missed
the SQL Native Client SDK.  This commit adds the SNAC_SDK as a detectable and
installable feature.  This commit also updates the README.
@glennsarti
Copy link
Contributor Author

glennsarti commented Jun 15, 2017

Note SQL Server 2016 (v13) installs SNAC and SNAC SDK v11 which equates to SQL Server 2012 e.g.

PS> puppet facts
...
    "sqlserver_features": {                  
      "SQL_2012": [                          
        "SNAC_SDK"                           
      ],                                     
      "SQL_2014": [                          
                                             
      ],                                     
      "SQL_2016": [                          
        "Conn",                              
        "SDK",                               
        "MDS",                               
        "BC",                                
        "DQC",                               
        "BOL",                               
        "DREPLAY_CTLR",                      
        "DREPLAY_CLT",                       
        "SQL_SHARED_MR",                     
        "IS"                                 
      ]                                      
    },                                       
...

@jpogran jpogran merged commit ca3904a into puppetlabs:master Jun 16, 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.

None yet

2 participants