Skip to content

Commit 7dcaf03

Browse files
wyardleywyardley
committed
fix: update resource names to snake_case
- Update resource names to snake_case - Add `moved` blocks for resources and consolidate into `moved.tf`
1 parent 843c56c commit 7dcaf03

File tree

40 files changed

+550
-150
lines changed

40 files changed

+550
-150
lines changed

‎autogen/main/dns.tf.tmpl‎

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
Manage kube-dns configmaps
2121
*****************************************/
2222

23-
resource "kubernetes_config_map_v1_data" "kube-dns" {
23+
resource "kubernetes_config_map_v1_data" "kube_dns" {
2424
count = local.custom_kube_dns_config && !local.upstream_nameservers_config ? 1 : 0
2525

2626
metadata {
@@ -44,7 +44,7 @@ EOF
4444
]
4545
}
4646

47-
resource "kubernetes_config_map_v1_data" "kube-dns-upstream-nameservers" {
47+
resource "kubernetes_config_map_v1_data" "kube_dns_upstream_nameservers" {
4848
count = !local.custom_kube_dns_config && local.upstream_nameservers_config ? 1 : 0
4949

5050
metadata {
@@ -68,13 +68,7 @@ EOF
6868
]
6969
}
7070

71-
# Fix the name typo in the previous ConfigMap creation call
72-
moved {
73-
from = kubernetes_config_map_v1_data.kube-dns-upstream-namservers
74-
to = kubernetes_config_map_v1_data.kube-dns-upstream-nameservers
75-
}
76-
77-
resource "kubernetes_config_map_v1_data" "kube-dns-upstream-nameservers-and-stub-domains" {
71+
resource "kubernetes_config_map_v1_data" "kube_dns_upstream_nameservers_and_stub_domains" {
7872
count = local.custom_kube_dns_config && local.upstream_nameservers_config ? 1 : 0
7973

8074
metadata {

‎autogen/main/masq.tf.tmpl‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
/******************************************
2020
Create ip-masq-agent confimap
2121
*****************************************/
22-
resource "kubernetes_config_map" "ip-masq-agent" {
22+
resource "kubernetes_config_map" "ip_masq_agent" {
2323
count = var.configure_ip_masq ? 1 : 0
2424

2525
metadata {

‎autogen/main/moved.tf.tmpl‎

Lines changed: 46 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,46 @@
1+
# Fix the name typo in the previous ConfigMap creation call
2+
moved {
3+
from = kubernetes_config_map_v1_data.kube-dns-upstream-namservers
4+
to = kubernetes_config_map_v1_data.kube-dns-upstream-nameservers
5+
}
6+
7+
# Updates for kebab to snake case, to match best practices and Google style.
8+
moved {
9+
from = kubernetes_config_map_v1_data.kube-dns
10+
to = kubernetes_config_map_v1_data.kube_dns
11+
}
12+
13+
moved {
14+
from = kubernetes_config_map_v1_data.kube-dns-upstream-nameservers
15+
to = kubernetes_config_map_v1_data.kube_dns_upstream_nameservers
16+
}
17+
18+
moved {
19+
from = kubernetes_config_map_v1_data.kube-dns-upstream-nameservers-and-stub-domains
20+
to = kubernetes_config_map_v1_data.kube_dns_upstream_nameservers_and_stub_domains
21+
}
22+
23+
moved {
24+
from = google_project_iam_member.cluster_service_account-nodeService_account
25+
to = google_project_iam_member.cluster_service_account_node_service_account
26+
}
27+
28+
moved {
29+
from = google_project_iam_member.cluster_service_account-metric_writer
30+
to = google_project_iam_member.cluster_service_account_metric_writer
31+
}
32+
33+
moved {
34+
from = google_project_iam_member.cluster_service_account-resourceMetadata-writer
35+
to = google_project_iam_member.cluster_service_account_resource_metadata_writer
36+
}
37+
38+
moved {
39+
from = google_project_iam_member.cluster_service_account-gcr
40+
to = google_project_iam_member.cluster_service_account_gcr
41+
}
42+
43+
moved {
44+
from = google_project_iam_member.cluster_service_account-artifact-registry
45+
to = google_project_iam_member.cluster_service_account_artifact_registry
46+
}

‎autogen/main/sa.tf.tmpl‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -46,35 +46,35 @@ resource "google_service_account" "cluster_service_account" {
4646
display_name = "Terraform-managed service account for cluster ${var.name}"
4747
}
4848

49-
resource "google_project_iam_member" "cluster_service_account-nodeService_account" {
49+
resource "google_project_iam_member" "cluster_service_account_node_service_account" {
5050
count = var.create_service_account ? 1 : 0
5151
project = google_service_account.cluster_service_account[0].project
5252
role = "roles/container.defaultNodeServiceAccount"
5353
member = google_service_account.cluster_service_account[0].member
5454
}
5555

56-
resource "google_project_iam_member" "cluster_service_account-metric_writer" {
56+
resource "google_project_iam_member" "cluster_service_account_metric_writer" {
5757
count = var.create_service_account ? 1 : 0
5858
project = google_service_account.cluster_service_account[0].project
5959
role = "roles/monitoring.metricWriter"
6060
member = google_service_account.cluster_service_account[0].member
6161
}
6262

63-
resource "google_project_iam_member" "cluster_service_account-resourceMetadata-writer" {
63+
resource "google_project_iam_member" "cluster_service_account_resource_metadata_writer" {
6464
count = var.create_service_account ? 1 : 0
6565
project = google_service_account.cluster_service_account[0].project
6666
role = "roles/stackdriver.resourceMetadata.writer"
6767
member = google_service_account.cluster_service_account[0].member
6868
}
6969

70-
resource "google_project_iam_member" "cluster_service_account-gcr" {
70+
resource "google_project_iam_member" "cluster_service_account_gcr" {
7171
for_each = var.create_service_account && var.grant_registry_access ? toset(local.registry_projects_list) : []
7272
project = each.key
7373
role = "roles/storage.objectViewer"
7474
member = "serviceAccount:${google_service_account.cluster_service_account[0].email}"
7575
}
7676

77-
resource "google_project_iam_member" "cluster_service_account-artifact-registry" {
77+
resource "google_project_iam_member" "cluster_service_account_artifact_registry" {
7878
for_each = var.create_service_account && var.grant_registry_access ? toset(local.registry_projects_list) : []
7979
project = each.key
8080
role = "roles/artifactregistry.reader"

‎dns.tf‎

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
Manage kube-dns configmaps
2121
*****************************************/
2222

23-
resource "kubernetes_config_map_v1_data" "kube-dns" {
23+
resource "kubernetes_config_map_v1_data" "kube_dns" {
2424
count = local.custom_kube_dns_config && !local.upstream_nameservers_config ? 1 : 0
2525

2626
metadata {
@@ -42,7 +42,7 @@ EOF
4242
]
4343
}
4444

45-
resource "kubernetes_config_map_v1_data" "kube-dns-upstream-nameservers" {
45+
resource "kubernetes_config_map_v1_data" "kube_dns_upstream_nameservers" {
4646
count = !local.custom_kube_dns_config && local.upstream_nameservers_config ? 1 : 0
4747

4848
metadata {
@@ -64,13 +64,7 @@ EOF
6464
]
6565
}
6666

67-
# Fix the name typo in the previous ConfigMap creation call
68-
moved {
69-
from = kubernetes_config_map_v1_data.kube-dns-upstream-namservers
70-
to = kubernetes_config_map_v1_data.kube-dns-upstream-nameservers
71-
}
72-
73-
resource "kubernetes_config_map_v1_data" "kube-dns-upstream-nameservers-and-stub-domains" {
67+
resource "kubernetes_config_map_v1_data" "kube_dns_upstream_nameservers_and_stub_domains" {
7468
count = local.custom_kube_dns_config && local.upstream_nameservers_config ? 1 : 0
7569

7670
metadata {

‎masq.tf‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
/******************************************
2020
Create ip-masq-agent confimap
2121
*****************************************/
22-
resource "kubernetes_config_map" "ip-masq-agent" {
22+
resource "kubernetes_config_map" "ip_masq_agent" {
2323
count = var.configure_ip_masq ? 1 : 0
2424

2525
metadata {

‎modules/beta-autopilot-private-cluster/dns.tf‎

Lines changed: 3 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -20,7 +20,7 @@
2020
Manage kube-dns configmaps
2121
*****************************************/
2222

23-
resource "kubernetes_config_map_v1_data" "kube-dns" {
23+
resource "kubernetes_config_map_v1_data" "kube_dns" {
2424
count = local.custom_kube_dns_config && !local.upstream_nameservers_config ? 1 : 0
2525

2626
metadata {
@@ -41,7 +41,7 @@ EOF
4141
]
4242
}
4343

44-
resource "kubernetes_config_map_v1_data" "kube-dns-upstream-nameservers" {
44+
resource "kubernetes_config_map_v1_data" "kube_dns_upstream_nameservers" {
4545
count = !local.custom_kube_dns_config && local.upstream_nameservers_config ? 1 : 0
4646

4747
metadata {
@@ -62,13 +62,7 @@ EOF
6262
]
6363
}
6464

65-
# Fix the name typo in the previous ConfigMap creation call
66-
moved {
67-
from = kubernetes_config_map_v1_data.kube-dns-upstream-namservers
68-
to = kubernetes_config_map_v1_data.kube-dns-upstream-nameservers
69-
}
70-
71-
resource "kubernetes_config_map_v1_data" "kube-dns-upstream-nameservers-and-stub-domains" {
65+
resource "kubernetes_config_map_v1_data" "kube_dns_upstream_nameservers_and_stub_domains" {
7266
count = local.custom_kube_dns_config && local.upstream_nameservers_config ? 1 : 0
7367

7468
metadata {

‎modules/beta-autopilot-private-cluster/masq.tf‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -19,7 +19,7 @@
1919
/******************************************
2020
Create ip-masq-agent confimap
2121
*****************************************/
22-
resource "kubernetes_config_map" "ip-masq-agent" {
22+
resource "kubernetes_config_map" "ip_masq_agent" {
2323
count = var.configure_ip_masq ? 1 : 0
2424

2525
metadata {

‎modules/beta-autopilot-private-cluster/moved.tf‎

Lines changed: 46 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,46 @@
1+
# Fix the name typo in the previous ConfigMap creation call
2+
moved {
3+
from = kubernetes_config_map_v1_data.kube-dns-upstream-namservers
4+
to = kubernetes_config_map_v1_data.kube-dns-upstream-nameservers
5+
}
6+
7+
# Updates for kebab to snake case, to match best practices and Google style.
8+
moved {
9+
from = kubernetes_config_map_v1_data.kube-dns
10+
to = kubernetes_config_map_v1_data.kube_dns
11+
}
12+
13+
moved {
14+
from = kubernetes_config_map_v1_data.kube-dns-upstream-nameservers
15+
to = kubernetes_config_map_v1_data.kube_dns_upstream_nameservers
16+
}
17+
18+
moved {
19+
from = kubernetes_config_map_v1_data.kube-dns-upstream-nameservers-and-stub-domains
20+
to = kubernetes_config_map_v1_data.kube_dns_upstream_nameservers_and_stub_domains
21+
}
22+
23+
moved {
24+
from = google_project_iam_member.cluster_service_account-nodeService_account
25+
to = google_project_iam_member.cluster_service_account_node_service_account
26+
}
27+
28+
moved {
29+
from = google_project_iam_member.cluster_service_account-metric_writer
30+
to = google_project_iam_member.cluster_service_account_metric_writer
31+
}
32+
33+
moved {
34+
from = google_project_iam_member.cluster_service_account-resourceMetadata-writer
35+
to = google_project_iam_member.cluster_service_account_resource_metadata_writer
36+
}
37+
38+
moved {
39+
from = google_project_iam_member.cluster_service_account-gcr
40+
to = google_project_iam_member.cluster_service_account_gcr
41+
}
42+
43+
moved {
44+
from = google_project_iam_member.cluster_service_account-artifact-registry
45+
to = google_project_iam_member.cluster_service_account_artifact_registry
46+
}

‎modules/beta-autopilot-private-cluster/sa.tf‎

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -46,35 +46,35 @@ resource "google_service_account" "cluster_service_account" {
4646
display_name = "Terraform-managed service account for cluster ${var.name}"
4747
}
4848

49-
resource "google_project_iam_member" "cluster_service_account-nodeService_account" {
49+
resource "google_project_iam_member" "cluster_service_account_node_service_account" {
5050
count = var.create_service_account ? 1 : 0
5151
project = google_service_account.cluster_service_account[0].project
5252
role = "roles/container.defaultNodeServiceAccount"
5353
member = google_service_account.cluster_service_account[0].member
5454
}
5555

56-
resource "google_project_iam_member" "cluster_service_account-metric_writer" {
56+
resource "google_project_iam_member" "cluster_service_account_metric_writer" {
5757
count = var.create_service_account ? 1 : 0
5858
project = google_service_account.cluster_service_account[0].project
5959
role = "roles/monitoring.metricWriter"
6060
member = google_service_account.cluster_service_account[0].member
6161
}
6262

63-
resource "google_project_iam_member" "cluster_service_account-resourceMetadata-writer" {
63+
resource "google_project_iam_member" "cluster_service_account_resource_metadata_writer" {
6464
count = var.create_service_account ? 1 : 0
6565
project = google_service_account.cluster_service_account[0].project
6666
role = "roles/stackdriver.resourceMetadata.writer"
6767
member = google_service_account.cluster_service_account[0].member
6868
}
6969

70-
resource "google_project_iam_member" "cluster_service_account-gcr" {
70+
resource "google_project_iam_member" "cluster_service_account_gcr" {
7171
for_each = var.create_service_account && var.grant_registry_access ? toset(local.registry_projects_list) : []
7272
project = each.key
7373
role = "roles/storage.objectViewer"
7474
member = "serviceAccount:${google_service_account.cluster_service_account[0].email}"
7575
}
7676

77-
resource "google_project_iam_member" "cluster_service_account-artifact-registry" {
77+
resource "google_project_iam_member" "cluster_service_account_artifact_registry" {
7878
for_each = var.create_service_account && var.grant_registry_access ? toset(local.registry_projects_list) : []
7979
project = each.key
8080
role = "roles/artifactregistry.reader"

0 commit comments

Comments
 (0)