Remove the check labels functionality for experiments

Author: jgustie

pkg/api/experiments/v1alpha1/naming.go

@@ -20,7 +20,6 @@ import (
 	"fmt"
 	"net/url"
 	"path"
-	"regexp"
 	"strconv"
 	"strings"
 
@@ -62,7 +61,7 @@ func JoinTrialName(e *Experiment, number int64) string {
 
 // SplitTrialName provides a consistent experience when trying to split a "trial name" into an experiment
 // name and a trial number. When the provided name does not contain a number, the resulting number will
-// be less then zero.
+// be less than zero.
 func SplitTrialName(name string) (ExperimentName, int64) {
 	// Names with slashes are always split (since the slash can't be in the name)
 	p := strings.LastIndex(name, "/")
@@ -84,27 +83,3 @@ func SplitTrialName(name string) (ExperimentName, int64) {
 
 	return ExperimentName(name), -1
 }
-
-// nameRegexp is used to validate the experiment labels that are required to be names.
-var nameRegexp = regexp.MustCompile(`^[a-z\d](?:[-a-z\d]{0,61}[a-z\d])?$`)
-
-// CheckLabels ensures the supplied experiment labels are valid.
-func CheckLabels(labels map[string]string) error {
-	if v, ok := labels["application"]; !ok {
-		return fmt.Errorf("missing required label: application")
-	} else if !nameRegexp.MatchString(v) {
-		return fmt.Errorf("invalid label value (must be lowercase): application=%q", v)
-	}
-
-	if v, ok := labels["scenario"]; !ok {
-		return fmt.Errorf("missing required label: scenario")
-	} else if !nameRegexp.MatchString(v) {
-		return fmt.Errorf("invalid label value (must be lowercase): scenario=%q", v)
-	}
-
-	if v, ok := labels["objective"]; ok && !nameRegexp.MatchString(v) {
-		return fmt.Errorf("invalid label value (must be lowercase): objective=%q", v)
-	}
-
-	return nil
-}