Merge pull request #19 from getindata/fix/custom_role_enabled_logic

fix: Custom role enabled parameter logic

.pre-commit-config.yaml

@@ -15,7 +15,7 @@ repos:
         args: ["."]
 
   - repo: https://github.com/bridgecrewio/checkov.git
-    rev: "3.2.213" # Get the latest from: https://github.com/bridgecrewio/checkov/releases
+    rev: "3.2.216" # Get the latest from: https://github.com/bridgecrewio/checkov/releases
     hooks:
       - id: checkov
         args: [--skip-check, "CKV_TF_1"] # Terraform module sources do not use a git url with a commit hash revision

.tflint.hcl

@@ -1,16 +1,3 @@
-# config {
-#   ignore_module = {
-#     "Invicton-Labs/deepmerge/null" = true
-#   }
-# }
-
-plugin "terraform" {
-  enabled = true
-  version = "0.5.0"
-  source  = "github.com/terraform-linters/tflint-ruleset-terraform"
-  preset  = "all"
-}
-
 rule "terraform_standard_module_structure" {
   enabled = false # Fails on context.tf
 }

examples/simple/README.md

@@ -30,7 +30,9 @@ No providers.
 
 ## Requirements
 
-No requirements.
+| Name | Version |
+|------|---------|
+| <a name="requirement_terraform"></a> [terraform](#requirement\_terraform) | >= 1.3 |
 
 ## Resources
 

examples/simple/versions.tf

@@ -0,0 +1,3 @@
+terraform {
+  required_version = ">= 1.3"
+}

main.tf

@@ -69,7 +69,7 @@ module "snowflake_custom_role" {
 
   name            = each.key
   attributes      = ["RMN", one(snowflake_resource_monitor.this[*].name)]
-  enabled         = local.create_default_roles && lookup(each.value, "enabled", true)
+  enabled         = lookup(each.value, "enabled", true)
   descriptor_name = lookup(each.value, "descriptor_name", "snowflake-role")
 
   role_ownership_grant = lookup(each.value, "role_ownership_grant", "SYSADMIN")