Warehouse creation fails due to quoted identifiers in the SHOW PARAMETERS IN WAREHOUSE query.

[sshah90]

Provider & Terraform Version

❯ terraform version
Terraform v1.1.4
on darwin_amd64

• provider registry.terraform.io/chanzuckerberg/snowflake v0.25.36
• provider registry.terraform.io/hashicorp/random v3.0.1

Describe the bug
I am trying to create a warehouse using snowflake_warehouse resource which fails during terraform apply command with Warehouse 'TEST' does not exist or not authorized.

It does create a warehouse in Snowflake but it's not able to update terraform state files while running SHOW PARAMETERS IN WAREHOUSE test.

Query History from Snowflake side:

All other queries do have quotes but not with warehouse one.

Expected behavior

terraform apply command should create a warehouse and successfully exit.

Code samples and commands

resource "snowflake_warehouse" "warehouse" {
    for_each       = var.warehouses
    name           = each.key
    comment        = each.value.comment
    warehouse_size = each.value.size
}

variable file :

warehouses = {
    test = {
      comment = "test warehouse"
      size    = "X-Small"
    }
}

Additional context

After debugging, I found this discussion and given suggestion works fine but it stores all object names as the upper case so in the next run it forces changes on other resources like schema or role.

example:

[sshah90]

looks like PR #745 introduced show parameter query in the warehouse resource.

@sonmezonur any thoughts?

[sonmezonur]

@sshah90 Hello. As far as I understand, provider creates warehouses using case-sensitive names (since it surrounded names with double quotes). I have not faced this issue since we only use UpperCase characters while naming our warehouses (only want case-insensitive warehouses). If you surround warehouse name with double quotes, it will probably work. Just need to change this line in the codebase:

https://github.com/chanzuckerberg/terraform-provider-snowflake/blob/d83c499910c0f2b6348191a93f917e450b9e69b2/pkg/snowflake/warehouse.go#L43

to

 return fmt.Sprintf("SHOW PARAMETERS IN WAREHOUSE %q", wb.Builder.name) 

[sonmezonur]

Did you try to change your variable file to:

warehouses = {
    TEST = {
      comment = "test warehouse"
      size    = "X-Small"
    }
}

If you don't have any special reason for using case-sensitive warehouses, I recommend trying uppercase characters in your warehouse setup.

[sshah90]

Hi Sonmezonur, thanks for the suggestion.

With upper case, it works fine. Ideally, it should work with case-insensitive naming standards as the provider allows that with other resouces.

[sfc-gh-asawicki]

We are closing this issue as part of a cleanup described in announcement. If you believe that the issue is still valid in v0.89.0, please open a new ticket.