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

feat: Align module with Snowflake provider changes #34

Merged
merged 18 commits into from
Aug 5, 2024

Conversation

moss2k13
Copy link
Member

@moss2k13 moss2k13 commented Jul 30, 2024

BREAKING CHANGES IN v2.x.x

Due to breaking changes in Snowflake provider and additional code optimizations, breaking changes were introduced in v2.0.0 version of this module.

List of code and variable (API) changes:

  • Switched to snowflake_database_role module to leverage new database_roles mechanism
  • database default_roles and custom_roles are now managed by getindata/database_role/snowflake module
  • snowflake_database resource was updated to use newly introduced changes in Snowflake provider
  • snowflake_schema resource was updated to use newly introduced changes in Snowflake provider
  • snowflake_grant_ownership resource has been added
  • variable add_grants_to_existing_objects was removed as it is no longer needed
  • minimum Snowflake provider version is 0.90.0

For more information, refer to variables.tf, list of inputs below and Snowflake provider documentation

When upgrading from v1.x, expect most of the resources to be recreated - if recreation is impossible, then it is possible to import some existing resources.

Due to breaking changes in Snowflake provider and additional code optimizations, **breaking changes** were introduced in this module.

List of code and variable (API) changes:
- Switched to `snowflake_database_role` module to leverage new `database_roles` mechanism
- `default_roles` and `custom_roles` are now combined and controlled by single logic in snowflake_database_role module
- snowflake_database resource was updated to use newly introduced changes in Snowflake provider
- snowflake_schema resource was updated to use newly introduced changes in Snowflake provider
- variable `add_grants_to_existing_objects` was removed as it is no longer needed
- minimum Snowflake provider version is `0.90.0`

For more information, refer to [variables.tf](variables.tf), list of inputs below and Snowflake provider documentation

When upgrading from `v1.x`, expect most of the resources to be recreated - if recreation is impossible, then it is possible to import some existing resources.
main.tf Outdated Show resolved Hide resolved
variables.tf Outdated Show resolved Hide resolved
@moss2k13 moss2k13 changed the title Align module with Snowflake provider changes feat: Align module with Snowflake provider changes Jul 31, 2024
locals.tf Outdated Show resolved Hide resolved
locals.tf Show resolved Hide resolved
main.tf Outdated Show resolved Hide resolved
main.tf Outdated Show resolved Hide resolved
variables.tf Show resolved Hide resolved
variables.tf Outdated Show resolved Hide resolved
variables.tf Outdated Show resolved Hide resolved
main.tf Show resolved Hide resolved
variables.tf Outdated Show resolved Hide resolved
variables.tf Outdated Show resolved Hide resolved
locals.tf Show resolved Hide resolved
@moss2k13 moss2k13 marked this pull request as ready for review August 1, 2024 14:33
outputs.tf Outdated Show resolved Hide resolved
main.tf Outdated Show resolved Hide resolved
main.tf Outdated Show resolved Hide resolved
variables.tf Show resolved Hide resolved
@moss2k13 moss2k13 requested a review from dgniewek August 2, 2024 12:33
variables.tf Outdated Show resolved Hide resolved
variables.tf Outdated Show resolved Hide resolved
outputs.tf Outdated Show resolved Hide resolved
@moss2k13 moss2k13 merged commit 2650829 into main Aug 5, 2024
8 checks passed
@moss2k13 moss2k13 deleted the feature/align_module_with_provider_changes branch August 5, 2024 08:12
Copy link
Contributor

github-actions bot commented Aug 5, 2024

The new version v2.0.0 has been released 🎉

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants