Review fixes

Author: YpNo

modules/vpc/variables.tf

@@ -93,17 +93,17 @@ variable "network_profile" {
 variable "bgp_always_compare_med" {
   type        = bool
   description = "If set to true, the Cloud Router will use MED values from the peer even if the AS paths differ. Default is false."
-  default     = null
+  default     = false
 }
 
 variable "bgp_best_path_selection_mode" {
   type        = string
-  description = "Specifies the BGP best path selection mode. Valid values are `STANDARD`, `EARLY_EXIT`, `MULTI_EXIT_DISC`. Default is `STANDARD`."
-  default     = null
+  description = "Specifies the BGP best path selection mode. Valid values are `STANDARD` or `LEGACY`. Default is `STANDARD`."
+  default     = "STANDARD"
 }
 
 variable "bgp_inter_region_cost" {
   type        = string
-  description = "Specifies the BGP inter-region cost mode. Valid values are `DEFAULT`, `LOW`. Default is `DEFAULT`."
-  default     = null
+  description = "Specifies the BGP inter-region cost mode. Valid values are `DEFAULT` or `ADD_COST_TO_MED`. Default is `DEFAULT`."
+  default     = "DEFAULT"
 }

variables.tf

@@ -215,17 +215,17 @@ variable "network_profile" {
 variable "bgp_always_compare_med" {
   type        = bool
   description = "If set to true, the Cloud Router will use MED values from the peer even if the AS paths differ. Default is false."
-  default     = null
+  default     = false
 }
 
 variable "bgp_best_path_selection_mode" {
   type        = string
-  description = "Specifies the BGP best path selection mode. Valid values are `STANDARD`, `EARLY_EXIT`, `MULTI_EXIT_DISC`. Default is `STANDARD`."
-  default     = null
+  description = "Specifies the BGP best path selection mode. Valid values are `STANDARD` or `LEGACY`. Default is `STANDARD`."
+  default     = "STANDARD"
 }
 
 variable "bgp_inter_region_cost" {
   type        = string
-  description = "Specifies the BGP inter-region cost mode. Valid values are `DEFAULT`, `LOW`. Default is `DEFAULT`."
-  default     = null
+  description = "Specifies the BGP inter-region cost mode. Valid values are `DEFAULT` or `ADD_COST_TO_MED`. Default is `DEFAULT`."
+  default     = "DEFAULT"
 }