Extend kepval to validate milestones

Author: wojtek-t

pkg/kepval/keps/proposals.go

@@ -75,7 +75,6 @@ type Proposal struct {
 	DisableSupported bool          `json:"disableSupported" yaml:"disable-supported"`
 	Metrics          []string      `json:"metrics" yaml:"metrics"`
 
-	Filename string `json:"-" yaml:"-"`
 	Error    error  `json:"-" yaml:"-"`
 	Contents string `json:"markdown" yaml:"-"`
 }

pkg/kepval/keps/validations/yaml.go

@@ -30,6 +30,7 @@ var (
 	reStatus      = regexp.MustCompile(strings.Join(statuses, "|"))
 	stages        = []string{"alpha", "beta", "stable"}
 	reStages      = regexp.MustCompile(strings.Join(stages, "|"))
+	reMilestone   = regexp.MustCompile("v1\\.[1-9][0-9]*")
 )
 
 func ValidateStructure(parsed map[interface{}]interface{}) error {
@@ -151,6 +152,47 @@ func ValidateStructure(parsed map[interface{}]interface{}) error {
 			case interface{}:
 				return util.NewValueMustBeListOfStrings(k, values)
 			}
+		case "latest-milestone":
+			switch v := value.(type) {
+			case []interface{}:
+				return util.NewValueMustBeString(k, v)
+			}
+			v, _ := value.(string)
+			if !reMilestone.Match([]byte(v)) {
+				return util.NewValueMustBeMilestone(k, v)
+			}
+		case "milestone":
+			switch v := value.(type) {
+			case map[interface{}]interface{}:
+				if err := validateMilestone(v); err != nil {
+					return err
+				}
+			case interface{}:
+				return util.NewValueMustBeStruct(k, v)
+			}
+		}
+	}
+	return nil
+}
+
+func validateMilestone(parsed map[interface{}]interface{}) error {
+	for key, value := range parsed {
+		// First off the key has to be a string. fact.
+		k, ok := key.(string)
+		if !ok {
+			return util.NewKeyMustBeString(k)
+		}
+
+		switch strings.ToLower(k) {
+		case "alpha", "beta", "stable":
+			switch v := value.(type) {
+			case []interface{}:
+				return util.NewValueMustBeString(k, v)
+			}
+			v, _ := value.(string)
+			if !reMilestone.Match([]byte(v)) {
+				return util.NewValueMustBeMilestone(k, v)
+			}
 		}
 	}
 	return nil

pkg/kepval/util/errors.go

@@ -81,6 +81,38 @@ func NewValueMustBeString(key string, value interface{}) error {
 	}
 }
 
+type ValueMustBeStruct struct {
+	key   string
+	value interface{}
+}
+
+func (v *ValueMustBeStruct) Error() string {
+	return fmt.Sprintf("%q must be a struct but it is a %T: %v", v.key, v.value, v.value)
+}
+
+func NewValueMustBeStruct(key string, value interface{}) error {
+	return &ValueMustBeStruct{
+		key:   key,
+		value: value,
+	}
+}
+
+type ValueMustBeMilestone struct {
+	key   string
+	value interface{}
+}
+
+func (v *ValueMustBeMilestone) Error() string {
+	return fmt.Sprintf("%q must be a milestone but it is a %T: %v", v.key, v.value, v.value)
+}
+
+func NewValueMustBeMilestone(key string, value interface{}) error {
+	return &ValueMustBeMilestone{
+		key:   key,
+		value: value,
+	}
+}
+
 type ValueMustBeOneOf struct {
 	key    string
 	value  string