diff --git a/MIGRATION_GUIDE.md b/MIGRATION_GUIDE.md index cb74c85116..940c9ea38f 100644 --- a/MIGRATION_GUIDE.md +++ b/MIGRATION_GUIDE.md @@ -11,9 +11,30 @@ across different versions. ### Removed deprecated objects All of the deprecated objects are removed from v1 release. This includes: + - Resources + - `snowflake_database_old` + - `snowflake_role` + - `snowflake_oauth_integration` + - `snowflake_saml_integration` + - `snowflake_stream` + - `snowflake_tag_masking_policy_association` - Data sources -- Fields + - `snowflake_role` +- Fields in the provider configuration: + - `account` + - `oauth_access_token` + - `oauth_client_id` + - `oauth_client_secret` + - `oauth_endpoint` + - `oauth_redirect_url` + - `oauth_refresh_token` + - `private_key_path` + - `region` + - `session_params` + - `username` + +Additionally, `JWT` value is no longer available for `authenticator` field in the provider configuration. ## v0.98.0 ➞ v0.99.0 diff --git a/docs/resources/tag_association.md b/docs/resources/tag_association.md index b24738371d..77acc40091 100644 --- a/docs/resources/tag_association.md +++ b/docs/resources/tag_association.md @@ -90,6 +90,7 @@ resource "snowflake_tag_association" "column_association" { ### Optional +- `object_name` (String, Deprecated) Specifies the object identifier for the tag association. - `skip_validation` (Boolean) If true, skips validation of the tag association. - `timeouts` (Block, Optional) (see [below for nested schema](#nestedblock--timeouts)) diff --git a/pkg/provider/resources/resources.go b/pkg/provider/resources/resources.go index 88471a06e3..1825d1cbaf 100644 --- a/pkg/provider/resources/resources.go +++ b/pkg/provider/resources/resources.go @@ -2,7 +2,6 @@ package resources type resource string -// TODO: revisit const ( Account resource = "snowflake_account" AccountRole resource = "snowflake_account_role" diff --git a/pkg/resources/tag_association.go b/pkg/resources/tag_association.go index b1a793a301..629aa234f9 100644 --- a/pkg/resources/tag_association.go +++ b/pkg/resources/tag_association.go @@ -19,6 +19,13 @@ import ( ) var tagAssociationSchema = map[string]*schema.Schema{ + "object_name": { + Type: schema.TypeString, + Optional: true, + Description: "Specifies the object identifier for the tag association.", + ForceNew: true, + Deprecated: "Use `object_identifier` instead", + }, "object_identifier": { Type: schema.TypeList, MinItems: 1, diff --git a/pkg/sdk/config.go b/pkg/sdk/config.go index 607b243a02..ce82b50111 100644 --- a/pkg/sdk/config.go +++ b/pkg/sdk/config.go @@ -426,7 +426,6 @@ var AllAuthenticationTypes = []AuthenticationType{ } func ToAuthenticatorType(s string) (gosnowflake.AuthType, error) { - // TODO: simplify switch strings.ToUpper(s) { case string(AuthenticationTypeSnowflake): return gosnowflake.AuthTypeSnowflake, nil diff --git a/pkg/sdk/config_test.go b/pkg/sdk/config_test.go index 8f67022ba0..436e09e661 100644 --- a/pkg/sdk/config_test.go +++ b/pkg/sdk/config_test.go @@ -284,7 +284,6 @@ func Test_toAuthenticationType(t *testing.T) { {input: "OAUTH", want: gosnowflake.AuthTypeOAuth}, {input: "EXTERNALBROWSER", want: gosnowflake.AuthTypeExternalBrowser}, {input: "OKTA", want: gosnowflake.AuthTypeOkta}, - {input: "JWT", want: gosnowflake.AuthTypeJwt}, {input: "SNOWFLAKE_JWT", want: gosnowflake.AuthTypeJwt}, {input: "TOKENACCESSOR", want: gosnowflake.AuthTypeTokenAccessor}, {input: "USERNAMEPASSWORDMFA", want: gosnowflake.AuthTypeUsernamePasswordMFA}, diff --git a/v1-preparations/LIST_OF_REMOVED_RESOURCES_FOR_V1.md b/v1-preparations/LIST_OF_REMOVED_RESOURCES_FOR_V1.md index 0624766156..d61fc063b7 100644 --- a/v1-preparations/LIST_OF_REMOVED_RESOURCES_FOR_V1.md +++ b/v1-preparations/LIST_OF_REMOVED_RESOURCES_FOR_V1.md @@ -6,3 +6,4 @@ Deprecated resources that are removed with the V1: * [snowflake_oauth_integration](https://registry.terraform.io/providers/Snowflake-Labs/snowflake/0.98.0/docs/resources/oauth_integration) * [snowflake_saml_integration](https://registry.terraform.io/providers/Snowflake-Labs/snowflake/0.98.0/docs/resources/saml_integration) * [snowflake_stream](https://registry.terraform.io/providers/Snowflake-Labs/snowflake/0.98.0/docs/resources/stream) +* [snowflake_tag_masking_policy_association](https://registry.terraform.io/providers/Snowflake-Labs/snowflake/0.98.0/docs/resources/tag_masking_policy_association)