[feature]Snowflake: ALL modules output privileges granted (#277)

Eduardo Lopez · web-flow · commit 62f0981ef768 · 2021-01-19T10:32:52.000-08:00
### Summary
ALL modules output privileges granted

### Test Plan
Using in internal repo
diff --git a/scripts/snowflake_generate_grant_all/main.go b/scripts/snowflake_generate_grant_all/main.go
@@ -45,6 +45,9 @@ type ModuleTemplate struct {
 
 	// required_providers: provider_name: version
 	Terraform map[string]map[string]map[string]string `json:"terraform,omitempty"`
+
+	// output name: arguments : values
+	Outputs map[string]map[string]interface{} `json:"output,omitempty"`
 }
 
 func main() {
@@ -116,6 +119,12 @@ func generateModule(name string, grant *resources.TerraformGrantResource) ([]byt
 		Locals: map[string]interface{}{
 			"privileges": privileges,
 		},
+		Outputs: map[string]map[string]interface{}{
+			"privileges": {
+				"value":       "${local.privileges}",
+				"description": "Privileges that make up the ALL set.",
+			},
+		},
 		Terraform: map[string]map[string]map[string]string{
 			"required_providers": {
 				"snowflake": map[string]string{
diff --git a/snowflake-account-grant-all/README.md b/snowflake-account-grant-all/README.md
@@ -21,6 +21,8 @@
 
 ## Outputs
 
-No output.
+| Name | Description |
+|------|-------------|
+| privileges | Privileges that make up the ALL set. |
 
 <!-- END -->
diff --git a/snowflake-account-grant-all/main.tf.json b/snowflake-account-grant-all/main.tf.json
@@ -47,5 +47,11 @@
         "version": "\u003e= 0.20.0"
       }
     }
+  },
+  "output": {
+    "privileges": {
+      "description": "Privileges that make up the ALL set.",
+      "value": "${local.privileges}"
+    }
   }
 }
diff --git a/snowflake-database-grant-all/README.md b/snowflake-database-grant-all/README.md
@@ -23,6 +23,8 @@
 
 ## Outputs
 
-No output.
+| Name | Description |
+|------|-------------|
+| privileges | Privileges that make up the ALL set. |
 
 <!-- END -->
diff --git a/snowflake-database-grant-all/main.tf.json b/snowflake-database-grant-all/main.tf.json
@@ -57,5 +57,11 @@
         "version": "\u003e= 0.20.0"
       }
     }
+  },
+  "output": {
+    "privileges": {
+      "description": "Privileges that make up the ALL set.",
+      "value": "${local.privileges}"
+    }
   }
 }
diff --git a/snowflake-external-table-grant-all/main.tf.json b/snowflake-external-table-grant-all/main.tf.json
@@ -70,5 +70,11 @@
         "version": "\u003e= 0.20.0"
       }
     }
+  },
+  "output": {
+    "privileges": {
+      "description": "Privileges that make up the ALL set.",
+      "value": "${local.privileges}"
+    }
   }
 }
diff --git a/snowflake-file-format-grant-all/main.tf.json b/snowflake-file-format-grant-all/main.tf.json
@@ -64,5 +64,11 @@
         "version": "\u003e= 0.20.0"
       }
     }
+  },
+  "output": {
+    "privileges": {
+      "description": "Privileges that make up the ALL set.",
+      "value": "${local.privileges}"
+    }
   }
 }
diff --git a/snowflake-function-grant-all/main.tf.json b/snowflake-function-grant-all/main.tf.json
@@ -82,5 +82,11 @@
         "version": "\u003e= 0.20.0"
       }
     }
+  },
+  "output": {
+    "privileges": {
+      "description": "Privileges that make up the ALL set.",
+      "value": "${local.privileges}"
+    }
   }
 }
diff --git a/snowflake-integration-grant-all/README.md b/snowflake-integration-grant-all/README.md
@@ -22,6 +22,8 @@
 
 ## Outputs
 
-No output.
+| Name | Description |
+|------|-------------|
+| privileges | Privileges that make up the ALL set. |
 
 <!-- END -->
diff --git a/snowflake-integration-grant-all/main.tf.json b/snowflake-integration-grant-all/main.tf.json
@@ -46,5 +46,11 @@
         "version": "\u003e= 0.20.0"
       }
     }
+  },
+  "output": {
+    "privileges": {
+      "description": "Privileges that make up the ALL set.",
+      "value": "${local.privileges}"
+    }
   }
 }
diff --git a/snowflake-materialized-view-grant-all/main.tf.json b/snowflake-materialized-view-grant-all/main.tf.json
@@ -70,5 +70,11 @@
         "version": "\u003e= 0.20.0"
       }
     }
+  },
+  "output": {
+    "privileges": {
+      "description": "Privileges that make up the ALL set.",
+      "value": "${local.privileges}"
+    }
   }
 }
diff --git a/snowflake-procedure-grant-all/main.tf.json b/snowflake-procedure-grant-all/main.tf.json
@@ -82,5 +82,11 @@
         "version": "\u003e= 0.20.0"
       }
     }
+  },
+  "output": {
+    "privileges": {
+      "description": "Privileges that make up the ALL set.",
+      "value": "${local.privileges}"
+    }
   }
 }
diff --git a/snowflake-resource-monitor-grant-all/README.md b/snowflake-resource-monitor-grant-all/README.md
@@ -22,6 +22,8 @@
 
 ## Outputs
 
-No output.
+| Name | Description |
+|------|-------------|
+| privileges | Privileges that make up the ALL set. |
 
 <!-- END -->
diff --git a/snowflake-resource-monitor-grant-all/main.tf.json b/snowflake-resource-monitor-grant-all/main.tf.json
@@ -46,5 +46,11 @@
         "version": "\u003e= 0.20.0"
       }
     }
+  },
+  "output": {
+    "privileges": {
+      "description": "Privileges that make up the ALL set.",
+      "value": "${local.privileges}"
+    }
   }
 }
diff --git a/snowflake-schema-grant-all/README.md b/snowflake-schema-grant-all/README.md
@@ -25,6 +25,8 @@
 
 ## Outputs
 
-No output.
+| Name | Description |
+|------|-------------|
+| privileges | Privileges that make up the ALL set. |
 
 <!-- END -->
diff --git a/snowflake-schema-grant-all/main.tf.json b/snowflake-schema-grant-all/main.tf.json
@@ -81,5 +81,11 @@
         "version": "\u003e= 0.20.0"
       }
     }
+  },
+  "output": {
+    "privileges": {
+      "description": "Privileges that make up the ALL set.",
+      "value": "${local.privileges}"
+    }
   }
 }
diff --git a/snowflake-sequence-grant-all/main.tf.json b/snowflake-sequence-grant-all/main.tf.json
@@ -64,5 +64,11 @@
         "version": "\u003e= 0.20.0"
       }
     }
+  },
+  "output": {
+    "privileges": {
+      "description": "Privileges that make up the ALL set.",
+      "value": "${local.privileges}"
+    }
   }
 }
diff --git a/snowflake-stage-grant-all/README.md b/snowflake-stage-grant-all/README.md
@@ -26,6 +26,8 @@
 
 ## Outputs
 
-No output.
+| Name | Description |
+|------|-------------|
+| privileges | Privileges that make up the ALL set. |
 
 <!-- END -->
diff --git a/snowflake-stage-grant-all/main.tf.json b/snowflake-stage-grant-all/main.tf.json
@@ -72,5 +72,11 @@
         "version": "\u003e= 0.20.0"
       }
     }
+  },
+  "output": {
+    "privileges": {
+      "description": "Privileges that make up the ALL set.",
+      "value": "${local.privileges}"
+    }
   }
 }
diff --git a/snowflake-stream-grant-all/main.tf.json b/snowflake-stream-grant-all/main.tf.json
@@ -64,5 +64,11 @@
         "version": "\u003e= 0.20.0"
       }
     }
+  },
+  "output": {
+    "privileges": {
+      "description": "Privileges that make up the ALL set.",
+      "value": "${local.privileges}"
+    }
   }
 }
diff --git a/snowflake-table-grant-all/README.md b/snowflake-table-grant-all/README.md
@@ -26,6 +26,8 @@
 
 ## Outputs
 
-No output.
+| Name | Description |
+|------|-------------|
+| privileges | Privileges that make up the ALL set. |
 
 <!-- END -->
diff --git a/snowflake-table-grant-all/main.tf.json b/snowflake-table-grant-all/main.tf.json
@@ -75,5 +75,11 @@
         "version": "\u003e= 0.20.0"
       }
     }
+  },
+  "output": {
+    "privileges": {
+      "description": "Privileges that make up the ALL set.",
+      "value": "${local.privileges}"
+    }
   }
 }
diff --git a/snowflake-view-grant-all/README.md b/snowflake-view-grant-all/README.md
@@ -26,6 +26,8 @@
 
 ## Outputs
 
-No output.
+| Name | Description |
+|------|-------------|
+| privileges | Privileges that make up the ALL set. |
 
 <!-- END -->
diff --git a/snowflake-view-grant-all/main.tf.json b/snowflake-view-grant-all/main.tf.json
@@ -70,5 +70,11 @@
         "version": "\u003e= 0.20.0"
       }
     }
+  },
+  "output": {
+    "privileges": {
+      "description": "Privileges that make up the ALL set.",
+      "value": "${local.privileges}"
+    }
   }
 }
diff --git a/snowflake-warehouse-grant-all/README.md b/snowflake-warehouse-grant-all/README.md
@@ -22,6 +22,8 @@
 
 ## Outputs
 
-No output.
+| Name | Description |
+|------|-------------|
+| privileges | Privileges that make up the ALL set. |
 
 <!-- END -->
diff --git a/snowflake-warehouse-grant-all/main.tf.json b/snowflake-warehouse-grant-all/main.tf.json
@@ -49,5 +49,11 @@
         "version": "\u003e= 0.20.0"
       }
     }
+  },
+  "output": {
+    "privileges": {
+      "description": "Privileges that make up the ALL set.",
+      "value": "${local.privileges}"
+    }
   }
 }

Original file line number	Diff line number	Diff line change
`@@ -47,5 +47,11 @@`
`47`	`47`	`"version": "\u003e= 0.20.0"`
`48`	`48`	`}`
`49`	`49`	`}`
	`50`	`+ },`
	`51`	`+ "output": {`
	`52`	`+ "privileges": {`
	`53`	`+ "description": "Privileges that make up the ALL set.",`
	`54`	`+ "value": "${local.privileges}"`
	`55`	`+ }`
`50`	`56`	`}`
`51`	`57`	`}`
Original file line number	Diff line number	Diff line change
`@@ -57,5 +57,11 @@`
`57`	`57`	`"version": "\u003e= 0.20.0"`
`58`	`58`	`}`
`59`	`59`	`}`
	`60`	`+ },`
	`61`	`+ "output": {`
	`62`	`+ "privileges": {`
	`63`	`+ "description": "Privileges that make up the ALL set.",`
	`64`	`+ "value": "${local.privileges}"`
	`65`	`+ }`
`60`	`66`	`}`
`61`	`67`	`}`
Original file line number	Diff line number	Diff line change
`@@ -70,5 +70,11 @@`
`70`	`70`	`"version": "\u003e= 0.20.0"`
`71`	`71`	`}`
`72`	`72`	`}`
	`73`	`+ },`
	`74`	`+ "output": {`
	`75`	`+ "privileges": {`
	`76`	`+ "description": "Privileges that make up the ALL set.",`
	`77`	`+ "value": "${local.privileges}"`
	`78`	`+ }`
