Skip to content

All tests use keypair auth#175

Open
vinay-gopalan wants to merge 5 commits intomainfrom
vinayg/fix-snowflake-tests
Open

All tests use keypair auth#175
vinay-gopalan wants to merge 5 commits intomainfrom
vinayg/fix-snowflake-tests

Conversation

@vinay-gopalan
Copy link
Contributor

Overview

Userpass auth is deprecated and phased out by Snowflake. Going forward all tests with use keypair auth, and will require the following variables to function:

  • SNOWFLAKE_PRIVATE_KEY
  • SNOWFLAKE_USER
  • SNOWFLAKE_ACCOUNT
  • SNOWFLAKE_DATABASE

@vinay-gopalan vinay-gopalan requested a review from a team as a code owner March 18, 2026 18:23
defaultRSAKeyCreationStmts = `
CREATE USER {{username}} RSA_PUBLIC_KEY='{{public_key}}';
GRANT ROLE public TO USER {{username}};
GRANT USAGE ON DATABASE %s TO USER {{username}};`
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

granting usage looks like a requirement when using keypair auth. Without this, created users do not have the ability to initiate a successful db.Ping()

Copy link
Member

@robmonte robmonte left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice fix!

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.

2 participants