Merge pull request #10 from domroutley/variablise

Variablise

main.tf

@@ -1,5 +1,5 @@
 resource "azurerm_resource_group" "web_rg" {
-  name     = "website-rg"
+  name     = var.resource_group_name
   location = var.location
 }
 
@@ -9,7 +9,7 @@ resource "azurerm_storage_account" "web_sa" {
   location                  = var.location
   account_kind              = "StorageV2"
   account_tier              = "Standard"
-  account_replication_type  = "GRS"
+  account_replication_type  = var.sa_replication_type
   allow_blob_public_access  = true
   enable_https_traffic_only = false
 

variables.tf

@@ -1,3 +1,11 @@
+variable "endpoint" {
+  description = "CDN endpoint name (XX.azureedge.net)"
+}
+
+variable "storage_account_name" {
+  description = "The name of the storage account, must be globaly unique."
+}
+
 variable "location" {
   default     = "UK South"
   description = "Location for the resource group and storage account."
@@ -8,10 +16,19 @@ variable "cdn_location" {
   description = "Location for the CDN profile and endpoint."
 }
 
-variable "endpoint" {
-  description = "CDN endpoint name (XX.azureedge.net)"
+variable "sa_replication_type" {
+  description = "The storage account replication type. Valid values are: LRS, ZRS, GRS, RA-GRS"
+  default     = "LRS"
+  validation {
+    condition = contains(
+      ["LRS", "ZRS", "GRS", "RA-GRS"],
+      var.sa_replication_type
+    )
+    error_message = "Variable sa_replication_type must be one of: LRS, ZRS, GRS, RA-GRS."
+  }
 }
 
-variable "storage_account_name" {
-  description = "The name of the storage account, must be globaly unique"
+variable "resource_group_name" {
+  description = "The name of the resource group to store all resources in."
+  default     = "website-rg"
 }