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

Setup terraform for Snowflake CI instance #1062

Closed
wants to merge 2 commits into from

Conversation

mikealfare
Copy link
Contributor

Problem

Our existing CI database was setup over the course of several manually run SQL statements. This has made it more difficult to figure out what configurations are needed to run dbt-snowflake. It also requires elevated access even to see what was done in the database. We are currently migrating to a different Snowflake account. This would have been simple if we had scripted our environment.

Solution

Let's take this opportunity to script out the infra required to support dbt-snowflake as well as testing dbt-snowflake in CI. We'll use terraform as both an industry and company standard.

Checklist

  • I have read the contributing guide and understand what's expected of me
  • I have run this code in development and it appears to resolve the stated issue
  • This PR includes tests, or tests are not required/relevant for this PR
  • This PR has no interface changes (e.g. macros, cli, logs, json artifacts, config files, adapter interface, etc) or this PR has already received feedback and approval from Product or DX

@mikealfare mikealfare self-assigned this May 24, 2024
@cla-bot cla-bot bot added the cla:yes label May 24, 2024
Copy link
Contributor

Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the dbt-snowflake contributing guide.

@amychen1776
Copy link

Closing this one because we are working with the Infra team to set this one up

@mikealfare mikealfare deleted the terraform-ci-database branch November 9, 2024 02:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants