reordered variables to make more readable and added documentation

external-network/main.tf

+# is created as a datasource this module and called in root module
+variable "public_network_name" {type = "string"}
+
+variable "name" {default = "dmz"}
 variable "admin_state_up" {}
-variable "public_network_name" {}
 variable "enable_dhcp" {}
-variable "name" {default = "dmz"}
-variable "public_network_name" {type = "string"}
+
+
 
 data "openstack_networking_network_v2" "public-network" {name = var.public_network_name}
 
@@ -21,7 +24,7 @@ resource "openstack_networking_subnet_v2" "external_subnet" {
   enable_dhcp     = var.enable_dhcp
 }
 
-# defines the router borderrouter using floating ip defined in datasources.tf to create the external network id
+# defines the router dmzrouter using floating ip defined in datasource above to create the external network id
 resource "openstack_networking_router_v2" "router" {
   name                = "${var.name}router"
   admin_state_up      = var.admin_state_up