Merge pull request #20 from getindata/fix/custom_role_logic

fix: Create custom role logic

Author: dgniewek

.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
 }

main.tf

@@ -78,7 +78,7 @@ module "snowflake_custom_role" {
 
   name            = each.key
   attributes      = ["WHS", one(snowflake_warehouse.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")