Merge pull request #32 from pjreed/limit-instance-label-length

LBGarber · web-flow · commit 0b6e44e1a840 · 2022-04-06T11:03:12.000-04:00
Limit instance label length to 64 characters
diff --git a/pkg/drivers/linode/linode.go b/pkg/drivers/linode/linode.go
@@ -310,6 +310,11 @@ func (d *Driver) SetConfigFromFlags(flags drivers.DriverOptions) error {
 		d.InstanceLabel = d.GetMachineName()
 	}
 
+	if len(d.InstanceLabel) > 64 {
+		d.InstanceLabel = d.InstanceLabel[:64]
+		log.Warnf("The name for this machine exceeds the 64 character Linode label limit. Truncating to %s", d.InstanceLabel)
+	}
+
 	return nil
 }
 

Original file line number	Diff line number	Diff line change
`@@ -310,6 +310,11 @@ func (d *Driver) SetConfigFromFlags(flags drivers.DriverOptions) error {`
`310`	`310`	`d.InstanceLabel = d.GetMachineName()`
`311`	`311`	`}`
`312`	`312`
	`313`	`+ if len(d.InstanceLabel) > 64 {`
	`314`	`+ d.InstanceLabel = d.InstanceLabel[:64]`
	`315`	`+ log.Warnf("The name for this machine exceeds the 64 character Linode label limit. Truncating to %s", d.InstanceLabel)`
	`316`	`+ }`
	`317`	`+`
`313`	`318`	`return nil`
`314`	`319`	`}`
`315`	`320`