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

chore: Deprecate mssql_accept_microsoft_odbc_driver_17_for_sql_server_eula #318

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

spetrosi
Copy link
Collaborator

Enhancement: Deprecate this variable to use a variable without a version number instead. The role now can install versions 17 and 18.

Reason: After adding mssql_tools_versions, this role is able to install odbc driver for mssql_tools 17 and 18. This update changes the variable for specific version 17 to be general.

Result: When users use the deprecated variable mssql_accept_microsoft_odbc_driver_17_for_sql_server_eula, the role notifies them to use the new variable, but continues to execute.

Deprecate this variable to use a variable without a version number
instead. The role now can install versions 17 and 18.
@spetrosi spetrosi requested a review from richm as a code owner November 25, 2024 17:39
@spetrosi
Copy link
Collaborator Author

[citest]

@spetrosi spetrosi mentioned this pull request Nov 25, 2024
@spetrosi spetrosi changed the title ref: Deprecate mssql_accept_microsoft_odbc_driver_17_for_sql_server_eula chore: Deprecate mssql_accept_microsoft_odbc_driver_17_for_sql_server_eula Nov 25, 2024
@spetrosi spetrosi merged commit 12c0021 into linux-system-roles:main Nov 26, 2024
12 of 16 checks passed
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.

1 participant