From 567bb154325d24b94ac286288da7c404b2c4975d Mon Sep 17 00:00:00 2001
From: Muhammed Uluyol <uluyol@google.com>
Date: Fri, 7 Aug 2015 11:26:44 -0700
Subject: [PATCH] Generate conversions/deep-copies for experimental. Currently
 we make (and register) duplicate functions but this is benign.

---
 .travis.yml                                   |   1 +
 cmd/genconversion/conversion.go               |  26 +-
 cmd/gendeepcopy/deep_copy.go                  |  25 +-
 .../update-generated-conversions.sh           |  25 +-
 .../update-generated-deep-copies.sh           |  48 ++--
 .../verify-generated-conversions.sh           |  42 +--
 .../verify-generated-deep-copies.sh           |  41 +--
 pkg/api/deep_copy_generated.go                |   3 +-
 pkg/api/v1/conversion_generated.go            |   3 +-
 pkg/api/v1/deep_copy_generated.go             |   3 +-
 pkg/expapi/deep_copy.go                       |  19 --
 pkg/expapi/deep_copy_generated.go             | 138 ++++++++++
 pkg/expapi/v1/conversion_generated.go         | 254 ++++++++++++++++++
 pkg/expapi/v1/deep_copy.go                    |  19 --
 pkg/expapi/v1/deep_copy_generated.go          | 139 ++++++++++
 pkg/expapi/v1/register.go                     |   1 -
 shippable.yml                                 |   1 +
 17 files changed, 661 insertions(+), 127 deletions(-)
 delete mode 100644 pkg/expapi/deep_copy.go
 create mode 100644 pkg/expapi/deep_copy_generated.go
 create mode 100644 pkg/expapi/v1/conversion_generated.go
 delete mode 100644 pkg/expapi/v1/deep_copy.go
 create mode 100644 pkg/expapi/v1/deep_copy_generated.go

diff --git a/.travis.yml b/.travis.yml
index 90bcffe0517..64c2f6f2117 100644
--- a/.travis.yml
+++ b/.travis.yml
@@ -8,6 +8,7 @@ matrix:
 
 install:
   - go get github.com/tools/godep
+  - go get golang.org/x/tools/cmd/goimports
   - ./hack/travis/install-etcd.sh
   - ./hack/build-go.sh
 
