Skip to content

Commit bf9a6b6

Browse files
committed
Remove default text for description and add NCC Hub Admin for test setup
1 parent 51db90a commit bf9a6b6

File tree

2 files changed

+5
-4
lines changed

2 files changed

+5
-4
lines changed

modules/network-connectivity-center/variables.tf

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,7 @@ variable "ncc_hub_name" {
2727
variable "ncc_hub_description" {
2828
description = "The description of the NCC Hub"
2929
type = string
30-
default = "Managed by Terraform"
30+
default = ""
3131
}
3232
variable "ncc_hub_labels" {
3333
description = "These labels will be added the NCC hub"
@@ -46,7 +46,7 @@ variable "vpc_spokes" {
4646
type = map(object({
4747
uri = string
4848
exclude_export_ranges = optional(set(string))
49-
description = optional(string, "Managed by Terraform")
49+
description = optional(string)
5050
labels = optional(map(string))
5151
}))
5252
default = {}
@@ -59,7 +59,7 @@ variable "hybrid_spokes" {
5959
uris = set(string)
6060
site_to_site_data_transfer = optional(bool, false)
6161
type = string
62-
description = optional(string, "Managed by Terraform")
62+
description = optional(string)
6363
labels = optional(map(string))
6464
}))
6565
default = {}
@@ -74,7 +74,7 @@ variable "router_appliance_spokes" {
7474
}))
7575
location = string
7676
site_to_site_data_transfer = optional(bool, false)
77-
description = optional(string, "Managed by Terraform")
77+
description = optional(string)
7878
labels = optional(map(string))
7979
}))
8080
default = {}

test/setup/iam.tf

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -25,6 +25,7 @@ locals {
2525
"roles/resourcemanager.tagAdmin",
2626
"roles/iam.serviceAccountAdmin",
2727
"roles/compute.orgFirewallPolicyAdmin",
28+
"roles/networkconnectivity.hubAdmin",
2829
]
2930
}
3031

0 commit comments

Comments
 (0)