made minor changes

Azure · Oct 3, 2023 · 757c964 · 757c964
1 parent e0c258c
commit 757c964
Show file tree

Hide file tree

Showing 3 changed files with 9 additions and 7 deletions.
diff --git a/quickstart/201-azfw-with-ipgroups/main.tf b/quickstart/201-azfw-with-ipgroups/main.tf
@@ -261,7 +261,7 @@ resource "azurerm_route_table" "rt" {
     name                   = "azfwDefaultRoute"
     address_prefix         = "0.0.0.0/0"
     next_hop_type          = "VirtualAppliance"
-    next_hop_in_ip_address = azurerm_firewall.fw.ip_configuration.0.private_ip_address
+    next_hop_in_ip_address = azurerm_firewall.fw.ip_configuration[0].private_ip_address
   }
 }
 

diff --git a/quickstart/201-azfw-with-ipgroups/readme.md b/quickstart/201-azfw-with-ipgroups/readme.md
@@ -21,16 +21,16 @@ This template deploys an [Azure Firewall](https://registry.terraform.io/provider
 - [azurerm_storage_account](https://registry.terraform.io/providers/hashicorp/azurerm/latest/docs/resources/storage_account)
 - [random_password](https://registry.terraform.io/providers/hashicorp/random/latest/docs/resources/password)
 - [random_pet](https://registry.terraform.io/providers/hashicorp/random/latest/docs/resources/pet)
-- [random_string](https://registry.terraform.io/providers/hashicorp/random/latest/docs/resources/string)}
+- [random_string](https://registry.terraform.io/providers/hashicorp/random/latest/docs/resources/string)
 
 ## Variables
 
 | Name | Description | Default value |
 |-|-|-|
-| `resource_group_location`    | location for your resources                       | eastus  |
-| `firewall_sku_tier`          | Sku size for your Firewall and Firewall Policy    | Premium |
-| `resource_group_name_prefix` | Prefix for your resource group                    | rg      |
-| `virtual_machine_size`       | Sku size for your jump and workload vms           | Standard_D2_v3 |
-| `admin_username`             | admin username for the jump and workload vms      | azureuser      |
+| `resource_group_location`    | The location of the resource group.                 | eastus  |
+| `firewall_sku_tier`          | the sku size for your Firewall and Firewall Policy.    | Possible values: Standard, Premium |
+| `resource_group_name_prefix` | The prefix of the resource group name that's combined with a random ID so that name is unique in your Azure subscription.                  | rg      |
+| `virtual_machine_size`       | The sku size for your jump and workload VMs.           | Standard_D2_v3 |
+| `admin_username`             | The admin username for the jump and workload VMs.     | azureuser      |
 
 ## Example
diff --git a/quickstart/201-azfw-with-ipgroups/variables.tf b/quickstart/201-azfw-with-ipgroups/variables.tf
@@ -27,5 +27,7 @@ variable "virtual_machine_size" {
 }
 
 variable "admin_username" {
+  type = string
+  description = "value of the admin username."
   default = "azureuser"
 }