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

WIP -- Update snowchange.py to drop region flag and support key authentication #1

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

nicholashauschild
Copy link

Details:

  • The region parameter that the python connector uses is deprecated. To refer to region, along with cloud provider, append it as appropriate in the account paramter. The change was propagated up to the snowchange entrypoint.
  • Private key authentication support has been added. To use it, a new flag was added to the the snowchange entrypoint to allow for optionally specifying a private key path (-k or --private-key-file). This also requires the passphrase used to decrypt the private key file is presented as an environment variable "SNOWSQL_PRIVATE_KEY_PASSPHRASE".

Details:
 - The region parameter that the python connector uses is deprecated. To refer to region, along with cloud provider, append it as appropriate in the account paramter. The change was propagated up to the snowchange entrypoint.
 - Private key authentication support has been added. To use it, a new flag was added to the the snowchange entrypoint to allow for optionally specifying a private key path (-k or --private-key-file). This also requires the passphrase used to decrypt the private key file is presented as an environment variable "SNOWSQL_PRIVATE_KEY_PASSPHRASE".
@nicholashauschild nicholashauschild changed the title Update snowchange.py to drop region flag and support key authentication WIP -- Update snowchange.py to drop region flag and support key authentication Jul 7, 2020
granthenke pushed a commit that referenced this pull request Mar 25, 2023
Expose SF Python connector autocommit option
granthenke pushed a commit that referenced this pull request Mar 25, 2023
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