Closed Gowiem closed 8 months ago
/terratest
major
revisionThis PR does a good overhaul of the module: It updates to modern practices / syntax + uses new syntax added in TF 1.3 + adds some new, significant functionality + introduces a breaking change (changes var from string
=> bool
type). Considering that, I believe we should release this as 1.0
.
/terratest
/terratest
/terratest
/terratest
/terratest
/terratest
/terratest
/terratest
major
revisionThis PR does a good overhaul of the module: It updates to modern practices / syntax + uses new syntax added in TF 1.3 + adds some new, significant functionality + introduces a breaking change (changes var from
string
=>bool
type). Considering that, I believe we should release this as1.0
.what
var.vpn_connection_static_routes_only
var.vpc_id
being required as this should not be required with TGW VPN Connectionnull
providervpn_connection_customer_gateway_configuration
tosensitive = true
transit_gateway_attachment_id
as an outputwhy
var.vpn_connection_static_routes_only
- Types should be correct, so this was bugging me. Is this possibly a 0.11 holdout that we're finding in 2024? I didn't check the blame, but that'd be funny.optional
invar.transit_gateway_routes
)null
provider no longer needed - Likely a holdout from an old version ofterraform-null-label
(this module had some dust on it)references