separate single use helpers and actual shared code

pkg/build/controller/jenkins/jenkins.go

@@ -4,6 +4,7 @@ import (
 	"fmt"
 
 	"github.com/golang/glog"
+	"github.com/openshift/origin/pkg/template/templateprocessing"
 	"k8s.io/kubernetes/pkg/api/legacyscheme"
 
 	kerrs "k8s.io/apimachinery/pkg/api/errors"
@@ -13,7 +14,6 @@ import (
 	kapi "k8s.io/kubernetes/pkg/apis/core"
 
 	serverapi "github.com/openshift/origin/pkg/cmd/server/apis/config"
-	"github.com/openshift/origin/pkg/template"
 	templateapi "github.com/openshift/origin/pkg/template/apis/template"
 	templateinternal "github.com/openshift/origin/pkg/template/client/internalversion"
 	templateclient "github.com/openshift/origin/pkg/template/generated/internalclientset"
@@ -104,7 +104,7 @@ func substituteTemplateParameters(params map[string]string, t *templateapi.Templ
 			errors = append(errors, fmt.Errorf("template parameter name cannot be empty (%q)", value))
 			continue
 		}
-		if v := template.GetParameterByName(t, name); v != nil {
+		if v := templateprocessing.GetParameterByName(t, name); v != nil {
 			v.Value = value
 			v.Generate = ""
 		} else {

pkg/oc/cli/cmd/process.go

@@ -8,6 +8,7 @@ import (
 	"strings"
 	"time"
 
+	"github.com/openshift/origin/pkg/template/templateprocessing"
 	"github.com/spf13/cobra"
 	"k8s.io/apimachinery/pkg/api/errors"
 	"k8s.io/apimachinery/pkg/api/meta"
@@ -28,7 +29,6 @@ import (
 	"github.com/openshift/origin/pkg/oc/cli/describe"
 	"github.com/openshift/origin/pkg/oc/cli/util/clientcmd"
 	"github.com/openshift/origin/pkg/oc/generate/app"
-	"github.com/openshift/origin/pkg/template"
 	templateapi "github.com/openshift/origin/pkg/template/apis/template"
 	templatevalidation "github.com/openshift/origin/pkg/template/apis/template/validation"
 	templateinternalclient "github.com/openshift/origin/pkg/template/client/internalversion"
@@ -363,7 +363,7 @@ func RunProcess(f *clientcmd.Factory, in io.Reader, out, errout io.Writer, cmd *
 func injectUserVars(values app.Environment, t *templateapi.Template, ignoreUnknownParameters bool) []error {
 	var errors []error
 	for param, val := range values {
-		v := template.GetParameterByName(t, param)
+		v := templateprocessing.GetParameterByName(t, param)
 		if v != nil {
 			v.Value = val
 			v.Generate = ""
@@ -380,7 +380,7 @@ func processTemplateLocally(tpl *templateapi.Template) error {
 	if errs := templatevalidation.ValidateProcessedTemplate(tpl); len(errs) > 0 {
 		return errors.NewInvalid(templateapi.Kind("Template"), tpl.Name, errs)
 	}
-	processor := template.NewProcessor(map[string]generator.Generator{
+	processor := templateprocessing.NewProcessor(map[string]generator.Generator{
 		"expression": generator.NewExpressionValueGenerator(rand.New(rand.NewSource(time.Now().UnixNano()))),
 	})
 	if errs := processor.Process(tpl); len(errs) > 0 {

pkg/oc/generate/app/templatelookup.go

@@ -13,7 +13,6 @@ import (
 	kapi "k8s.io/kubernetes/pkg/apis/core"
 	"k8s.io/kubernetes/pkg/kubectl/resource"
 
-	"github.com/openshift/origin/pkg/template"
 	templateapi "github.com/openshift/origin/pkg/template/apis/template"
 	templateclient "github.com/openshift/origin/pkg/template/generated/internalclientset/typed/template/internalversion"
 )
@@ -34,7 +33,7 @@ func (r TemplateSearcher) Search(precise bool, terms ...string) (ComponentMatche
 	matches := ComponentMatches{}
 	var errs []error
 	for _, term := range terms {
-		ref, err := template.ParseTemplateReference(term)
+		ref, err := parseTemplateReference(term)
 		if err != nil {
 			glog.V(2).Infof("template references must be of the form [<namespace>/]<name>, term %q did not qualify", term)
 			continue
@@ -45,7 +44,7 @@ func (r TemplateSearcher) Search(precise bool, terms ...string) (ComponentMatche
 		}
 
 		namespaces := r.Namespaces
-		if ref.HasNamespace() {
+		if ref.hasNamespace() {
 			namespaces = []string{ref.Namespace}
 		}
 
@@ -182,3 +181,41 @@ func (r *TemplateFileSearcher) Search(precise bool, terms ...string) (ComponentM
 
 	return matches, errs
 }
+
+// TemplateReference points to a stored template
+type templateReference struct {
+	Namespace string
+	Name      string
+}
+
+// ParseTemplateReference parses the reference to a template into a
+// TemplateReference.
+func parseTemplateReference(s string) (templateReference, error) {
+	var ref templateReference
+	parts := strings.Split(s, "/")
+	switch len(parts) {
+	case 2:
+		// namespace/name
+		ref.Namespace = parts[0]
+		ref.Name = parts[1]
+		break
+	case 1:
+		// name
+		ref.Name = parts[0]
+		break
+	default:
+		return ref, fmt.Errorf("the template reference must be either the template name or namespace and template name separated by slashes")
+	}
+	return ref, nil
+}
+
+func (r templateReference) hasNamespace() bool {
+	return len(r.Namespace) > 0
+}
+
+func (r templateReference) String() string {
+	if r.hasNamespace() {
+		return fmt.Sprintf("%s/%s", r.Namespace, r.Name)
+	}
+	return r.Name
+}

pkg/oc/generate/cmd/template.go

@@ -5,13 +5,13 @@ import (
 	"io"
 	"strings"
 
+	"github.com/openshift/origin/pkg/template/templateprocessing"
 	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/apimachinery/pkg/util/errors"
 	"k8s.io/kubernetes/pkg/api/legacyscheme"
 
 	"github.com/openshift/origin/pkg/api/latest"
 	"github.com/openshift/origin/pkg/oc/generate/app"
-	"github.com/openshift/origin/pkg/template"
 	templateapi "github.com/openshift/origin/pkg/template/apis/template"
 	templateinternalclient "github.com/openshift/origin/pkg/template/client/internalversion"
 	"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
@@ -21,7 +21,7 @@ import (
 func TransformTemplate(tpl *templateapi.Template, templateProcessor templateinternalclient.TemplateProcessorInterface, namespace string, parameters map[string]string, ignoreUnknownParameters bool) (*templateapi.Template, error) {
 	// only set values that match what's expected by the template.
 	for k, value := range parameters {
-		v := template.GetParameterByName(tpl, k)
+		v := templateprocessing.GetParameterByName(tpl, k)
 		if v != nil {
 			v.Value = value
 			v.Generate = ""

pkg/template/registry/template/rest.go

@@ -5,13 +5,13 @@ import (
 	"time"
 
 	"github.com/golang/glog"
+	"github.com/openshift/origin/pkg/template/templateprocessing"
 	"k8s.io/apimachinery/pkg/api/errors"
 	"k8s.io/apimachinery/pkg/apis/meta/v1/unstructured"
 	"k8s.io/apimachinery/pkg/runtime"
 	apirequest "k8s.io/apiserver/pkg/endpoints/request"
 	"k8s.io/apiserver/pkg/registry/rest"
 
-	"github.com/openshift/origin/pkg/template"
 	templateapi "github.com/openshift/origin/pkg/template/apis/template"
 	templatevalidation "github.com/openshift/origin/pkg/template/apis/template/validation"
 	"github.com/openshift/origin/pkg/template/generator"
@@ -50,7 +50,7 @@ func (s *REST) Create(ctx apirequest.Context, obj runtime.Object, _ rest.Validat
 	generators := map[string]generator.Generator{
 		"expression": generator.NewExpressionValueGenerator(rand.New(rand.NewSource(time.Now().UnixNano()))),
 	}
-	processor := template.NewProcessor(generators)
+	processor := templateprocessing.NewProcessor(generators)
 	if errs := processor.Process(tpl); len(errs) > 0 {
 		glog.V(1).Infof(errs.ToAggregate().Error())
 		return nil, errors.NewInvalid(templateapi.Kind("Template"), tpl.Name, errs)

pkg/template/registry/templateinstance/strategy.go

@@ -17,7 +17,6 @@ import (
 	rbacregistry "k8s.io/kubernetes/pkg/registry/rbac"
 
 	"github.com/openshift/origin/pkg/authorization/util"
-	template "github.com/openshift/origin/pkg/template"
 	templateapi "github.com/openshift/origin/pkg/template/apis/template"
 	"github.com/openshift/origin/pkg/template/apis/template/validation"
 )
@@ -60,7 +59,7 @@ func (templateInstanceStrategy) PrepareForCreate(ctx apirequest.Context, obj run
 	if templateInstance.Spec.Requester == nil {
 
 		if user, ok := apirequest.UserFrom(ctx); ok {
-			templateReq := template.ConvertUserToTemplateInstanceRequester(user)
+			templateReq := convertUserToTemplateInstanceRequester(user)
 			templateInstance.Spec.Requester = &templateReq
 		}
 	}
@@ -192,3 +191,24 @@ func (statusStrategy) Canonicalize(obj runtime.Object) {
 func (statusStrategy) ValidateUpdate(ctx apirequest.Context, obj, old runtime.Object) field.ErrorList {
 	return validation.ValidateTemplateInstanceUpdate(obj.(*templateapi.TemplateInstance), old.(*templateapi.TemplateInstance))
 }
+
+// ConvertUserToTemplateInstanceRequester copies analogous fields from user.Info to TemplateInstanceRequester
+func convertUserToTemplateInstanceRequester(u user.Info) templateapi.TemplateInstanceRequester {
+	templatereq := templateapi.TemplateInstanceRequester{}
+
+	if u != nil {
+		extra := map[string]templateapi.ExtraValue{}
+		if u.GetExtra() != nil {
+			for k, v := range u.GetExtra() {
+				extra[k] = templateapi.ExtraValue(v)
+			}
+		}
+
+		templatereq.Username = u.GetName()
+		templatereq.UID = u.GetUID()
+		templatereq.Groups = u.GetGroups()
+		templatereq.Extra = extra
+	}
+
+	return templatereq
+}

pkg/template/template.go → pkg/template/templateprocessing/template.go

@@ -1,4 +1,4 @@
-package template
+package templateprocessing
 
 import (
 	"fmt"

pkg/template/template_test.go → pkg/template/templateprocessing/template_test.go

@@ -1,4 +1,4 @@
-package template
+package templateprocessing
 
 import (
 	"encoding/json"