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

SNOW-1677435: Session builder does not check for private_key_path when connecting using key-pair authentication #2343

Closed
sfc-gh-ccardillo opened this issue Sep 23, 2024 · 3 comments
Assignees
Labels
bug Something isn't working status-triage_done Initial triage done, will be further handled by the driver team

Comments

@sfc-gh-ccardillo
Copy link

(must use key_path_file)

Please answer these questions before submitting your issue. Thanks!

  1. What version of Python are you using?

3.10.11

  1. What operating system and processor architecture are you using?

macOS-14.6.1-arm64-arm-64bit

  1. What are the component versions in the environment (pip freeze)?

(Cannot copy/paste)
snowflake-connector-python==3.11.0

  1. What did you do?

Tried to create a session based on a connection created via the snow CLI. In short, snow CLI stores key_path_path in the config.tom, whereas the snowpark connector looks for key_path_file. Adding key_path_file to the relevant connection block in config.toml resolves the issue.

  1. What did you expect to see?

I expected the connection created with snow CLI to work out of the box in the snowpark connector

@sfc-gh-ccardillo sfc-gh-ccardillo added bug Something isn't working needs triage Initial RCA is required labels Sep 23, 2024
@github-actions github-actions bot changed the title Session builder does not check for private_key_path when connecting using key-pair authentication SNOW-1677435: Session builder does not check for private_key_path when connecting using key-pair authentication Sep 23, 2024
@sfc-gh-sghosh sfc-gh-sghosh self-assigned this Sep 24, 2024
@sfc-gh-sghosh
Copy link

Hello @sfc-gh-ccardillo ,

Thanks for raising the issue, could you please provide more detail, how you using snowcli with snowpark, are you creating a snowpark project using snowcli ?

https://docs.snowflake.com/en/developer-guide/snowflake-cli-v2/snowpark/overview

Could you please provide the code snippet about the issue you hitting?

Regards,
Sujan

@sfc-gh-sghosh sfc-gh-sghosh added status-triage_needed This is a new issue, and initial triage is needed status-triage Issue is under initial triage and removed needs triage Initial RCA is required labels Sep 24, 2024
@sfc-gh-dszmolka sfc-gh-dszmolka added status-information_needed Additional information is required from the reporter and removed status-triage_needed This is a new issue, and initial triage is needed labels Sep 24, 2024
@sfc-gh-sghosh
Copy link

Hello @sfc-gh-ccardillo ,

I guess you hitting this error, snowflakedb/snowflake-cli#1480
and this issue is fixed.

I am able to reproduce the above issue as well.

Regards,
Sujan

@sfc-gh-sghosh sfc-gh-sghosh added status-triage_done Initial triage done, will be further handled by the driver team and removed status-information_needed Additional information is required from the reporter status-triage Issue is under initial triage labels Sep 30, 2024
@sfc-gh-sghosh
Copy link

Closing the issue as the fix rolled out in snowcli 3.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working status-triage_done Initial triage done, will be further handled by the driver team
Projects
None yet
Development

No branches or pull requests

3 participants