Skip to content

fix: add ODBC Driver 17 for sql when process custom data #706

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.

Sign up for GitHub

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

Jump to bottom
Merged
Avijit-Microsoft merged 1 commit into from
Jan 12, 2026
Merged

fix: add ODBC Driver 17 for sql when process custom data #706

Avijit-Microsoft merged 1 commit into from
Jan 12, 2026

Conversation

@cyberflying
Copy link
Contributor

@cyberflying cyberflying commented Dec 27, 2025

Purpose

  • add ODBC Driver 17 for sql when process custom data, same code with 03 sample data script,
  • to avoid error:
Traceback (most recent call last):
  File "/Conversation-Knowledge-Mining-Solution-Accelerator/infra/scripts/index_scripts/04_cu_process_custom_data.py", line 179, in <module>
    conn = pyodbc.connect(connection_string, attrs_before={SQL_COPT_SS_ACCESS_TOKEN: token_struct})
           ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
pyodbc.Error: ('01000', "[01000] [unixODBC][Driver Manager]Can't open lib 'ODBC Driver 18 for SQL Server' : file not found (0) (SQLDriverConnect)")
Error: 04_cu_process_custom_data.py failed.

Does this introduce a breaking change?

  • Yes
  • No

Golden Path Validation

  • I have tested the primary workflows (the "golden path") to ensure they function correctly without errors.

Deployment Validation

  • I have validated the deployment process successfully and all services are running as expected with this change.

What to Check

Verify that the following are valid

  • ...

Other Information

@Avijit-Microsoft Avijit-Microsoft merged commit 6f493b5 into microsoft:main Jan 12, 2026
6 checks passed
@github-actions
Copy link
Contributor

github-actions bot commented Jan 12, 2026

🎉 This PR is included in version 3.17.3 🎉

The release is available on GitHub release

Your semantic-release bot 📦🚀

@Avijit-Microsoft
Copy link
Contributor

Avijit-Microsoft commented Jan 12, 2026

Hi @cyberflying,
Thanks for the PR and your efforts on this. The changes look good, and we’re happy to merge it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Avijit-Microsoft Avijit-Microsoft Avijit-Microsoft approved these changes

@Roopan-Microsoft Roopan-Microsoft Awaiting requested review from Roopan-Microsoft Roopan-Microsoft is a code owner

@Prajwal-Microsoft Prajwal-Microsoft Awaiting requested review from Prajwal-Microsoft Prajwal-Microsoft is a code owner

@aniaroramsft aniaroramsft Awaiting requested review from aniaroramsft aniaroramsft is a code owner

@brittneek brittneek Awaiting requested review from brittneek brittneek is a code owner

@Vinay-Microsoft Vinay-Microsoft Awaiting requested review from Vinay-Microsoft Vinay-Microsoft is a code owner

@toherman-msft toherman-msft Awaiting requested review from toherman-msft toherman-msft is a code owner

@nchandhi nchandhi Awaiting requested review from nchandhi nchandhi is a code owner

@dgp10801 dgp10801 Awaiting requested review from dgp10801 dgp10801 is a code owner

Assignees

No one assigned

Labels

released

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cyberflying @Avijit-Microsoft