`73`	`79`	`}`
`74`	`80`	`}`
Original file line number	Diff line number	Diff line change
`@@ -64,5 +64,11 @@`
`64`	`64`	`"version": "\u003e= 0.20.0"`
`65`	`65`	`}`
`66`	`66`	`}`
	`67`	`+ },`
	`68`	`+ "output": {`
	`69`	`+ "privileges": {`
	`70`	`+ "description": "Privileges that make up the ALL set.",`
	`71`	`+ "value": "${local.privileges}"`
	`72`	`+ }`
`67`	`73`	`}`
`68`	`74`	`}`
Original file line number	Diff line number	Diff line change
`@@ -82,5 +82,11 @@`
`82`	`82`	`"version": "\u003e= 0.20.0"`
`83`	`83`	`}`
`84`	`84`	`}`
	`85`	`+ },`
	`86`	`+ "output": {`
	`87`	`+ "privileges": {`
	`88`	`+ "description": "Privileges that make up the ALL set.",`
	`89`	`+ "value": "${local.privileges}"`
	`90`	`+ }`
`85`	`91`	`}`
`86`	`92`	`}`
Original file line number	Diff line number	Diff line change
`@@ -46,5 +46,11 @@`
`46`	`46`	`"version": "\u003e= 0.20.0"`
`47`	`47`	`}`
`48`	`48`	`}`
	`49`	`+ },`
	`50`	`+ "output": {`
	`51`	`+ "privileges": {`
	`52`	`+ "description": "Privileges that make up the ALL set.",`
	`53`	`+ "value": "${local.privileges}"`
	`54`	`+ }`
`49`	`55`	`}`
`50`	`56`	`}`
Original file line number	Diff line number	Diff line change
`@@ -81,5 +81,11 @@`
`81`	`81`	`"version": "\u003e= 0.20.0"`
`82`	`82`	`}`
`83`	`83`	`}`
	`84`	`+ },`
	`85`	`+ "output": {`
	`86`	`+ "privileges": {`
	`87`	`+ "description": "Privileges that make up the ALL set.",`
	`88`	`+ "value": "${local.privileges}"`
	`89`	`+ }`
`84`	`90`	`}`
`85`	`91`	`}`
Original file line number	Diff line number	Diff line change
`@@ -72,5 +72,11 @@`
`72`	`72`	`"version": "\u003e= 0.20.0"`
`73`	`73`	`}`
`74`	`74`	`}`
	`75`	`+ },`
	`76`	`+ "output": {`
	`77`	`+ "privileges": {`
	`78`	`+ "description": "Privileges that make up the ALL set.",`
	`79`	`+ "value": "${local.privileges}"`
	`80`	`+ }`
`75`	`81`	`}`
`76`	`82`	`}`
Original file line number	Diff line number	Diff line change
`@@ -75,5 +75,11 @@`
`75`	`75`	`"version": "\u003e= 0.20.0"`
`76`	`76`	`}`
`77`	`77`	`}`
	`78`	`+ },`
	`79`	`+ "output": {`
	`80`	`+ "privileges": {`
	`81`	`+ "description": "Privileges that make up the ALL set.",`
	`82`	`+ "value": "${local.privileges}"`
	`83`	`+ }`
`78`	`84`	`}`
`79`	`85`	`}`
Original file line number	Diff line number	Diff line change
`@@ -49,5 +49,11 @@`
`49`	`49`	`"version": "\u003e= 0.20.0"`
`50`	`50`	`}`
`51`	`51`	`}`
	`52`	`+ },`
	`53`	`+ "output": {`
	`54`	`+ "privileges": {`
	`55`	`+ "description": "Privileges that make up the ALL set.",`
	`56`	`+ "value": "${local.privileges}"`
	`57`	`+ }`
`52`	`58`	`}`
`53`	`59`	`}`