Skip to content

Commit e1de284

Browse files
authored
Merge pull request #101 from sbellone/fix/normalize-terraform-name
fix: dots are not allowed in Terraform names
2 parents 54b4533 + 3736596 commit e1de284

File tree

2 files changed

+11
-1
lines changed

2 files changed

+11
-1
lines changed

pkg/tfkschema/name_mapper.go

+2-1
Original file line numberDiff line numberDiff line change
@@ -100,8 +100,9 @@ func NormalizeTerraformName(s string, toSingular bool, path string) string {
100100
}
101101
s = strcase.ToSnake(s)
102102

103-
// colons are not allowed by Terraform
103+
// colons and dots are not allowed by Terraform
104104
s = strings.ReplaceAll(s, ":", "_")
105+
s = strings.ReplaceAll(s, ".", "_")
105106

106107
return s
107108
}

pkg/tfkschema/name_mapper_test.go

+9
Original file line numberDiff line numberDiff line change
@@ -252,6 +252,15 @@ func Test_NormalizeTerraformName(t *testing.T) {
252252
},
253253
"metallb_system_speaker",
254254
},
255+
{
256+
"custom.metrics.k8s.io",
257+
args{
258+
"custom.metrics.k8s.io",
259+
false,
260+
"",
261+
},
262+
"custom_metrics_k_8_s_io",
263+
},
255264
}
256265
for _, tt := range tests {
257266
t.Run(tt.name, func(t *testing.T) {

0 commit comments

Comments
 (0)