Merge pull request #268 from mesosphere/jdef-fix-netinfo-validation

add netinfo to validateIPSources config validation
d2iq-archive · Sep 18, 2015 · ef2e0d8 · ef2e0d8
2 parents 92897a1 + 1ba7f14
commit ef2e0d8
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/records/config_test.go b/records/config_test.go
@@ -14,3 +14,12 @@ func TestNonLocalAddies(t *testing.T) {
 		}
 	}
 }
+
+func TestNewConfigValidates(t *testing.T) {
+	c := NewConfig()
+	err := validateIPSources(c.IPSources)
+	if err != nil {
+		t.Fatal(err)
+	}
+	//TODO(jdef) add other validators here..
+}
diff --git a/records/validation.go b/records/validation.go
@@ -64,7 +64,7 @@ func validateIPSources(srcs []string) error {
 	}
 	for _, src := range srcs {
 		switch src {
-		case "host", "docker", "mesos":
+		case "host", "docker", "mesos", "netinfo":
 		default:
 			return fmt.Errorf("invalid ip source %q", src)
 		}