add cluster name validation to infractl

cmd/infractl/cluster/create/command.go

@@ -118,7 +118,12 @@ func run(ctx context.Context, conn *grpc.ClientConn, cmd *cobra.Command, args []
 	displayUserNotes(cmd, args, &req)
 
 	if len(args) > 1 {
-		req.Parameters["name"] = args[1]
+		name := args[1]
+		err := validateName(name)
+		if err != nil {
+			return nil, err
+		}
+		req.Parameters["name"] = name
 	} else {
 		name, err := determineName(ctx, conn, args[0])
 		if err != nil {
@@ -146,6 +151,25 @@ func run(ctx context.Context, conn *grpc.ClientConn, cmd *cobra.Command, args []
 	return prettyResourceByID(*clusterID), nil
 }
 
+func validateName(name string) error {
+	if len(name) < 3 {
+		return errors.New("cluster name too short")
+	}
+	if len(name) > 28 {
+		return errors.New("cluster name too long")
+	}
+
+	match, err := regexp.MatchString(`^(?:[a-z](?:[-a-z0-9]{0,28}[a-z0-9])?)$`, name)
+	if err != nil {
+		return err
+	}
+	if !match {
+		return errors.New("The name does not match its requirements. Only lowercase letters, numbers, and '-' allowed, must start with a letter and end with a letter or number.")
+	}
+
+	return nil
+}
+
 func determineWorkingEnvironment() {
 	workingEnvironment.gitTopLevel = ""
 	workingEnvironment.tag = ""

cmd/infractl/cluster/create/create.bats

@@ -123,6 +123,16 @@ setup() {
   assert_output --partial "parameter \"main-image\" was not provided"
 }
 
+@test "provided name is too short" {
+  run infractl create test-qa-demo ab
+  assert_failure
+  assert_output --partial "Error: cluster name too short"
+}
+
+# @test "provided name is too long"
+
+# @test "provided name does not match regex"
+
 infractl() {
   "$ROOT"/bin/infractl -e localhost:8443 -k "$@"
 }