Skip to content

Add terraform info note to Snowflake source/warehouse #6702

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

Merged
merged 2 commits into from
Jun 13, 2024

Conversation

forstisabella
Copy link
Contributor

Proposed changes

Added note that a user can use terraform to create RETL source/warehouse for Snowflake

Merge timing

ASAP after approval!

Related issues (optional)

#6559

@forstisabella forstisabella requested review from a team and pwseg and removed request for a team June 12, 2024 16:47
@@ -9,6 +9,9 @@ redirect_from:

[Snowflake](https://docs.snowflake.net/manuals/index.html){:target="_blank"} is a data warehouse, built for the cloud, that delivers performance, simplicity, concurrency and affordability.

> info ""
> Segment has a Terraform provider, powered by the Public API, that you can use to create a Snowflake warehouse. See the [segment_warehouse (Resource)](https://registry.terraform.io/providers/segmentio/segment/latest/docs/resources/warehouse){:target="_blank”} documentation for more information.
Copy link

@kashypAkash kashypAkash Jun 12, 2024

Choose a reason for hiding this comment

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

@forstisabella are we gonna add note for snowflake as source as well? Under reverse etl section?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes!! it's in the PR now, I just forgot to save the file before committing! sorry

@forstisabella forstisabella requested a review from a team as a code owner June 12, 2024 16:56
Copy link

@kashypAkash kashypAkash left a comment

Choose a reason for hiding this comment

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

LGTM

@forstisabella forstisabella merged commit 7d8e349 into develop Jun 13, 2024
4 checks passed
@forstisabella forstisabella deleted the add-terraform-instructions-keypair-auth branch June 13, 2024 15:28
Copy link
Contributor

Thank you for your contribution! Your pull request is merged, but may take a day or two to appear on the site.

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.

3 participants