Cluster command nit fixes (#2306)

Author: vishalbollu

pkg/lib/configreader/errors.go

@@ -35,6 +35,7 @@ const (
 	ErrLeadingWhitespace             = "configreader.leading_whitespace"
 	ErrTrailingWhitespace            = "configreader.trailing_whitespace"
 	ErrAlphaNumericDashUnderscore    = "configreader.alpha_numeric_dash_underscore"
+	ErrAlphaNumericDash              = "configreader.alpha_numeric_dash"
 	ErrAlphaNumericDotUnderscore     = "configreader.alpha_numeric_dot_underscore"
 	ErrAlphaNumericDashDotUnderscore = "configreader.alpha_numeric_dash_dot_underscore"
 	ErrInvalidAWSTag                 = "configreader.invalid_aws_tag"
@@ -140,6 +141,13 @@ func ErrorAlphaNumericDashUnderscore(provided string) error {
 	})
 }
 
+func ErrorAlphaNumericDash(provided string) error {
+	return errors.WithStack(&errors.Error{
+		Kind:    ErrAlphaNumericDash,
+		Message: fmt.Sprintf("%s must contain only letters, numbers, and dashes", s.UserStr(provided)),
+	})
+}
+
 func ErrorAlphaNumericDotUnderscore(provided string) error {
 	return errors.WithStack(&errors.Error{
 		Kind:    ErrAlphaNumericDotUnderscore,

pkg/lib/configreader/string.go

@@ -54,6 +54,7 @@ type StringValidation struct {
 	AlphaNumericDashUnderscoreOrEmpty    bool
 	AlphaNumericDashUnderscore           bool
 	AlphaNumericDotUnderscore            bool
+	AlphaNumericDash                     bool
 	AWSTag                               bool
 	DNS1035                              bool
 	DNS1123                              bool
@@ -334,6 +335,12 @@ func ValidateStringVal(val string, v *StringValidation) error {
 		}
 	}
 
+	if v.AlphaNumericDash {
+		if !regex.IsAlphaNumericDash(val) {
+			return ErrorAlphaNumericDash(val)
+		}
+	}
+
 	if v.AlphaNumericDashUnderscoreOrEmpty {
 		if !regex.IsAlphaNumericDashUnderscore(val) && val != "" {
 			return ErrorAlphaNumericDashUnderscore(val)

pkg/lib/regex/regex.go

@@ -67,6 +67,12 @@ func IsAlphaNumericDotUnderscore(s string) bool {
 	return _alphaNumericDotUnderscoreRegex.MatchString(s)
 }
 
+var _alphaNumericDashRegex = regexp.MustCompile(`^[a-zA-Z0-9\-]+$`)
+
+func IsAlphaNumericDash(s string) bool {
+	return _alphaNumericDashRegex.MatchString(s)
+}
+
 // used the evaluated form of
 // https://github.com/docker/distribution/blob/3150937b9f2b1b5b096b2634d0e7c44d4a0f89fb/reference/regexp.go#L68-L70
 var _dockerValidImage = regexp.MustCompile(

pkg/types/clusterconfig/cluster_config.go

@@ -718,9 +718,9 @@ var nodeGroupsFieldValidation *cr.StructValidation = &cr.StructValidation{
 		{
 			StructField: "Name",
 			StringValidation: &cr.StringValidation{
-				Required:                   true,
-				AlphaNumericDashUnderscore: true,
-				MaxLength:                  _maxNodeGroupLength,
+				Required:         true,
+				AlphaNumericDash: true,
+				MaxLength:        _maxNodeGroupLength,
 			},
 		},
 		{

pkg/types/clusterstate/clusterstate.go

@@ -142,9 +142,13 @@ func GetClusterState(stacks ClusterStacks) State {
 		}
 
 		if slices.HasString([]string{
+			cloudformation.StackStatusCreateInProgress,
 			cloudformation.StackStatusCreateComplete,
+			cloudformation.StackStatusUpdateInProgress,
 			cloudformation.StackStatusUpdateComplete,
+			cloudformation.StackStatusRollbackInProgress,
 			cloudformation.StackStatusRollbackComplete,
+			cloudformation.StackStatusUpdateRollbackInProgress,
 			cloudformation.StackStatusUpdateRollbackComplete,
 		}, controlPlaneStatus) {
 			return StateClusterExists