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

Snowflake Parameters Getting Failed | Terraform | Error [Bug]: #2767

Closed
1 task done
ravisingh27 opened this issue May 2, 2024 · 6 comments
Closed
1 task done

Snowflake Parameters Getting Failed | Terraform | Error [Bug]: #2767

ravisingh27 opened this issue May 2, 2024 · 6 comments
Labels
category:sdk feature-request Used to mark issues with provider's missing functionalities

Comments

@ravisingh27
Copy link

Terraform CLI Version

1.7.5

Terraform Provider Version

0.89.0

Terraform Configuration

We presently use terraform automation to apply snowflake parameters, and we do so for 13 parameters total. These parameters are passed in as Terraform variables (map). We tried adding the two arguments below to our code recently, however we were receiving the error below. A remedy for this would be beneficial.

TERRAFORM_VERSION "1.7.5"

Error:
╷
│ Error: Invalid object parameter: ENABLE_TRI_SECRET_AND_REKEY_OPT_OUT_FOR_IMAGE_REPOSITORY
│ 
│   with module.pre_snowflake_account_parameters["Account_Parameters-1"].snowflake_account_parameter.child_account_parameter,
│   on .terraform/modules/pre_snowflake_account_parameters/main.tf line 1, in resource "snowflake_account_parameter" "child_account_parameter":
│    1: resource "snowflake_account_parameter" "child_account_parameter" {
│ 
╵
╷
│ Error: Invalid object parameter: ENABLE_TRI_SECRET_AND_REKEY_OPT_OUT_FOR_SPCS_BLOCK_STORAGE
│ 
│   with module.pre_snowflake_account_parameters["Account_Parameters-2"].snowflake_account_parameter.child_account_parameter,
│   on .terraform/modules/pre_snowflake_account_parameters/main.tf line 1, in resource "snowflake_account_parameter" "child_account_parameter":1: resource "snowflake_account_parameter" "child_account_parameter" {
│ 
╵
Cleaning up project directory and file based variables
00:00
ERROR: Job failed: command terminated with exit code 1

/** Variable Section **/
Account_Parameters-1 = {
key  = "ENABLE_TRI_SECRET_AND_REKEY_OPT_OUT_FOR_IMAGE_REPOSITORY",
    value = true,
    },
Account_Parameters-2 = {
    key  = "ENABLE_TRI_SECRET_AND_REKEY_OPT_OUT_FOR_SPCS_BLOCK_STORAGE",
    value = true,
    },
Account_Parameters-14 = {
      key  = "PERIODIC_DATA_REKEYING",
      value = true,
    },

Category

category:resource

Object type(s)

No response

Expected Behavior

ENABLE_TRI_SECRET_AND_REKEY_OPT_OUT_FOR_IMAGE_REPOSITORY = true
ENABLE_TRI_SECRET_AND_REKEY_OPT_OUT_FOR_SPCS_BLOCK_STORAGE = true

Actual Behavior

Error:

│ Error: Invalid object parameter: ENABLE_TRI_SECRET_AND_REKEY_OPT_OUT_FOR_IMAGE_REPOSITORY

│ with module.pre_snowflake_account_parameters["Account_Parameters-1"].snowflake_account_parameter.child_account_parameter,
│ on .terraform/modules/pre_snowflake_account_parameters/main.tf line 1, in resource "snowflake_account_parameter" "child_account_parameter":
│ 1: resource "snowflake_account_parameter" "child_account_parameter" {



│ Error: Invalid object parameter: ENABLE_TRI_SECRET_AND_REKEY_OPT_OUT_FOR_SPCS_BLOCK_STORAGE

│ with module.pre_snowflake_account_parameters["Account_Parameters-2"].snowflake_account_parameter.child_account_parameter,
│ on .terraform/modules/pre_snowflake_account_parameters/main.tf line 1, in resource "snowflake_account_parameter" "child_account_parameter":
│ 1: resource "snowflake_account_parameter" "child_account_parameter" {


Cleaning up project directory and file based variables
00:00
ERROR: Job failed: command terminated with exit code 1

Account_Parameter_Error

Steps to Reproduce

sourcing the snowflake_account_parameters as a module.
mapping the key : value - as a terraform variable.
for_each = var.snowflake_account_parameters
key = each.value.key
value = each.value.value

terraform apply

How much impact is this issue causing?

Medium

Logs

No response

Additional Information

No response

Would you like to implement a fix?

  • Yeah, I'll take it 😎
@ravisingh27 ravisingh27 added the bug Used to mark issues with provider's incorrect behavior label May 2, 2024
@sfc-gh-asawicki
Copy link
Collaborator

Hey @ravisingh27. Thanks for reaching out to us.

These two parameters must have been added recently and we are currently not supporting them.

The change is relatively easy, though. I will squeeze it in with one of my other PRs and it should be available with the next release.

@sfc-gh-asawicki sfc-gh-asawicki added feature-request Used to mark issues with provider's missing functionalities and removed bug Used to mark issues with provider's incorrect behavior labels May 2, 2024
sfc-gh-asawicki added a commit that referenced this issue May 6, 2024
sfc-gh-asawicki added a commit that referenced this issue May 7, 2024
- Add missing parameters (References #2676)
- Temporary fix the warehouse resource behavior; it will be addressed
with the warehouse redesign because the existing conditional logic is
questionable and we plan to get rid of the defaults (SNOW-1348102)
(References #2763)
- Prove data type problem with functions; it will be addressed with
function redesign because it requires deeper refactor (SNOW-1348103)
(References #2735)
- Suppress the diff for synonym columns in table resource; it will be
addressed with table redesign because the collate test had to be skipped
(SNOW-1348114) (References #2733)
sfc-gh-jcieslak pushed a commit that referenced this issue May 8, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.90.0](v0.89.0...v0.90.0)
(2024-05-08)


### 🎉 **What's new:**

* Adjust owner_role_type and schema_evolution_record columns
([#2740](#2740))
([424e393](424e393))


### 🔧 **Misc**

* Add a guide on creating issues
([#2758](#2758))
([2b006aa](2b006aa))
* Add missing documentation
([#2781](#2781))
([cc0a6a7](cc0a6a7))
* Add scripts to close issues from Pre Snowflake bucket
([#2762](#2762))
([44c0c37](44c0c37))
* Add small adjustments
([#2783](#2783))
([e5b0b4b](e5b0b4b))
* Adjust issue templates
([#2748](#2748))
([64ab76d](64ab76d))
* Cleanup helpers part 3
([#2730](#2730))
([eb7bee4](eb7bee4))
* Cleanup helpers part 5
([#2744](#2744))
([1f165bf](1f165bf))
* Cleanup helpers part4
([#2741](#2741))
([9475e35](9475e35))
* Cleanup helpers part6
([#2745](#2745))
([eba3029](eba3029))
* Cleanup helpers poc
([#2724](#2724))
([70b99fb](70b99fb))
* Helpers cleanup continuation
([#2726](#2726))
([a70d1af](a70d1af))
* Prepare new roadmap entry
([#2773](#2773))
([b0bf28f](b0bf28f))
* Prepare parallel builds
([#2737](#2737))
([6974e25](6974e25))
* Update the contribution guidelines (and small fixes)
([#2753](#2753))
([aafdc72](aafdc72))


### 🐛 **Bug fixes:**

* Fix issue templates
([#2760](#2760))
([d0d5048](d0d5048))
* Fix setup for two tests
([#2757](#2757))
([df025b0](df025b0))
* Fix Test (wrong order of arguments)
([#2780](#2780))
([02f467e](02f467e))
* Fix/prove issues
[#2733](#2733)
[#2735](#2735)
[#2763](#2763)
[#2767](#2767)
([#2777](#2777))
([7b1c67e](7b1c67e))
* Prove problems with procedure resource data types
([#2782](#2782))
([68d0318](68d0318))
* read after create
([#2718](#2718))
([2e9b68f](2e9b68f))
* UNSET and empty SET in network policies
([#2759](#2759))
([3eacb0b](3eacb0b))
* unset network policy should not have single quotes
([#2584](#2584))
([8f2a363](8f2a363))
* Update failover group allowed integration types
([#2776](#2776))
([efde48d](efde48d))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: snowflake-release-please[bot] <105954990+snowflake-release-please[bot]@users.noreply.github.com>
@sfc-gh-jcieslak
Copy link
Collaborator

sfc-gh-jcieslak commented May 8, 2024

Hey @ravisingh27 👋
Today, we released a new version v0.90.0 that contained a fix for your issue. Please, let us know if you're still experiencing this issue in the latest version (also mind there were small breaking changes documented here).

@ravisingh27
Copy link
Author

Thanks, I'll check and close.

@sfc-gh-asawicki
Copy link
Collaborator

Hey @ravisingh27. Were you able to test the new version?

@ravisingh27
Copy link
Author

Hey @sfc-gh-asawicki , Not yet but we'll be including this in next quarter.

@sfc-gh-asawicki
Copy link
Collaborator

I will close the issue then. Please create a new one if the added parameters won't work.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:sdk feature-request Used to mark issues with provider's missing functionalities
Projects
None yet
Development

No branches or pull requests

3 participants