diff --git a/cmd/genconversion/conversion.go b/cmd/genconversion/conversion.go
index dc178e9a871..461fd10527e 100644
--- a/cmd/genconversion/conversion.go
+++ b/cmd/genconversion/conversion.go
@@ -22,9 +22,12 @@ import (
 	"os"
 	"path"
 	"runtime"
+	"strings"
 
 	"k8s.io/kubernetes/pkg/api"
 	_ "k8s.io/kubernetes/pkg/api/v1"
+	_ "k8s.io/kubernetes/pkg/expapi"
+	_ "k8s.io/kubernetes/pkg/expapi/v1"
 	pkg_runtime "k8s.io/kubernetes/pkg/runtime"
 	"k8s.io/kubernetes/pkg/util"
 
@@ -32,9 +35,11 @@ import (
 	flag "github.com/spf13/pflag"
 )
 
+const pkgBase = "k8s.io/kubernetes/pkg"
+
 var (
 	functionDest = flag.StringP("funcDest", "f", "-", "Output for conversion functions; '-' means stdout")
-	version      = flag.StringP("version", "v", "v1", "Version for conversion.")
+	groupVersion = flag.StringP("version", "v", "api/v1", "groupPath/version for conversion.")
 )
 
 func main() {
@@ -53,13 +58,20 @@ func main() {
 		funcOut = file
 	}
 
-	generator := pkg_runtime.NewConversionGenerator(api.Scheme.Raw(), path.Join("k8s.io/kubernetes/pkg/api", *version))
-	apiShort := generator.AddImport("k8s.io/kubernetes/pkg/api")
-	generator.AddImport("k8s.io/kubernetes/pkg/api/resource")
+	group, version := path.Split(*groupVersion)
+	group = strings.TrimRight(group, "/")
+
+	versionPath := path.Join(pkgBase, group, version)
+	generator := pkg_runtime.NewConversionGenerator(api.Scheme.Raw(), versionPath)
+	apiShort := generator.AddImport(path.Join(pkgBase, "api"))
+	generator.AddImport(path.Join(pkgBase, "api/resource"))
 	// TODO(wojtek-t): Change the overwrites to a flag.
-	generator.OverwritePackage(*version, "")
-	for _, knownType := range api.Scheme.KnownTypes(*version) {
-		if err := generator.GenerateConversionsForType(*version, knownType); err != nil {
+	generator.OverwritePackage(version, "")
+	for _, knownType := range api.Scheme.KnownTypes(version) {
+		if !strings.HasPrefix(knownType.PkgPath(), versionPath) {
+			continue
+		}
+		if err := generator.GenerateConversionsForType(version, knownType); err != nil {
 			glog.Errorf("error while generating conversion functions for %v: %v", knownType, err)
 		}
 	}
diff --git a/cmd/gendeepcopy/deep_copy.go b/cmd/gendeepcopy/deep_copy.go
index 9aace9757fb..fb7fa12f5ff 100644
--- a/cmd/gendeepcopy/deep_copy.go
+++ b/cmd/gendeepcopy/deep_copy.go
@@ -25,6 +25,8 @@ import (
 
 	"k8s.io/kubernetes/pkg/api"
 	_ "k8s.io/kubernetes/pkg/api/v1"
+	_ "k8s.io/kubernetes/pkg/expapi"
+	_ "k8s.io/kubernetes/pkg/expapi/v1"
 	pkg_runtime "k8s.io/kubernetes/pkg/runtime"
 	"k8s.io/kubernetes/pkg/util"
 
@@ -32,9 +34,11 @@ import (
 	flag "github.com/spf13/pflag"
 )
 
+const pkgBase = "k8s.io/kubernetes/pkg"
+
 var (
-	functionDest = flag.StringP("func-dest", "f", "-", "Output for deep copy functions; '-' means stdout")
-	version      = flag.StringP("version", "v", "v1", "Version for deep copies.")
+	functionDest = flag.StringP("funcDest", "f", "-", "Output for deep copy functions; '-' means stdout")
+	groupVersion = flag.StringP("version", "v", "", "groupPath/version for deep copies.")
 	overwrites   = flag.StringP("overwrites", "o", "", "Comma-separated overwrites for package names")
 )
 
@@ -54,15 +58,15 @@ func main() {
 		funcOut = file
 	}
 
-	knownVersion := *version
+	group, version := path.Split(*groupVersion)
+	group = strings.TrimRight(group, "/")
 	registerTo := "api.Scheme"
-	if knownVersion == "api" {
-		knownVersion = api.Scheme.Raw().InternalVersion
+	if *groupVersion == "api/" {
 		registerTo = "Scheme"
 	}
-	pkgPath := path.Join("k8s.io/kubernetes/pkg/api", knownVersion)
-	generator := pkg_runtime.NewDeepCopyGenerator(api.Scheme.Raw(), pkgPath, util.NewStringSet("k8s.io/kubernetes"))
-	generator.AddImport("k8s.io/kubernetes/pkg/api")
+	versionPath := path.Join(pkgBase, group, version)
+	generator := pkg_runtime.NewDeepCopyGenerator(api.Scheme.Raw(), versionPath, util.NewStringSet("k8s.io/kubernetes"))
+	generator.AddImport(path.Join(pkgBase, "api"))
 
 	if len(*overwrites) > 0 {
 		for _, overwrite := range strings.Split(*overwrites, ",") {
@@ -73,7 +77,10 @@ func main() {
 			generator.OverwritePackage(vals[0], vals[1])
 		}
 	}
-	for _, knownType := range api.Scheme.KnownTypes(knownVersion) {
+	for _, knownType := range api.Scheme.KnownTypes(version) {
+		if !strings.HasPrefix(knownType.PkgPath(), versionPath) {
+			continue
+		}
 		if err := generator.AddType(knownType); err != nil {
 			glog.Errorf("error while generating deep copy functions for %v: %v", knownType, err)
 		}
diff --git a/hack/after-build/update-generated-conversions.sh b/hack/after-build/update-generated-conversions.sh
index 318ef754a86..bd9d5c7c1d2 100755
--- a/hack/after-build/update-generated-conversions.sh
+++ b/hack/after-build/update-generated-conversions.sh
@@ -29,11 +29,11 @@ function generate_version() {
 	local version=$1
 	local TMPFILE="/tmp/conversion_generated.$(date +%s).go"
 
-	echo "Generating for version ${version}"
+	echo "Generating for ${version}"
 
 	sed 's/YEAR/2015/' hack/boilerplate/boilerplate.go.txt > "$TMPFILE"
 	cat >> "$TMPFILE" <<EOF
-package ${version}
+package ${version##*/}
 
 // AUTO-GENERATED FUNCTIONS START HERE
 EOF
@@ -44,14 +44,19 @@ EOF
 // AUTO-GENERATED FUNCTIONS END HERE
 EOF
 
-	mv "$TMPFILE" "pkg/api/${version}/conversion_generated.go"
+	env GOPATH=$(godep path):$GOPATH goimports -w "$TMPFILE"
+	mv "$TMPFILE" "pkg/${version}/conversion_generated.go"
 }
 
-VERSIONS="v1"
-for ver in $VERSIONS; do
-  # Ensure that the version being processed is registered by setting
-  # KUBE_API_VERSIONS.
-  KUBE_API_VERSIONS="${ver}" generate_version "${ver}"
-done
+if ! which goimports >/dev/null; then
+	echo "goimports not in path, run go get golang.org/x/tools/cmd/goimports"
+	exit 1
+fi
 
-# ex: ts=2 sw=2 et filetype=sh
+DEFAULT_VERSIONS="api/v1 expapi/v1"
+VERSIONS=${VERSIONS:-$DEFAULT_VERSIONS}
+for ver in $VERSIONS; do
+	# Ensure that the version being processed is registered by setting
+	# KUBE_API_VERSIONS.
+	KUBE_API_VERSIONS="${ver##*/}" generate_version "${ver}"
+done
diff --git a/hack/after-build/update-generated-deep-copies.sh b/hack/after-build/update-generated-deep-copies.sh
index 38d21677959..b848b5e9a75 100755
--- a/hack/after-build/update-generated-deep-copies.sh
+++ b/hack/after-build/update-generated-deep-copies.sh
@@ -27,22 +27,25 @@ gendeepcopy=$(kube::util::find-binary "gendeepcopy")
 
 function result_file_name() {
 	local version=$1
-	if [ "${version}" == "api" ]; then
-		echo "pkg/api/deep_copy_generated.go"
-	else
-		echo "pkg/api/${version}/deep_copy_generated.go"
-	fi
+	echo "pkg/${version}/deep_copy_generated.go"
 }
 
 function generate_version() {
 	local version=$1
 	local TMPFILE="/tmp/deep_copy_generated.$(date +%s).go"
 
-	echo "Generating for version ${version}"
+	echo "Generating for ${version}"
+
+	# version is group/version, so use the version number as the package name unless
+	# this is an internal version, in which case use the group name. 
+	pkgname=${version##*/}
+	if [[ -z $pkgname ]]; then
+		pkgname=${version%/*}
+	fi
 
 	sed 's/YEAR/2015/' hack/boilerplate/boilerplate.go.txt > $TMPFILE
 	cat >> $TMPFILE <<EOF
-package ${version}
+package $pkgname
 
 // AUTO-GENERATED FUNCTIONS START HERE
 EOF
@@ -53,27 +56,24 @@ EOF
 // AUTO-GENERATED FUNCTIONS END HERE
 EOF
 
-	gofmt -w -s "$TMPFILE"
+	env GOPATH=$(godep path):$GOPATH goimports -w "$TMPFILE"
 	mv "$TMPFILE" `result_file_name ${version}`
 }
 
 function generate_deep_copies() {
-  local versions="api v1"
-  # To avoid compile errors, remove the currently existing files.
-  for ver in ${versions}; do
-    rm -f `result_file_name ${ver}`
-  done
-  apiVersions=""
-  for ver in ${versions}; do
-    # Ensure that the version being processed is registered by setting
-    # KUBE_API_VERSIONS.
-    if [ "${ver}" != "api" ]; then
-      apiVersions="${ver}"
-    fi
-    KUBE_API_VERSIONS="${apiVersions}" generate_version "${ver}"
-  done
+	local versions="$@"
+	# To avoid compile errors, remove the currently existing files.
+	for ver in ${versions}; do
+		rm -f `result_file_name ${ver}`
+	done
+	for ver in ${versions}; do
+		# Ensure that the version being processed is registered by setting
+		# KUBE_API_VERSIONS.
+		apiVersions="${ver##*/}"
+		KUBE_API_VERSIONS="${apiVersions}" generate_version "${ver}"
+	done
 }
 
+DEFAULT_VERSIONS="api/ api/v1 expapi/ expapi/v1"
+VERSIONS=${VERSIONS:-$DEFAULT_VERSIONS}
 generate_deep_copies
-
-# ex: ts=2 sw=2 et filetype=sh
diff --git a/hack/after-build/verify-generated-conversions.sh b/hack/after-build/verify-generated-conversions.sh
index b9dafa17018..a41afc4aba0 100755
--- a/hack/after-build/verify-generated-conversions.sh
+++ b/hack/after-build/verify-generated-conversions.sh
@@ -23,25 +23,31 @@ source "${KUBE_ROOT}/hack/lib/init.sh"
 
 kube::golang::setup_env
 
-APIROOT="${KUBE_ROOT}/pkg/api"
-TMP_APIROOT="${KUBE_ROOT}/_tmp/api"
+APIROOTS=${APIROOTS:-pkg/api pkg/expapi}
 _tmp="${KUBE_ROOT}/_tmp"
 
-mkdir -p "${_tmp}"
-cp -a "${APIROOT}" "${TMP_APIROOT}"
+cleanup() {
+	rm -rf "${_tmp}"
+}
+
+trap "cleanup" EXIT SIGINT
+
+for APIROOT in ${APIROOTS}; do
+	mkdir -p "${_tmp}/${APIROOT%/*}"
+	cp -a "${KUBE_ROOT}/${APIROOT}" "${_tmp}/${APIROOT}"
+done
 
 "${KUBE_ROOT}/hack/update-generated-conversions.sh"
-echo "diffing ${APIROOT} against freshly generated conversions"
-ret=0
-diff -Naupr -I 'Auto generated by' "${APIROOT}" "${TMP_APIROOT}" || ret=$?
-cp -a ${TMP_APIROOT} "${KUBE_ROOT}/pkg"
-rm -rf "${_tmp}"
-if [[ $ret -eq 0 ]]
-then
-  echo "${APIROOT} up to date."
-else
-  echo "${APIROOT} is out of date. Please run hack/update-generated-conversions.sh"
-  exit 1
-fi
-
-# ex: ts=2 sw=2 et filetype=sh
+for APIROOT in ${APIROOTS}; do
+	TMP_APIROOT="${_tmp}/${APIROOT}"
+	echo "diffing ${APIROOT} against freshly generated conversions"
+	ret=0
+	diff -Naupr -I 'Auto generated by' "${KUBE_ROOT}/${APIROOT}" "${TMP_APIROOT}" || ret=$?
+	cp -a "${TMP_APIROOT}" "${KUBE_ROOT}/${APIROOT%/*}"
+	if [[ $ret -eq 0 ]]; then
+		echo "${APIROOT} up to date."
+	else
+		echo "${APIROOT} is out of date. Please run hack/update-generated-conversions.sh"
+		exit 1
+	fi
+done
diff --git a/hack/after-build/verify-generated-deep-copies.sh b/hack/after-build/verify-generated-deep-copies.sh
index 16a91baebcd..f677c514bf1 100755
--- a/hack/after-build/verify-generated-deep-copies.sh
+++ b/hack/after-build/verify-generated-deep-copies.sh
@@ -25,25 +25,32 @@ kube::golang::setup_env
 
 gendeepcopy=$(kube::util::find-binary "gendeepcopy")
 
-APIROOT="${KUBE_ROOT}/pkg/api"
-TMP_APIROOT="${KUBE_ROOT}/_tmp/api"
+APIROOTS=${APIROOTS:-pkg/api pkg/expapi}
 _tmp="${KUBE_ROOT}/_tmp"
 
-mkdir -p "${_tmp}"
-cp -a "${APIROOT}" "${TMP_APIROOT}"
+cleanup() {
+	rm -rf "${_tmp}"
+}
+
+trap "cleanup" EXIT SIGINT
+
+for APIROOT in ${APIROOTS}; do
+	mkdir -p "${_tmp}/${APIROOT%/*}"
+	cp -a "${KUBE_ROOT}/${APIROOT}" "${_tmp}/${APIROOT}"
+done
 
 "${KUBE_ROOT}/hack/update-generated-deep-copies.sh"
-echo "diffing ${APIROOT} against freshly generated deep copies"
-ret=0
-diff -Naupr -I 'Auto generated by' "${APIROOT}" "${TMP_APIROOT}" || ret=$?
-cp -a ${TMP_APIROOT} "${KUBE_ROOT}/pkg"
-rm -rf "${_tmp}"
-if [[ $ret -eq 0 ]]
-then
-  echo "${APIROOT} up to date."
-else
-  echo "${APIROOT} is out of date. Please run hack/update-generated-deep-copies.sh"
-  exit 1
-fi
 
-# ex: ts=2 sw=2 et filetype=sh
+for APIROOT in ${APIROOTS}; do
+	TMP_APIROOT="${_tmp}/${APIROOT}"
+	echo "diffing ${APIROOT} against freshly generated deep copies"
+	ret=0
+	diff -Naupr -I 'Auto generated by' "${KUBE_ROOT}/${APIROOT}" "${TMP_APIROOT}" || ret=$?
+	cp -a ${TMP_APIROOT} "${KUBE_ROOT}/${APIROOT%/*}"
+	if [[ $ret -eq 0 ]]; then
+		echo "${APIROOT} up to date."
+	else
+		echo "${APIROOT} is out of date. Please run hack/update-generated-deep-copies.sh"
+		exit 1
+	fi
+done
diff --git a/pkg/api/deep_copy_generated.go b/pkg/api/deep_copy_generated.go
index 15ecb80bb5f..4021137fc1d 100644
--- a/pkg/api/deep_copy_generated.go
+++ b/pkg/api/deep_copy_generated.go
@@ -18,6 +18,8 @@ package api
 
 // AUTO-GENERATED FUNCTIONS START HERE
 import (
+	time "time"
+
 	resource "k8s.io/kubernetes/pkg/api/resource"
 	conversion "k8s.io/kubernetes/pkg/conversion"
 	fields "k8s.io/kubernetes/pkg/fields"
@@ -25,7 +27,6 @@ import (
 	runtime "k8s.io/kubernetes/pkg/runtime"
 	util "k8s.io/kubernetes/pkg/util"
 	inf "speter.net/go/exp/math/dec/inf"
-	time "time"
 )
 
 func deepCopy_api_AWSElasticBlockStoreVolumeSource(in AWSElasticBlockStoreVolumeSource, out *AWSElasticBlockStoreVolumeSource, c *conversion.Cloner) error {
diff --git a/pkg/api/v1/conversion_generated.go b/pkg/api/v1/conversion_generated.go
index 2328de7b1b4..e94609a6a12 100644
--- a/pkg/api/v1/conversion_generated.go
+++ b/pkg/api/v1/conversion_generated.go
@@ -18,10 +18,11 @@ package v1
 
 // AUTO-GENERATED FUNCTIONS START HERE
 import (
+	reflect "reflect"
+
 	api "k8s.io/kubernetes/pkg/api"
 	resource "k8s.io/kubernetes/pkg/api/resource"
 	conversion "k8s.io/kubernetes/pkg/conversion"
-	reflect "reflect"
 )
 
 func convert_api_AWSElasticBlockStoreVolumeSource_To_v1_AWSElasticBlockStoreVolumeSource(in *api.AWSElasticBlockStoreVolumeSource, out *AWSElasticBlockStoreVolumeSource, s conversion.Scope) error {
diff --git a/pkg/api/v1/deep_copy_generated.go b/pkg/api/v1/deep_copy_generated.go
index 192b44d48cc..cd710336ed6 100644
--- a/pkg/api/v1/deep_copy_generated.go
+++ b/pkg/api/v1/deep_copy_generated.go
@@ -18,13 +18,14 @@ package v1
 
 // AUTO-GENERATED FUNCTIONS START HERE
 import (
+	time "time"
+
 	api "k8s.io/kubernetes/pkg/api"
 	resource "k8s.io/kubernetes/pkg/api/resource"
 	conversion "k8s.io/kubernetes/pkg/conversion"
 	runtime "k8s.io/kubernetes/pkg/runtime"
 	util "k8s.io/kubernetes/pkg/util"
 	inf "speter.net/go/exp/math/dec/inf"
-	time "time"
 )
 
 func deepCopy_resource_Quantity(in resource.Quantity, out *resource.Quantity, c *conversion.Cloner) error {
diff --git a/pkg/expapi/deep_copy.go b/pkg/expapi/deep_copy.go
deleted file mode 100644
index 2becaae4d02..00000000000
--- a/pkg/expapi/deep_copy.go
+++ /dev/null
@@ -1,19 +0,0 @@
-/*
-Copyright 2015 The Kubernetes Authors All rights reserved.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-package expapi
-
-func addDeepCopyFuncs() {}
diff --git a/pkg/expapi/deep_copy_generated.go b/pkg/expapi/deep_copy_generated.go
new file mode 100644
index 00000000000..da038264d97
--- /dev/null
+++ b/pkg/expapi/deep_copy_generated.go
@@ -0,0 +1,138 @@
+/*
+Copyright 2015 The Kubernetes Authors All rights reserved.
+
+Licensed under the Apache License, Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
+
+package expapi
+
+// AUTO-GENERATED FUNCTIONS START HERE
+import (
+	time "time"
+
+	api "k8s.io/kubernetes/pkg/api"
+	conversion "k8s.io/kubernetes/pkg/conversion"
+	util "k8s.io/kubernetes/pkg/util"
+)
+
+func deepCopy_api_ObjectMeta(in api.ObjectMeta, out *api.ObjectMeta, c *conversion.Cloner) error {
+	out.Name = in.Name
+	out.GenerateName = in.GenerateName
+	out.Namespace = in.Namespace
+	out.SelfLink = in.SelfLink
+	out.UID = in.UID
+	out.ResourceVersion = in.ResourceVersion
+	out.Generation = in.Generation
+	if err := deepCopy_util_Time(in.CreationTimestamp, &out.CreationTimestamp, c); err != nil {
+		return err
+	}
+	if in.DeletionTimestamp != nil {
+		out.DeletionTimestamp = new(util.Time)
+		if err := deepCopy_util_Time(*in.DeletionTimestamp, out.DeletionTimestamp, c); err != nil {
+			return err
+		}
+	} else {
+		out.DeletionTimestamp = nil
+	}
+	if in.Labels != nil {
+		out.Labels = make(map[string]string)
+		for key, val := range in.Labels {
+			out.Labels[key] = val
+		}
+	} else {
+		out.Labels = nil
+	}
+	if in.Annotations != nil {
+		out.Annotations = make(map[string]string)
+		for key, val := range in.Annotations {
+			out.Annotations[key] = val
+		}
+	} else {
+		out.Annotations = nil
+	}
+	return nil
+}
+
+func deepCopy_api_TypeMeta(in api.TypeMeta, out *api.TypeMeta, c *conversion.Cloner) error {
+	out.Kind = in.Kind
+	out.APIVersion = in.APIVersion
+	return nil
+}
+
+func deepCopy_expapi_ReplicationControllerDummy(in ReplicationControllerDummy, out *ReplicationControllerDummy, c *conversion.Cloner) error {
+	if err := deepCopy_api_TypeMeta(in.TypeMeta, &out.TypeMeta, c); err != nil {
+		return err
+	}
+	return nil
+}
+
+func deepCopy_expapi_Scale(in Scale, out *Scale, c *conversion.Cloner) error {
+	if err := deepCopy_api_TypeMeta(in.TypeMeta, &out.TypeMeta, c); err != nil {
+		return err
+	}
+	if err := deepCopy_api_ObjectMeta(in.ObjectMeta, &out.ObjectMeta, c); err != nil {
+		return err
+	}
+	if err := deepCopy_expapi_ScaleSpec(in.Spec, &out.Spec, c); err != nil {
+		return err
+	}
+	if err := deepCopy_expapi_ScaleStatus(in.Status, &out.Status, c); err != nil {
+		return err
+	}
+	return nil
+}
+
+func deepCopy_expapi_ScaleSpec(in ScaleSpec, out *ScaleSpec, c *conversion.Cloner) error {
+	out.Replicas = in.Replicas
+	return nil
+}
+
+func deepCopy_expapi_ScaleStatus(in ScaleStatus, out *ScaleStatus, c *conversion.Cloner) error {
+	out.Replicas = in.Replicas
+	if in.Selector != nil {
+		out.Selector = make(map[string]string)
+		for key, val := range in.Selector {
+			out.Selector[key] = val
+		}
+	} else {
+		out.Selector = nil
+	}
+	return nil
+}
+
+func deepCopy_util_Time(in util.Time, out *util.Time, c *conversion.Cloner) error {
+	if newVal, err := c.DeepCopy(in.Time); err != nil {
+		return err
+	} else {
+		out.Time = newVal.(time.Time)
+	}
+	return nil
+}
+
+func init() {
+	err := api.Scheme.AddGeneratedDeepCopyFuncs(
+		deepCopy_api_ObjectMeta,
+		deepCopy_api_TypeMeta,
+		deepCopy_expapi_ReplicationControllerDummy,
+		deepCopy_expapi_Scale,
+		deepCopy_expapi_ScaleSpec,
+		deepCopy_expapi_ScaleStatus,
+		deepCopy_util_Time,
+	)
+	if err != nil {
+		// if one of the deep copy functions is malformed, detect it immediately.
+		panic(err)
+	}
+}
+
+// AUTO-GENERATED FUNCTIONS END HERE
diff --git a/pkg/expapi/v1/conversion_generated.go b/pkg/expapi/v1/conversion_generated.go
new file mode 100644
index 00000000000..259daf0597f
--- /dev/null
+++ b/pkg/expapi/v1/conversion_generated.go
@@ -0,0 +1,254 @@
+/*
+Copyright 2015 The Kubernetes Authors All rights reserved.
+
+Licensed under the Apache License, Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
+
+package v1
+
+// AUTO-GENERATED FUNCTIONS START HERE
+import (
+	reflect "reflect"
+
+	api "k8s.io/kubernetes/pkg/api"
+	v1 "k8s.io/kubernetes/pkg/api/v1"
+	conversion "k8s.io/kubernetes/pkg/conversion"
+	expapi "k8s.io/kubernetes/pkg/expapi"
+)
+
+func convert_api_ObjectMeta_To_v1_ObjectMeta(in *api.ObjectMeta, out *v1.ObjectMeta, s conversion.Scope) error {
+	if defaulting, found := s.DefaultingInterface(reflect.TypeOf(*in)); found {
+		defaulting.(func(*api.ObjectMeta))(in)
+	}
+	out.Name = in.Name
+	out.GenerateName = in.GenerateName
+	out.Namespace = in.Namespace
+	out.SelfLink = in.SelfLink
+	out.UID = in.UID
+	out.ResourceVersion = in.ResourceVersion
+	out.Generation = in.Generation
+	if err := s.Convert(&in.CreationTimestamp, &out.CreationTimestamp, 0); err != nil {
+		return err
+	}
+	if in.DeletionTimestamp != nil {
+		if err := s.Convert(&in.DeletionTimestamp, &out.DeletionTimestamp, 0); err != nil {
+			return err
+		}
+	} else {
+		out.DeletionTimestamp = nil
+	}
+	if in.Labels != nil {
+		out.Labels = make(map[string]string)
+		for key, val := range in.Labels {
+			out.Labels[key] = val
+		}
+	} else {
+		out.Labels = nil
+	}
+	if in.Annotations != nil {
+		out.Annotations = make(map[string]string)
+		for key, val := range in.Annotations {
+			out.Annotations[key] = val
+		}
+	} else {
+		out.Annotations = nil
+	}
+	return nil
+}
+
+func convert_api_TypeMeta_To_v1_TypeMeta(in *api.TypeMeta, out *v1.TypeMeta, s conversion.Scope) error {
+	if defaulting, found := s.DefaultingInterface(reflect.TypeOf(*in)); found {
+		defaulting.(func(*api.TypeMeta))(in)
+	}
+	out.Kind = in.Kind
+	out.APIVersion = in.APIVersion
+	return nil
+}
+
+func convert_v1_ObjectMeta_To_api_ObjectMeta(in *v1.ObjectMeta, out *api.ObjectMeta, s conversion.Scope) error {
+	if defaulting, found := s.DefaultingInterface(reflect.TypeOf(*in)); found {
+		defaulting.(func(*v1.ObjectMeta))(in)
+	}
+	out.Name = in.Name
+	out.GenerateName = in.GenerateName
+	out.Namespace = in.Namespace
+	out.SelfLink = in.SelfLink
+	out.UID = in.UID
+	out.ResourceVersion = in.ResourceVersion
+	out.Generation = in.Generation
+	if err := s.Convert(&in.CreationTimestamp, &out.CreationTimestamp, 0); err != nil {
+		return err
+	}
+	if in.DeletionTimestamp != nil {
+		if err := s.Convert(&in.DeletionTimestamp, &out.DeletionTimestamp, 0); err != nil {
+			return err
+		}
+	} else {
+		out.DeletionTimestamp = nil
+	}
+	if in.Labels != nil {
+		out.Labels = make(map[string]string)
+		for key, val := range in.Labels {
+			out.Labels[key] = val
+		}
+	} else {
+		out.Labels = nil
+	}
+	if in.Annotations != nil {
+		out.Annotations = make(map[string]string)
+		for key, val := range in.Annotations {
+			out.Annotations[key] = val
+		}
+	} else {
+		out.Annotations = nil
+	}
+	return nil
+}
+
+func convert_v1_TypeMeta_To_api_TypeMeta(in *v1.TypeMeta, out *api.TypeMeta, s conversion.Scope) error {
+	if defaulting, found := s.DefaultingInterface(reflect.TypeOf(*in)); found {
+		defaulting.(func(*v1.TypeMeta))(in)
+	}
+	out.Kind = in.Kind
+	out.APIVersion = in.APIVersion
+	return nil
+}
+
+func convert_expapi_ReplicationControllerDummy_To_v1_ReplicationControllerDummy(in *expapi.ReplicationControllerDummy, out *ReplicationControllerDummy, s conversion.Scope) error {
+	if defaulting, found := s.DefaultingInterface(reflect.TypeOf(*in)); found {
+		defaulting.(func(*expapi.ReplicationControllerDummy))(in)
+	}
+	if err := convert_api_TypeMeta_To_v1_TypeMeta(&in.TypeMeta, &out.TypeMeta, s); err != nil {
+		return err
+	}
+	return nil
+}
+
+func convert_expapi_Scale_To_v1_Scale(in *expapi.Scale, out *Scale, s conversion.Scope) error {
+	if defaulting, found := s.DefaultingInterface(reflect.TypeOf(*in)); found {
+		defaulting.(func(*expapi.Scale))(in)
+	}
+	if err := convert_api_TypeMeta_To_v1_TypeMeta(&in.TypeMeta, &out.TypeMeta, s); err != nil {
+		return err
+	}
+	if err := convert_api_ObjectMeta_To_v1_ObjectMeta(&in.ObjectMeta, &out.ObjectMeta, s); err != nil {
+		return err
+	}
+	if err := convert_expapi_ScaleSpec_To_v1_ScaleSpec(&in.Spec, &out.Spec, s); err != nil {
+		return err
+	}
+	if err := convert_expapi_ScaleStatus_To_v1_ScaleStatus(&in.Status, &out.Status, s); err != nil {
+		return err
+	}
+	return nil
+}
+
+func convert_expapi_ScaleSpec_To_v1_ScaleSpec(in *expapi.ScaleSpec, out *ScaleSpec, s conversion.Scope) error {
+	if defaulting, found := s.DefaultingInterface(reflect.TypeOf(*in)); found {
+		defaulting.(func(*expapi.ScaleSpec))(in)
+	}
+	out.Replicas = in.Replicas
+	return nil
+}
+
+func convert_expapi_ScaleStatus_To_v1_ScaleStatus(in *expapi.ScaleStatus, out *ScaleStatus, s conversion.Scope) error {
+	if defaulting, found := s.DefaultingInterface(reflect.TypeOf(*in)); found {
+		defaulting.(func(*expapi.ScaleStatus))(in)
+	}
+	out.Replicas = in.Replicas
+	if in.Selector != nil {
+		out.Selector = make(map[string]string)
+		for key, val := range in.Selector {
+			out.Selector[key] = val
+		}
+	} else {
+		out.Selector = nil
+	}
+	return nil
+}
+
+func convert_v1_ReplicationControllerDummy_To_expapi_ReplicationControllerDummy(in *ReplicationControllerDummy, out *expapi.ReplicationControllerDummy, s conversion.Scope) error {
+	if defaulting, found := s.DefaultingInterface(reflect.TypeOf(*in)); found {
+		defaulting.(func(*ReplicationControllerDummy))(in)
+	}
+	if err := convert_v1_TypeMeta_To_api_TypeMeta(&in.TypeMeta, &out.TypeMeta, s); err != nil {
+		return err
+	}
+	return nil
+}
+
+func convert_v1_Scale_To_expapi_Scale(in *Scale, out *expapi.Scale, s conversion.Scope) error {
+	if defaulting, found := s.DefaultingInterface(reflect.TypeOf(*in)); found {
+		defaulting.(func(*Scale))(in)
+	}
+	if err := convert_v1_TypeMeta_To_api_TypeMeta(&in.TypeMeta, &out.TypeMeta, s); err != nil {
+		return err
+	}
+	if err := convert_v1_ObjectMeta_To_api_ObjectMeta(&in.ObjectMeta, &out.ObjectMeta, s); err != nil {
+		return err
+	}
+	if err := convert_v1_ScaleSpec_To_expapi_ScaleSpec(&in.Spec, &out.Spec, s); err != nil {
+		return err
+	}
+	if err := convert_v1_ScaleStatus_To_expapi_ScaleStatus(&in.Status, &out.Status, s); err != nil {
+		return err
+	}
+	return nil
+}
+
+func convert_v1_ScaleSpec_To_expapi_ScaleSpec(in *ScaleSpec, out *expapi.ScaleSpec, s conversion.Scope) error {
+	if defaulting, found := s.DefaultingInterface(reflect.TypeOf(*in)); found {
+		defaulting.(func(*ScaleSpec))(in)
+	}
+	out.Replicas = in.Replicas
+	return nil
+}
+
+func convert_v1_ScaleStatus_To_expapi_ScaleStatus(in *ScaleStatus, out *expapi.ScaleStatus, s conversion.Scope) error {
+	if defaulting, found := s.DefaultingInterface(reflect.TypeOf(*in)); found {
+		defaulting.(func(*ScaleStatus))(in)
+	}
+	out.Replicas = in.Replicas
+	if in.Selector != nil {
+		out.Selector = make(map[string]string)
+		for key, val := range in.Selector {
+			out.Selector[key] = val
+		}
+	} else {
+		out.Selector = nil
+	}
+	return nil
+}
+
+func init() {
+	err := api.Scheme.AddGeneratedConversionFuncs(
+		convert_api_ObjectMeta_To_v1_ObjectMeta,
+		convert_api_TypeMeta_To_v1_TypeMeta,
+		convert_expapi_ReplicationControllerDummy_To_v1_ReplicationControllerDummy,
+		convert_expapi_ScaleSpec_To_v1_ScaleSpec,
+		convert_expapi_ScaleStatus_To_v1_ScaleStatus,
+		convert_expapi_Scale_To_v1_Scale,
+		convert_v1_ObjectMeta_To_api_ObjectMeta,
+		convert_v1_ReplicationControllerDummy_To_expapi_ReplicationControllerDummy,
+		convert_v1_ScaleSpec_To_expapi_ScaleSpec,
+		convert_v1_ScaleStatus_To_expapi_ScaleStatus,
+		convert_v1_Scale_To_expapi_Scale,
+		convert_v1_TypeMeta_To_api_TypeMeta,
+	)
+	if err != nil {
+		// If one of the conversion functions is malformed, detect it immediately.
+		panic(err)
+	}
+}
+
+// AUTO-GENERATED FUNCTIONS END HERE
diff --git a/pkg/expapi/v1/deep_copy.go b/pkg/expapi/v1/deep_copy.go
deleted file mode 100644
index af00429d520..00000000000
--- a/pkg/expapi/v1/deep_copy.go
+++ /dev/null
@@ -1,19 +0,0 @@
-/*
-Copyright 2015 The Kubernetes Authors All rights reserved.
-
-Licensed under the Apache License, Version 2.0 (the "License");
-you may not use this file except in compliance with the License.
-You may obtain a copy of the License at
-
-    http://www.apache.org/licenses/LICENSE-2.0
-
-Unless required by applicable law or agreed to in writing, software
-distributed under the License is distributed on an "AS IS" BASIS,
-WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
-See the License for the specific language governing permissions and
-limitations under the License.
-*/
-
-package v1
-
-func addDeepCopyFuncs() {}
diff --git a/pkg/expapi/v1/deep_copy_generated.go b/pkg/expapi/v1/deep_copy_generated.go
new file mode 100644
index 00000000000..c60f6ffcc13
--- /dev/null
+++ b/pkg/expapi/v1/deep_copy_generated.go
@@ -0,0 +1,139 @@
+/*
+Copyright 2015 The Kubernetes Authors All rights reserved.
+
+Licensed under the Apache License, Version 2.0 (the "License");
+you may not use this file except in compliance with the License.
+You may obtain a copy of the License at
+
+    http://www.apache.org/licenses/LICENSE-2.0
+
+Unless required by applicable law or agreed to in writing, software
+distributed under the License is distributed on an "AS IS" BASIS,
+WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
+See the License for the specific language governing permissions and
+limitations under the License.
+*/
+
+package v1
+
+// AUTO-GENERATED FUNCTIONS START HERE
+import (
+	time "time"
+
+	api "k8s.io/kubernetes/pkg/api"
+	v1 "k8s.io/kubernetes/pkg/api/v1"
+	conversion "k8s.io/kubernetes/pkg/conversion"
+	util "k8s.io/kubernetes/pkg/util"
+)
+
+func deepCopy_v1_ObjectMeta(in v1.ObjectMeta, out *v1.ObjectMeta, c *conversion.Cloner) error {
+	out.Name = in.Name
+	out.GenerateName = in.GenerateName
+	out.Namespace = in.Namespace
+	out.SelfLink = in.SelfLink
+	out.UID = in.UID
+	out.ResourceVersion = in.ResourceVersion
+	out.Generation = in.Generation
+	if err := deepCopy_util_Time(in.CreationTimestamp, &out.CreationTimestamp, c); err != nil {
+		return err
+	}
+	if in.DeletionTimestamp != nil {
+		out.DeletionTimestamp = new(util.Time)
+		if err := deepCopy_util_Time(*in.DeletionTimestamp, out.DeletionTimestamp, c); err != nil {
+			return err
+		}
+	} else {
+		out.DeletionTimestamp = nil
+	}
+	if in.Labels != nil {
+		out.Labels = make(map[string]string)
+		for key, val := range in.Labels {
+			out.Labels[key] = val
+		}
+	} else {
+		out.Labels = nil
+	}
+	if in.Annotations != nil {
+		out.Annotations = make(map[string]string)
+		for key, val := range in.Annotations {
+			out.Annotations[key] = val
+		}
+	} else {
+		out.Annotations = nil
+	}
+	return nil
+}
+
+func deepCopy_v1_TypeMeta(in v1.TypeMeta, out *v1.TypeMeta, c *conversion.Cloner) error {
+	out.Kind = in.Kind
+	out.APIVersion = in.APIVersion
+	return nil
+}
+
+func deepCopy_v1_ReplicationControllerDummy(in ReplicationControllerDummy, out *ReplicationControllerDummy, c *conversion.Cloner) error {
+	if err := deepCopy_v1_TypeMeta(in.TypeMeta, &out.TypeMeta, c); err != nil {
+		return err
+	}
+	return nil
+}
+
+func deepCopy_v1_Scale(in Scale, out *Scale, c *conversion.Cloner) error {
+	if err := deepCopy_v1_TypeMeta(in.TypeMeta, &out.TypeMeta, c); err != nil {
+		return err
+	}
+	if err := deepCopy_v1_ObjectMeta(in.ObjectMeta, &out.ObjectMeta, c); err != nil {
+		return err
+	}
+	if err := deepCopy_v1_ScaleSpec(in.Spec, &out.Spec, c); err != nil {
+		return err
+	}
+	if err := deepCopy_v1_ScaleStatus(in.Status, &out.Status, c); err != nil {
+		return err
+	}
+	return nil
+}
+
+func deepCopy_v1_ScaleSpec(in ScaleSpec, out *ScaleSpec, c *conversion.Cloner) error {
+	out.Replicas = in.Replicas
+	return nil
+}
+
+func deepCopy_v1_ScaleStatus(in ScaleStatus, out *ScaleStatus, c *conversion.Cloner) error {
+	out.Replicas = in.Replicas
+	if in.Selector != nil {
+		out.Selector = make(map[string]string)
+		for key, val := range in.Selector {
+			out.Selector[key] = val
+		}
+	} else {
+		out.Selector = nil
+	}
+	return nil
+}
+
+func deepCopy_util_Time(in util.Time, out *util.Time, c *conversion.Cloner) error {
+	if newVal, err := c.DeepCopy(in.Time); err != nil {
+		return err
+	} else {
+		out.Time = newVal.(time.Time)
+	}
+	return nil
+}
+
+func init() {
+	err := api.Scheme.AddGeneratedDeepCopyFuncs(
+		deepCopy_v1_ObjectMeta,
+		deepCopy_v1_TypeMeta,
+		deepCopy_v1_ReplicationControllerDummy,
+		deepCopy_v1_Scale,
+		deepCopy_v1_ScaleSpec,
+		deepCopy_v1_ScaleStatus,
+		deepCopy_util_Time,
+	)
+	if err != nil {
+		// if one of the deep copy functions is malformed, detect it immediately.
+		panic(err)
+	}
+}
+
+// AUTO-GENERATED FUNCTIONS END HERE
diff --git a/pkg/expapi/v1/register.go b/pkg/expapi/v1/register.go
index 882359d7b4c..19da3fea772 100644
--- a/pkg/expapi/v1/register.go
+++ b/pkg/expapi/v1/register.go
@@ -24,7 +24,6 @@ import (
 var Codec = runtime.CodecFor(api.Scheme, "v1")
 
 func init() {
-	addDeepCopyFuncs()
 	addConversionFuncs()
 	addDefaultingFuncs()
 	addKnownTypes()
diff --git a/shippable.yml b/shippable.yml
index c712401fcc6..4a147c1f161 100644
--- a/shippable.yml
+++ b/shippable.yml
@@ -21,6 +21,7 @@ before_install:
 
 install:
   - go get golang.org/x/tools/cmd/cover
+  - go get golang.org/x/tools/cmd/goimports
   - go get github.com/mattn/goveralls
   - go get github.com/tools/godep
   - ./hack/travis/install-etcd.sh