fix: Too long host names for docker machines (#549)

npalm · web-flow · commit 2fc8e770337e · 2022-10-09T19:03:18.000+02:00
diff --git a/tags.tf b/tags.tf
@@ -24,7 +24,7 @@ locals {
     local.tags,
     var.runner_tags,
     # overwrites the `Name` key from `local.tags`
-    var.overrides["name_docker_machine_runners"] == "" ? { Name = format("%s-docker-machine", var.environment) } : { Name = var.overrides["name_docker_machine_runners"] },
+    var.overrides["name_docker_machine_runners"] == "" ? { Name = substr(format("%s", var.environment), 0, 16) } : { Name = var.overrides["name_docker_machine_runners"] },
   )
 
   # remove the `Name` tag if docker+machine adds one to avoid a failure due to a duplicate `Name` tag

Original file line number	Diff line number	Diff line change
`@@ -24,7 +24,7 @@ locals {`
`24`	`24`	`local.tags,`
`25`	`25`	`var.runner_tags,`
`26`	`26`	# overwrites the `Name` key from `local.tags`
`27`		`- var.overrides["name_docker_machine_runners"] == "" ? { Name = format("%s-docker-machine", var.environment) } : { Name = var.overrides["name_docker_machine_runners"] },`
	`27`	`+ var.overrides["name_docker_machine_runners"] == "" ? { Name = substr(format("%s", var.environment), 0, 16) } : { Name = var.overrides["name_docker_machine_runners"] },`
`28`	`28`	`)`
`29`	`29`
`30`	`30`	# remove the `Name` tag if docker+machine adds one to avoid a failure due to a duplicate `Name` tag