@@ -87,7 +87,6 @@ func TestExampleObjectSchemas(t *testing.T) {

 			"test-route":             &routeapi.Route{},

 			"test-service":           &kapi.Service{},

 			"test-buildcli":          &kapi.List{},

-			"test-buildcli-beta2":    &kapi.List{},

 		},

 		"../test/templates/fixtures": {

 			"crunchydata-pod": nil, // Explicitly fails validation, but should pass transformation

@@ -177,25 +177,18 @@ This section covers how to perform all the steps of building, deploying, and upd

 8. Confirm the registry is accessible (you may need to run this more than once):

-        $ export DOCKER_REGISTRY=`osc get service docker-registry --template="{{ .portalIP}}:{{ .port }}"`

-		$ curl $DOCKER_REGISTRY

+        $ curl `osc get service docker-registry --template="{{ .portalIP}}:{{ .port }}"`

     You should see:

         "docker-registry server (dev) (v0.9.0)"

-9. Push builder image to private docker-registry:

-

-		$ docker pull openshift/ruby-20-centos7:latest

-		$ docker tag -f openshift/ruby-20-centos7:latest ${DOCKER_REGISTRY}/test/ruby-20-centos7:latest

-		$ docker push ${DOCKER_REGISTRY}/test/ruby-20-centos7:latest

-

-10. Create a new project in OpenShift. This creates a namespace `test` to contain the builds and app that we will generate below.

+9. Create a new project in OpenShift. This creates a namespace `test` to contain the builds and app that we will generate below.

         $ openshift ex new-project test --display-name="OpenShift 3 Sample" --description="This is an example project to demonstrate OpenShift v3" --admin=anypassword:test-admin

-11. *Optional:* View the OpenShift web console in your browser by browsing to `https://<host>:8444`.  Login using the user `test-admin` and any password.

+10. *Optional:* View the OpenShift web console in your browser by browsing to `https://<host>:8444`.  Login using the user `test-admin` and any password.

     * You will need to have the browser accept the certificate at

       `https://<host>:8443` before the console can consult the OpenShift

@@ -206,7 +199,7 @@ This section covers how to perform all the steps of building, deploying, and upd

       and run builds.

-12. *Optional:* Fork the [ruby sample repository](https://github.com/openshift/ruby-hello-world)

+11. *Optional:* Fork the [ruby sample repository](https://github.com/openshift/ruby-hello-world)

     to an OpenShift-visible git account that you control, preferably

     somewhere that can also reach your OpenShift server with a webhook.

     A github.com account is an obvious place for this, but an in-house

@@ -219,7 +212,7 @@ This section covers how to perform all the steps of building, deploying, and upd

     Without your own fork, you can still run the initial build from

     OpenShift's public repository, just not a changed build.

-13. *Optional:* Add the following webhook under the settings in your new GitHub repository:

+12. *Optional:* Add the following webhook under the settings in your new GitHub repository:

         $ https://<host>:8443/osapi/v1beta1/buildConfigHooks/ruby-sample-build/secret101/github?namespace=test

@@ -230,12 +223,12 @@ This section covers how to perform all the steps of building, deploying, and upd

     instance as the certificate chain generated is not publicly verified.

-14. Edit application-template-stibuild.json which will define the sample application

+13. Edit application-template-stibuild.json which will define the sample application

  * Update the BuildConfig's sourceURI (git://github.com/openshift/ruby-hello-world.git) to point to your forked repository.

    *Note:* You can skip this step if you did not create a forked repository.

-15. Submit the application template for processing (generating shared parameters requested in the template)

+14. Submit the application template for processing (generating shared parameters requested in the template)

     and then request creation of the processed template:

         $ osc process -n test -f application-template-stibuild.json | osc create -n test -f -

@@ -254,13 +247,13 @@ This section covers how to perform all the steps of building, deploying, and upd

     Note that no build has actually occurred yet, so at this time there

     is no image to deploy and no application to visit.

-16. Trigger an initial build of your application

+15. Trigger an initial build of your application

  * If you setup the GitHub webhook, push a change to app.rb in your ruby sample repository.

  * Otherwise you can request a new build by running:

             $ osc start-build -n test ruby-sample-build

-17. Monitor the builds and wait for the status to go to "complete" (this can take a few minutes):

+16. Monitor the builds and wait for the status to go to "complete" (this can take a few minutes):

         $ osc get -n test builds

@@ -290,7 +283,7 @@ This section covers how to perform all the steps of building, deploying, and upd

     automatically trigger a deployment of the application, creating a

     pod each for the frontend (your Ruby code) and backend.

-18. Wait for the application's frontend pod and database pods to be started (this can take a few minutes):

+17. Wait for the application's frontend pod and database pods to be started (this can take a few minutes):

         $ osc get -n test pods

@@ -301,7 +294,7 @@ This section covers how to perform all the steps of building, deploying, and upd

         1b978f62-605f-11e4-b0db-3c970e3bf0b7                mysql                                                                                                             localhost.localdomain/   deploymentConfig=,deploymentID=database,name=database,replicationController=1b960e56-605f-11e4-b0db-3c970e3bf0b7,template=ruby-helloworld-sample             Running

         4a792f55-605f-11e4-b0db-3c970e3bf0b7                172.30.17.3:5001/openshift/origin-ruby-sample:9477bdb99a409b9c747e699361ae7934fd83bb4092627e2ee35f9f0b0869885b   localhost.localdomain/   deploymentConfig=frontend,deploymentID=frontend-1,name=frontend,replicationController=4a749831-605f-11e4-b0db-3c970e3bf0b7,template=ruby-helloworld-sample   Running

-19. Determine the IP for the frontend service:

+18. Determine the IP for the frontend service:

         $ osc get -n test services

@@ -317,7 +310,7 @@ This section covers how to perform all the steps of building, deploying, and upd

     *Note:* you can also get this information from the web console.

-20. Confirm the application is now accessible via the frontend service on port 5432.  Go to http://172.30.17.4:5432 (or whatever IP address was reported above) in your browser if you're running this locally; otherwise you can use curl to see the HTML, or port forward the address to your local workstation to visit it.

+19. Confirm the application is now accessible via the frontend service on port 5432.  Go to http://172.30.17.4:5432 (or whatever IP address was reported above) in your browser if you're running this locally; otherwise you can use curl to see the HTML, or port forward the address to your local workstation to visit it.

 	- - -

 	**VAGRANT USERS:**

@@ -330,14 +323,14 @@ This section covers how to perform all the steps of building, deploying, and upd

     You should see a welcome page and a form that allows you to query and update key/value pairs.  The keys are stored in the database container running in the database pod.

-21. Make a change to your ruby sample main.html file, commit, and push it via git.

+20. Make a change to your ruby sample main.html file, commit, and push it via git.

  * If you do not have the webhook enabled, you'll have to manually trigger another build:

             $ osc start-build -n test ruby-sample-build

-22. Repeat step 17 (waiting for the build to complete).  Once the build is complete, refreshing your browser should show your changes.

+21. Repeat step 16 (waiting for the build to complete).  Once the build is complete, refreshing your browser should show your changes.

 Congratulations, you've successfully deployed and updated an application on OpenShift.

@@ -329,14 +329,12 @@ openshift ex new-project recreated-project --admin="anypassword:createuser2"

 osc describe policybinding master -n recreated-project | grep anypassword:createuser2

 echo "ex new-project: ok"

-# Test running a router

 [ ! "$(openshift ex router | grep 'does not exist')"]

 [ "$(openshift ex router -o yaml --credentials="${OPENSHIFTCONFIG}" | grep 'openshift/origin-haproxy-')" ]

 openshift ex router --create --credentials="${OPENSHIFTCONFIG}"

 [ "$(openshift ex router | grep 'service exists')" ]

 echo "ex router: ok"

-# Test running a registry

 [ ! "$(openshift ex registry | grep 'does not exist')"]

 [ "$(openshift ex registry -o yaml --credentials="${OPENSHIFTCONFIG}" | grep 'openshift/origin-docker-registry')" ]

 openshift ex registry --create --credentials="${OPENSHIFTCONFIG}"

@@ -232,11 +232,11 @@ echo "[INFO] Installing the registry"

 # TODO: add --images="${USE_IMAGES}" when the Docker registry is built alongside OpenShift

 openshift ex registry --create --credentials="${CERT_DIR}/openshift-registry/.kubeconfig" --mount-host="/tmp/openshift.local.registry" --images='openshift/origin-${component}:latest'

-echo "[INFO] Pre-pulling and pushing ruby-20-centos7"

-docker pull openshift/ruby-20-centos7:latest

+echo "[INFO] Pre-pulling and pushing centos7"

+docker pull centos:centos7

 # TODO: remove after this becomes part of the build

 docker pull openshift/origin-docker-registry

-echo "[INFO] Pulled ruby-20-centos7"

+echo "[INFO] Pulled centos7"

 echo "[INFO] Waiting for Docker registry pod to start"

 # TODO: simplify when #4702 is fixed upstream

@@ -250,9 +250,9 @@ wait_for_url_timed "http://${DOCKER_REGISTRY}" "[INFO] Docker registry says: " $

 [ "$(dig @${API_HOST} "docker-registry.default.local." A)" ]

-docker tag -f openshift/ruby-20-centos7:latest ${DOCKER_REGISTRY}/test/ruby-20-centos7:latest

-docker push ${DOCKER_REGISTRY}/test/ruby-20-centos7:latest

-echo "[INFO] Pushed ruby-20-centos7"

+docker tag -f centos:centos7 ${DOCKER_REGISTRY}/cached/centos:centos7

+docker push ${DOCKER_REGISTRY}/cached/centos:centos7

+echo "[INFO] Pushed centos7"

 # Process template and create

 echo "[INFO] Submitting application template json for processing..."

@@ -48,7 +48,7 @@ type BuildParameters struct {

 	Revision *SourceRevision `json:"revision,omitempty"`

 	// Strategy defines how to perform a build.

-	Strategy BuildStrategy `json:"strategy"`

+	Strategy BuildStrategy `json:"strategy,omitempty"`

 	// Output describes the Docker image the Strategy should produce.

 	Output BuildOutput `json:"output,omitempty"`

@@ -142,7 +142,7 @@ type SourceControlUser struct {

 // BuildStrategy contains the details of how to perform a build.

 type BuildStrategy struct {

 	// Type is the kind of build strategy.

-	Type BuildStrategyType `json:"type"`

+	Type BuildStrategyType `json:"type,omitempty"`

 	// DockerStrategy holds the parameters to the Docker build strategy.

 	DockerStrategy *DockerBuildStrategy `json:"dockerStrategy,omitempty"`

@@ -209,16 +209,8 @@ type DockerBuildStrategy struct {

 // STIBuildStrategy defines input parameters specific to an STI build.

 type STIBuildStrategy struct {

 	// Image is the image used to execute the build.

-	// Only valid if From is not present.

 	Image string `json:"image,omitempty"`

-	// From is reference to an image repository from where the docker image should be pulled

-	From *kapi.ObjectReference `json:"from,omitempty"`

-

-	// Tag is the name of image repository tag to be used as the build image, it only

-	// applies when From is specified.

-	Tag string `json:"tag,omitempty`

-

 	// Additional environment variables you want to pass into a builder container

 	Env []kapi.EnvVar `json:"env,omitempty"`

@@ -53,27 +53,11 @@ func init() {

 		func(in *newer.STIBuildStrategy, out *STIBuildStrategy, s conversion.Scope) error {

 			out.BuilderImage = in.Image

 			out.Image = in.Image

-			if in.From != nil {

-				out.From = &kapi.ObjectReference{

-					Name:      in.From.Name,

-					Namespace: in.From.Namespace,

-					Kind:      "ImageRepository",

-				}

-			}

-			out.Tag = in.Tag

 			out.Scripts = in.Scripts

 			out.Clean = !in.Incremental

 			return s.Convert(&in.Env, &out.Env, 0)

 		},

 		func(in *STIBuildStrategy, out *newer.STIBuildStrategy, s conversion.Scope) error {

-			if in.From != nil {

-				out.From = &api.ObjectReference{

-					Name:      in.From.Name,

-					Namespace: in.From.Namespace,

-					Kind:      "ImageRepository",

-				}

-			}

-			out.Tag = in.Tag

 			out.Scripts = in.Scripts

 			out.Incremental = !in.Clean

 			if len(in.Image) != 0 {

@@ -26,12 +26,6 @@ func TestSTIBuildStrategyConversion(t *testing.T) {

 	if actual.Image != oldVersion.BuilderImage {

 		t.Errorf("expected %v, actual %v", oldVersion.BuilderImage, actual.Image)

 	}

-	if actual.From != nil {

-		t.Errorf("expected %v, actual %v", nil, actual.From)

-	}

-	if actual.Tag != oldVersion.Tag {

-		t.Errorf("expected %v, actual %v", oldVersion.Tag, actual.Tag)

-	}

 	if actual.Incremental == oldVersion.Clean {

 		t.Errorf("expected %v, actual %v", oldVersion.Clean, actual.Incremental)

 	}

@@ -48,7 +48,7 @@ type BuildParameters struct {

 	Revision *SourceRevision `json:"revision,omitempty"`

 	// Strategy defines how to perform a build.

-	Strategy BuildStrategy `json:"strategy"`

+	Strategy BuildStrategy `json:"strategy,omitempty"`

 	// Output describes the Docker image the Strategy should produce.

 	Output BuildOutput `json:"output,omitempty"`

@@ -142,7 +142,7 @@ type SourceControlUser struct {

 // BuildStrategy contains the details of how to perform a build.

 type BuildStrategy struct {

 	// Type is the kind of build strategy.

-	Type BuildStrategyType `json:"type"`

+	Type BuildStrategyType `json:"type,omitempty"`

 	// DockerStrategy holds the parameters to the Docker build strategy.

 	DockerStrategy *DockerBuildStrategy `json:"dockerStrategy,omitempty"`

@@ -214,17 +214,8 @@ type STIBuildStrategy struct {

 	BuilderImage string `json:"builderImage,omitempty"`

 	// Image is the image used to execute the build.

-	// For BuildConfigs, From takes precedence.

 	Image string `json:"image,omitempty"`

-	// Tag is the name of image repository tag to be used as the build image, it only

-	// applies when From is specified.

-	Tag string `json:"tag,omitempty"`

-

-	// From is reference to an image repository from where the docker image should be pulled

-	// Only allowed in BuildConfigs, Builds use the Image field exclusively.

-	From *kapi.ObjectReference `json:"from,omitempty"`

-

 	// Additional environment variables you want to pass into a builder container

 	Env []kapi.EnvVar `json:"env,omitempty"`

@@ -144,22 +144,23 @@ func validateBuildConfigOutput(output *buildapi.BuildOutput) errs.ValidationErro

 func validateStrategy(strategy *buildapi.BuildStrategy) errs.ValidationErrorList {

 	allErrs := errs.ValidationErrorList{}

-	switch {

-	case len(strategy.Type) == 0:

+	if len(strategy.Type) == 0 {

 		allErrs = append(allErrs, errs.NewFieldRequired("type"))

+	}

-	case strategy.Type == buildapi.STIBuildStrategyType:

+	switch strategy.Type {

+	case buildapi.STIBuildStrategyType:

 		if strategy.STIStrategy == nil {

 			allErrs = append(allErrs, errs.NewFieldRequired("stiStrategy"))

 		} else {

 			allErrs = append(allErrs, validateSTIStrategy(strategy.STIStrategy).Prefix("stiStrategy")...)

 		}

-	case strategy.Type == buildapi.DockerBuildStrategyType:

+	case buildapi.DockerBuildStrategyType:

 		// DockerStrategy is currently optional, initialize it to a default state if it's not set.

 		if strategy.DockerStrategy == nil {

 			strategy.DockerStrategy = &buildapi.DockerBuildStrategy{}

 		}

-	case strategy.Type == buildapi.CustomBuildStrategyType:

+	case buildapi.CustomBuildStrategyType:

 		if strategy.CustomStrategy == nil {

 			allErrs = append(allErrs, errs.NewFieldRequired("customStrategy"))

 		} else {

@@ -177,11 +178,8 @@ func validateStrategy(strategy *buildapi.BuildStrategy) errs.ValidationErrorList

 func validateSTIStrategy(strategy *buildapi.STIBuildStrategy) errs.ValidationErrorList {

 	allErrs := errs.ValidationErrorList{}

-	if (strategy.From == nil || len(strategy.From.Name) == 0) && len(strategy.Image) == 0 {

-		allErrs = append(allErrs, errs.NewFieldRequired("from"))

-	}

-	if (strategy.From != nil && len(strategy.From.Name) != 0) && len(strategy.Image) != 0 {

-		allErrs = append(allErrs, errs.NewFieldInvalid("image", strategy.Image, "only one of 'image' and 'from' may be set"))

+	if len(strategy.Image) == 0 {

+		allErrs = append(allErrs, errs.NewFieldRequired("image"))

 	}

 	return allErrs

 }

@@ -76,9 +76,7 @@ func TestBuildConfigValidationSuccess(t *testing.T) {

 				Type:           buildapi.DockerBuildStrategyType,

 				DockerStrategy: &buildapi.DockerBuildStrategy{},

 			},

-			Output: buildapi.BuildOutput{

-				DockerImageReference: "repository/data",

-			},

+			Output: buildapi.BuildOutput{},

 		},

 	}

 	if result := ValidateBuildConfig(buildConfig); len(result) > 0 {

@@ -277,7 +275,7 @@ func TestValidateBuildParameters(t *testing.T) {

 			},

 		},

 		{

-			string(errs.ValidationErrorTypeInvalid) + "strategy.type",

+			string(errs.ValidationErrorTypeRequired) + "strategy.stiStrategy.image",

 			&buildapi.BuildParameters{

 				Source: buildapi.BuildSource{

 					Type: buildapi.BuildSourceGit,

@@ -285,70 +283,15 @@ func TestValidateBuildParameters(t *testing.T) {

 						URI: "http://github.com/my/repository",

 					},

 				},

-				Strategy: buildapi.BuildStrategy{Type: "classic-joke"},

 				Output: buildapi.BuildOutput{

 					DockerImageReference: "repository/data",

 				},

-			},

-		},

-		{

-			string(errs.ValidationErrorTypeRequired) + "strategy.type",

-			&buildapi.BuildParameters{

-				Source: buildapi.BuildSource{

-					Type: buildapi.BuildSourceGit,

-					Git: &buildapi.GitBuildSource{

-						URI: "http://github.com/my/repository",

-					},

-				},

-				Strategy: buildapi.BuildStrategy{},

-				Output: buildapi.BuildOutput{

-					DockerImageReference: "repository/data",

-				},

-			},

-		},

-		// invalid because both image and from are specified in the

-		// sti strategy definition

-		{

-			string(errs.ValidationErrorTypeInvalid) + "strategy.stiStrategy.image",

-			&buildapi.BuildParameters{

-				Source: buildapi.BuildSource{

-					Type: buildapi.BuildSourceGit,

-					Git: &buildapi.GitBuildSource{

-						URI: "http://github.com/my/repository",

-					},

-				},

 				Strategy: buildapi.BuildStrategy{

 					Type: buildapi.STIBuildStrategyType,

 					STIStrategy: &buildapi.STIBuildStrategy{

-						Image: "image",

-						From: &kapi.ObjectReference{

-							Name: "reponame",

-						},

-					},

-				},

-				Output: buildapi.BuildOutput{

-					DockerImageReference: "repository/data",

-				},

-			},

-		},

-		// invalid because neither image nor from are specified in the

-		// sti strategy definition

-		{

-			string(errs.ValidationErrorTypeRequired) + "strategy.stiStrategy.from",

-			&buildapi.BuildParameters{

-				Source: buildapi.BuildSource{

-					Type: buildapi.BuildSourceGit,

-					Git: &buildapi.GitBuildSource{

-						URI: "http://github.com/my/repository",

+						Image: "",

 					},

 				},

-				Strategy: buildapi.BuildStrategy{

-					Type:        buildapi.STIBuildStrategyType,

-					STIStrategy: &buildapi.STIBuildStrategy{},

-				},

-				Output: buildapi.BuildOutput{

-					DockerImageReference: "repository/data",

-				},

 			},

 		},

 	}

@@ -366,61 +309,6 @@ func TestValidateBuildParameters(t *testing.T) {

 	}

 }

-func TestValidateBuildParametersSuccess(t *testing.T) {

-	testCases := []struct {

-		*buildapi.BuildParameters

-	}{

-		{

-			&buildapi.BuildParameters{

-				Source: buildapi.BuildSource{

-					Type: buildapi.BuildSourceGit,

-					Git: &buildapi.GitBuildSource{

-						URI: "http://github.com/my/repository",

-					},

-				},

-				Strategy: buildapi.BuildStrategy{

-					Type: buildapi.STIBuildStrategyType,

-					STIStrategy: &buildapi.STIBuildStrategy{

-						Image: "repository/builder-image",

-					},

-				},

-				Output: buildapi.BuildOutput{

-					DockerImageReference: "repository/data",

-				},

-			},

-		},

-		{

-			&buildapi.BuildParameters{

-				Source: buildapi.BuildSource{

-					Type: buildapi.BuildSourceGit,

-					Git: &buildapi.GitBuildSource{

-						URI: "http://github.com/my/repository",

-					},

-				},

-				Strategy: buildapi.BuildStrategy{

-					Type: buildapi.STIBuildStrategyType,

-					STIStrategy: &buildapi.STIBuildStrategy{

-						From: &kapi.ObjectReference{

-							Name: "reponame",

-						},

-					},

-				},

-				Output: buildapi.BuildOutput{

-					DockerImageReference: "repository/data",

-				},

-			},

-		},

-	}

-

-	for _, config := range testCases {

-		errors := validateBuildParameters(config.BuildParameters)

-		if len(errors) != 0 {

-			t.Errorf("Unexpected validation error: %v", errors)

-		}

-	}

-

-}

-

 func TestValidateTrigger(t *testing.T) {

 	tests := map[string]struct {

 		trigger  buildapi.BuildTriggerPolicy

@@ -8,7 +8,6 @@ import (

 	"github.com/GoogleCloudPlatform/kubernetes/pkg/client/cache"

 	"github.com/GoogleCloudPlatform/kubernetes/pkg/util"

-	kapi "github.com/GoogleCloudPlatform/kubernetes/pkg/api"

 	buildapi "github.com/openshift/origin/pkg/build/api"

 	buildclient "github.com/openshift/origin/pkg/build/client"

 	buildutil "github.com/openshift/origin/pkg/build/util"

@@ -38,8 +37,7 @@ type ImageChangeController struct {

 // HandleImageRepo processes the next ImageRepository event.

 func (c *ImageChangeController) HandleImageRepo(repo *imageapi.ImageRepository) error {

 	glog.V(4).Infof("Build image change controller detected imagerepo change %s", repo.Status.DockerImageRepository)

-	imageSubs := make(map[string]string)

-	repoSubs := make(map[kapi.ObjectReference]string)

+	subs := make(map[string]string)

 	// TODO: this is inefficient

 	for _, bc := range c.BuildConfigStore.List() {

@@ -75,8 +73,7 @@ func (c *ImageChangeController) HandleImageRepo(repo *imageapi.ImageRepository)

 				next = latest.DockerImageReference

 			}

 			if len(last) == 0 || next != last {

-				imageSubs[change.Image] = latest.DockerImageReference

-				repoSubs[change.From] = latest.DockerImageReference

+				subs[change.Image] = latest.DockerImageReference

 				change.LastTriggeredImageID = next

 				shouldBuild = true

 			}

@@ -84,7 +81,7 @@ func (c *ImageChangeController) HandleImageRepo(repo *imageapi.ImageRepository)

 		if shouldBuild {

 			glog.V(4).Infof("Running build for buildConfig %s in namespace %s", config.Name, config.Namespace)

-			b := buildutil.GenerateBuildFromConfig(config, nil, imageSubs, repoSubs)

+			b := buildutil.GenerateBuildFromConfig(config, nil, subs)

 			if err := c.BuildCreator.Create(config.Namespace, b); err != nil {

 				return fmt.Errorf("error starting build for buildConfig %s: %v", config.Name, err)

 			}

@@ -255,37 +255,6 @@ func mockBuildConfig() *api.BuildConfig {

 	}

 }

-func mockBuildConfigImageRef() *api.BuildConfig {

-	return &api.BuildConfig{

-		ObjectMeta: kapi.ObjectMeta{

-			Name:      "data-build",

-			Namespace: kapi.NamespaceDefault,

-			Labels: map[string]string{

-				"name": "data-build",

-			},

-		},

-		Parameters: api.BuildParameters{

-			Source: api.BuildSource{

-				Type: api.BuildSourceGit,

-				Git: &api.GitBuildSource{

-					URI: "http://my.build.com/the/build/Dockerfile",

-				},

-			},

-			Strategy: api.BuildStrategy{

-				Type: api.STIBuildStrategyType,

-				STIStrategy: &api.STIBuildStrategy{

-					From: &kapi.ObjectReference{

-						Name: "builder/image",

-					},

-				},

-			},

-			Output: api.BuildOutput{

-				DockerImageReference: "repository/data-build",

-			},

-		},

-	}

-}

-

 func TestUpdateBuildConfig(t *testing.T) {

 	mockRegistry := test.BuildConfigRegistry{}

 	storage := REST{&mockRegistry}

@@ -321,143 +290,69 @@ func TestUpdateBuildConfigError(t *testing.T) {

 func TestBuildConfigRESTValidatesCreate(t *testing.T) {

 	mockRegistry := test.BuildConfigRegistry{}

 	storage := REST{&mockRegistry}

-	failureCases := map[string]struct {

-		expectSuccess bool

-		data          api.BuildConfig

-	}{

+	failureCases := map[string]api.BuildConfig{

 		"blank sourceURI": {

-			false,

-			api.BuildConfig{

-				ObjectMeta: kapi.ObjectMeta{Name: "abc"},

-				Parameters: api.BuildParameters{

-					Source: api.BuildSource{

-						Type: api.BuildSourceGit,

-						Git: &api.GitBuildSource{

-							URI: "",

-						},

+			ObjectMeta: kapi.ObjectMeta{Name: "abc"},

+			Parameters: api.BuildParameters{

+				Source: api.BuildSource{

+					Type: api.BuildSourceGit,

+					Git: &api.GitBuildSource{

+						URI: "",

 					},

-					Strategy: api.BuildStrategy{

-						Type: api.STIBuildStrategyType,

-						STIStrategy: &api.STIBuildStrategy{

-							From: &kapi.ObjectReference{

-								Name: "builder/image",

-							},

-						},

-					},

-					Output: api.BuildOutput{

-						DockerImageReference: "data/image",

+				},

+				Strategy: api.BuildStrategy{

+					Type: api.STIBuildStrategyType,

+					STIStrategy: &api.STIBuildStrategy{

+						Image: "builder/image",

 					},

 				},

+				Output: api.BuildOutput{

+					DockerImageReference: "data/image",

+				},

 			},

 		},

 		"blank DockerImageReference": {

-			true,

-			api.BuildConfig{

-				ObjectMeta: kapi.ObjectMeta{Name: "abc"},

-				Parameters: api.BuildParameters{

-					Source: api.BuildSource{

-						Type: api.BuildSourceGit,

-						Git: &api.GitBuildSource{

-							URI: "http://github.com/test/source",

-						},

-					},

-					Strategy: api.BuildStrategy{

-						Type: api.STIBuildStrategyType,

-						STIStrategy: &api.STIBuildStrategy{

-							From: &kapi.ObjectReference{

-								Name: "builder/image",

-							},

-						},

-					},

-					Output: api.BuildOutput{

-						DockerImageReference: "",

+			ObjectMeta: kapi.ObjectMeta{Name: "abc"},

+			Parameters: api.BuildParameters{

+				Source: api.BuildSource{

+					Type: api.BuildSourceGit,

+					Git: &api.GitBuildSource{

+						URI: "http://github.com/test/source",

 					},

 				},

-			},

-		},

-		"blank From.Name and blank Image": {

-			false,

-			api.BuildConfig{

-				ObjectMeta: kapi.ObjectMeta{Name: "abc"},

-				Parameters: api.BuildParameters{

-					Source: api.BuildSource{

-						Type: api.BuildSourceGit,

-						Git: &api.GitBuildSource{

-							URI: "http://github.com/test/source",

-						},

-					},

-					Strategy: api.BuildStrategy{

-						Type:        api.STIBuildStrategyType,

-						STIStrategy: &api.STIBuildStrategy{},

-					},

-					Output: api.BuildOutput{

-						DockerImageReference: "data/image",

-					},

+				Output: api.BuildOutput{

+					DockerImageReference: "",

 				},

 			},

 		},

-		"blank From.Name and Image present": {

-			true,

-			api.BuildConfig{

-				ObjectMeta: kapi.ObjectMeta{Name: "abc"},

-				Parameters: api.BuildParameters{

-					Source: api.BuildSource{

-						Type: api.BuildSourceGit,

-						Git: &api.GitBuildSource{

-							URI: "http://github.com/test/source",

-						},

-					},

-					Strategy: api.BuildStrategy{

-						Type: api.STIBuildStrategyType,

-						STIStrategy: &api.STIBuildStrategy{

-							Image: "builder/image",

-						},

-					},

-					Output: api.BuildOutput{

-						DockerImageReference: "data/image",

+		"blank Image": {

+			ObjectMeta: kapi.ObjectMeta{Name: "abc"},

+			Parameters: api.BuildParameters{

+				Source: api.BuildSource{

+					Type: api.BuildSourceGit,

+					Git: &api.GitBuildSource{

+						URI: "http://github.com/test/source",

 					},

 				},

-			},

-		},

-		"blank Image and From.Name present": {

-			true,

-			api.BuildConfig{

-				ObjectMeta: kapi.ObjectMeta{Name: "abc"},

-				Parameters: api.BuildParameters{

-					Source: api.BuildSource{

-						Type: api.BuildSourceGit,

-						Git: &api.GitBuildSource{

-							URI: "http://github.com/test/source",

-						},

-					},

-					Strategy: api.BuildStrategy{

-						Type: api.STIBuildStrategyType,

-						STIStrategy: &api.STIBuildStrategy{

-							From: &kapi.ObjectReference{

-								Name: "builder/image",

-							},

-						},

-					},

-					Output: api.BuildOutput{

-						DockerImageReference: "data/image",

+				Strategy: api.BuildStrategy{

+					Type: api.STIBuildStrategyType,

+					STIStrategy: &api.STIBuildStrategy{

+						Image: "",

 					},

 				},

+				Output: api.BuildOutput{

+					DockerImageReference: "data/image",

+				},

 			},

 		},

 	}

-	for desc, testCase := range failureCases {

-		c, err := storage.Create(kapi.NewDefaultContext(), &testCase.data)

-		if testCase.expectSuccess {

-			if c == nil {

-				t.Errorf("%s: Expected success, got error: %s", desc, err)

-			}

-		} else {

-			if c != nil {

-				t.Errorf("%s: Expected nil object", desc)

-			}

-			if !errors.IsInvalid(err) {

-				t.Errorf("%s: Expected to get an invalid resource error, got %v", desc, err)

-			}

+	for desc, failureCase := range failureCases {

+		c, err := storage.Create(kapi.NewDefaultContext(), &failureCase)

+		if c != nil {

+			t.Errorf("%s: Expected nil object", desc)

+		}

+		if !errors.IsInvalid(err) {

+			t.Errorf("%s: Expected to get an invalid resource error, got %v", desc, err)

 		}

 	}

 }

@@ -465,175 +360,83 @@ func TestBuildConfigRESTValidatesCreate(t *testing.T) {

 func TestBuildRESTValidatesUpdate(t *testing.T) {

 	mockRegistry := test.BuildConfigRegistry{}

 	storage := REST{&mockRegistry}

-	failureCases := map[string]struct {

-		expectSuccess bool

-		data          api.BuildConfig

-	}{

+	failureCases := map[string]api.BuildConfig{

 		"empty ID": {

-			false,

-			api.BuildConfig{

-				ObjectMeta: kapi.ObjectMeta{

-					Name:      "",

-					Namespace: kapi.NamespaceDefault,

-				},

-				Parameters: api.BuildParameters{

-					Source: api.BuildSource{

-						Type: api.BuildSourceGit,

-						Git: &api.GitBuildSource{

-							URI: "http://github.com/test/source",

-						},

-					},

-					Output: api.BuildOutput{

-						DockerImageReference: "data/image",

+			ObjectMeta: kapi.ObjectMeta{Name: ""},

+			Parameters: api.BuildParameters{

+				Source: api.BuildSource{

+					Type: api.BuildSourceGit,

+					Git: &api.GitBuildSource{

+						URI: "http://github.com/test/source",

 					},

 				},

+				Output: api.BuildOutput{

+					DockerImageReference: "data/image",

+				},

 			},

 		},

 		"blank sourceURI": {

-			false,

-			api.BuildConfig{

-				ObjectMeta: kapi.ObjectMeta{

-					Name:      "abc",

-					Namespace: kapi.NamespaceDefault,

-				},

-				Parameters: api.BuildParameters{

-					Source: api.BuildSource{

-						Type: api.BuildSourceGit,

-						Git: &api.GitBuildSource{

-							URI: "",

-						},

-					},

-					Strategy: api.BuildStrategy{

-						Type: api.STIBuildStrategyType,

-						STIStrategy: &api.STIBuildStrategy{

-							From: &kapi.ObjectReference{

-								Name: "builder/image",

-							},

-						},

+			ObjectMeta: kapi.ObjectMeta{Name: "abc"},

+			Parameters: api.BuildParameters{

+				Source: api.BuildSource{

+					Type: api.BuildSourceGit,

+					Git: &api.GitBuildSource{

+						URI: "",

 					},

-					Output: api.BuildOutput{

-						DockerImageReference: "data/image",

+				},

+				Strategy: api.BuildStrategy{

+					Type: api.STIBuildStrategyType,

+					STIStrategy: &api.STIBuildStrategy{

+						Image: "builder/image",

 					},

 				},

+				Output: api.BuildOutput{

+					DockerImageReference: "data/image",

+				},

 			},

 		},

 		"blank DockerImageReference": {

-			false,

-			api.BuildConfig{

-				ObjectMeta: kapi.ObjectMeta{

-					Name:      "abc",

-					Namespace: kapi.NamespaceDefault,

-				},

-				Parameters: api.BuildParameters{

-					Source: api.BuildSource{

-						Type: api.BuildSourceGit,

-						Git: &api.GitBuildSource{

-							URI: "http://github.com/test/source",

-						},

-					},

-					Output: api.BuildOutput{

-						DockerImageReference: "",

+			ObjectMeta: kapi.ObjectMeta{Name: "abc"},

+			Parameters: api.BuildParameters{

+				Source: api.BuildSource{

+					Type: api.BuildSourceGit,

+					Git: &api.GitBuildSource{

+						URI: "http://github.com/test/source",

 					},

 				},

-			},

-		},

-		"blank From.Name and blank Image": {

-			false,

-			api.BuildConfig{

-				ObjectMeta: kapi.ObjectMeta{

-					Name:      "abc",

-					Namespace: kapi.NamespaceDefault,

-				},

-				Parameters: api.BuildParameters{

-					Source: api.BuildSource{

-						Type: api.BuildSourceGit,

-						Git: &api.GitBuildSource{

-							URI: "http://github.com/test/source",

-						},

-					},

-					Strategy: api.BuildStrategy{

-						Type: api.STIBuildStrategyType,

-						STIStrategy: &api.STIBuildStrategy{

-							From: &kapi.ObjectReference{

-								Name: "",

-							},

-						},