Fix missing portalNets in tests once and for all by adding a default.

cmd/integration/integration.go

@@ -129,12 +129,6 @@ func startComponents(manifestURL string) (apiServerURL string) {
 	}
 
 	// Master
-	_, portalNet, err := net.ParseCIDR("10.0.0.0/24")
-	if err != nil {
-		glog.Fatalf("Unable to parse CIDR: %v", err)
-	}
-	glog.Infof("Using portalNet '%v'", portalNet)
-
 	host, port, err := net.SplitHostPort(strings.TrimLeft(apiServer.URL, "http://"))
 	if err != nil {
 		glog.Fatalf("Unable to parse URL '%v': %v", apiServer.URL, err)
@@ -150,7 +144,6 @@ func startComponents(manifestURL string) (apiServerURL string) {
 		EtcdHelper:        helper,
 		Minions:           machineList,
 		KubeletClient:     fakeKubeletClient{},
-		PortalNet:         portalNet,
 		Mux:               mux,
 		EnableLogsSupport: false,
 		APIPrefix:         "/api",

pkg/master/master.go

@@ -131,6 +131,15 @@ func NewEtcdHelper(client tools.EtcdGetSet, version string) (helper tools.EtcdHe
 
 // setDefaults fills in any fields not set that are required to have valid data.
 func setDefaults(c *Config) {
+	if c.PortalNet == nil {
+		defaultNet := "10.0.0.0/24"
+		glog.Warningf("Portal net unspecified. Defaulting to %v.", defaultNet)
+		_, portalNet, err := net.ParseCIDR(defaultNet)
+		if err != nil {
+			glog.Fatalf("Unable to parse CIDR: %v", err)
+		}
+		c.PortalNet = portalNet
+	}
 	if c.MasterCount == 0 {
 		// Clearly, there will be at least one master.
 		c.MasterCount = 1

test/integration/client_test.go

@@ -19,7 +19,6 @@ limitations under the License.
 package integration
 
 import (
-	"net"
 	"net/http"
 	"net/http/httptest"
 	"reflect"
@@ -30,8 +29,6 @@ import (
 	"github.com/GoogleCloudPlatform/kubernetes/pkg/labels"
 	"github.com/GoogleCloudPlatform/kubernetes/pkg/master"
 	"github.com/GoogleCloudPlatform/kubernetes/pkg/version"
-
-	"github.com/golang/glog"
 )
 
 func init() {
@@ -45,13 +42,8 @@ func TestClient(t *testing.T) {
 	}
 	mux := http.NewServeMux()
 
-	_, portalNet, err := net.ParseCIDR("10.0.0.0/24")
-	if err != nil {
-		glog.Fatalf("Unable to parse CIDR: %v", err)
-	}
 	master.New(&master.Config{
 		EtcdHelper:        helper,
-		PortalNet:         portalNet,
 		Mux:               mux,
 		EnableLogsSupport: false,
 		EnableUISupport:   false,