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

Valid warehouse sizes not accepted #1844

Closed
ian-r-rose opened this issue May 31, 2023 · 4 comments
Closed

Valid warehouse sizes not accepted #1844

ian-r-rose opened this issue May 31, 2023 · 4 comments
Labels
bug Used to mark issues with provider's incorrect behavior category:resource resource:warehouse Issue connected to the snowflake_warehouse resource

Comments

@ian-r-rose
Copy link

Provider Version

0.65.0

Terraform Version

1.x

Describe the bug

In the most recent release of the provider (0.65), valid warehouse sizes are no longer accepted (possibly related to #1792). If I choose a warehouse size "X-LARGE", I get an error message:

│ Error: expected warehouse_size to be one of [XSMALL SMALL MEDIUM LARGE XLARGE XXLARGE XXXLARGE X4LARGE X5LARGE X6LARGE], got X-LARGE

Previous versions of the provider worked, and were able to normalize warehouse sizes like X-SMALL or X-LARGE. Worse, it seems that Snowflake itself stores the warehouse sizes in the hyphenated form, so in my configurations the provider is unable to identify that XSMALL and X-SMALL are the same, and thinks there are changes to apply.

Expected behavior

Both hyphenated and non-hyphenated forms of warehouse sizes should be acceptable.

Additional context

This may be a consequence of #1792

@ian-r-rose ian-r-rose added the bug Used to mark issues with provider's incorrect behavior label May 31, 2023
@jreslock
Copy link
Contributor

jreslock commented Jun 8, 2023

We are seeing this behavior after updating our provider to 0.66.1 today.

@WojtekWaga
Copy link

In our case it misinterprets 3XLARGE read from Snowflake:

snowflake_warehouse.warehouses_se["ETL_3XL"] will be updated in-place

~ resource "snowflake_warehouse" "warehouses_se" {
id = "ETL_3XL"
name = "ETL_3XL"
~ warehouse_size = "3XLARGE" -> "xxxlarge"
# (12 unchanged attributes hidden)
}

@sfc-gh-jcieslak sfc-gh-jcieslak added category:resource resource:warehouse Issue connected to the snowflake_warehouse resource labels May 20, 2024
@sfc-gh-asawicki
Copy link
Collaborator

Hey @ian-r-rose @jreslock @WojtekWaga.

We are currently reworking the warehouse resource as part of https://github.com/Snowflake-Labs/terraform-provider-snowflake/blob/main/ROADMAP.md#preparing-essential-ga-objects-for-the-provider-v1. However, the customized diff suppression was already added in earlier provider versions (#1873 #1946), so this problem should not happen in recent ones.

Can you confirm that?

@sfc-gh-asawicki
Copy link
Collaborator

I am closing this issue due to inactivity. The problem should not appear in the newer versions of the provider. Please create a new issue if the problem persists in v0.93.0+ versions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Used to mark issues with provider's incorrect behavior category:resource resource:warehouse Issue connected to the snowflake_warehouse resource
Projects
None yet
Development

No branches or pull requests

5 participants