From 893b740ea0a4a506f64c7933803caa120a948416 Mon Sep 17 00:00:00 2001
From: jackgr <jackgr@google.com>
Date: Tue, 22 Mar 2016 16:01:02 -0700
Subject: [PATCH] Move kubernetes types out of common

---
 pkg/common/types.go                         | 16 ----------------
 pkg/registry/secrets_credential_provider.go |  4 ++--
 pkg/repo/secrets_credential_provider.go     |  5 ++---
 pkg/util/kubernetes.go                      | 16 ++++++++++++++++
 pkg/util/kubernetesutil.go                  |  2 +-
 5 files changed, 21 insertions(+), 22 deletions(-)

diff --git a/pkg/common/types.go b/pkg/common/types.go
index 5d943e3dd..b9d923b30 100644
--- a/pkg/common/types.go
+++ b/pkg/common/types.go
@@ -180,22 +180,6 @@ type TypeInstance struct {
 	Path       string `json:"path"`       // JSON path within manifest
 }
 
-// KubernetesObject represents a native 'bare' Kubernetes object.
-type KubernetesObject struct {
-	Kind       string                 `json:"kind"`
-	APIVersion string                 `json:"apiVersion"`
-	Metadata   map[string]interface{} `json:"metadata"`
-	Spec       map[string]interface{} `json:"spec"`
-}
-
-// KubernetesSecret represents a Kubernetes secret
-type KubernetesSecret struct {
-	Kind       string            `json:"kind"`
-	APIVersion string            `json:"apiVersion"`
-	Metadata   map[string]string `json:"metadata"`
-	Data       map[string]string `json:"data,omitempty"`
-}
-
 // TODO: Remove the remainder of this file when the refactoring of pkg/registry is complete.
 
 // BasicAuthCredential holds a username and password.
diff --git a/pkg/registry/secrets_credential_provider.go b/pkg/registry/secrets_credential_provider.go
index 03c217417..2adaaf725 100644
--- a/pkg/registry/secrets_credential_provider.go
+++ b/pkg/registry/secrets_credential_provider.go
@@ -71,7 +71,7 @@ func NewSecretsCredentialProvider() common.CredentialProvider {
 }
 
 func parseCredential(credential string) (*common.RegistryCredential, error) {
-	var c common.KubernetesSecret
+	var c util.KubernetesSecret
 	if err := json.Unmarshal([]byte(credential), &c); err != nil {
 		return nil, fmt.Errorf("cannot unmarshal credential '%s' (%#v)", credential, err)
 	}
@@ -112,7 +112,7 @@ func (scp *SecretsCredentialProvider) SetCredential(name string, credential *com
 	metadata["name"] = name
 	data := make(map[string]string)
 	data["credential"] = enc
-	obj := &common.KubernetesSecret{
+	obj := &util.KubernetesSecret{
 		Kind:       secretType,
 		APIVersion: "v1",
 		Metadata:   metadata,
diff --git a/pkg/repo/secrets_credential_provider.go b/pkg/repo/secrets_credential_provider.go
index 6dfe0491f..ffd029a3f 100644
--- a/pkg/repo/secrets_credential_provider.go
+++ b/pkg/repo/secrets_credential_provider.go
@@ -24,7 +24,6 @@ import (
 	"log"
 
 	"github.com/ghodss/yaml"
-	"github.com/kubernetes/helm/pkg/common"
 	"github.com/kubernetes/helm/pkg/util"
 )
 
@@ -71,7 +70,7 @@ func NewSecretsCredentialProvider() ICredentialProvider {
 }
 
 func parseCredential(credential string) (*Credential, error) {
-	var c common.KubernetesSecret
+	var c util.KubernetesSecret
 	if err := json.Unmarshal([]byte(credential), &c); err != nil {
 		return nil, fmt.Errorf("cannot unmarshal credential (%s): %s", credential, err)
 	}
@@ -116,7 +115,7 @@ func (scp *SecretsCredentialProvider) SetCredential(name string, credential *Cre
 	metadata["name"] = name
 	data := make(map[string]string)
 	data["credential"] = enc
-	obj := &common.KubernetesSecret{
+	obj := &util.KubernetesSecret{
 		Kind:       secretType,
 		APIVersion: "v1",
 		Metadata:   metadata,
diff --git a/pkg/util/kubernetes.go b/pkg/util/kubernetes.go
index 1e59b7bec..81bc3aa23 100644
--- a/pkg/util/kubernetes.go
+++ b/pkg/util/kubernetes.go
@@ -40,3 +40,19 @@ type Kubernetes interface {
 	Delete(resource string) (string, error)
 	Replace(resource string) (string, error)
 }
+
+// KubernetesObject represents a native 'bare' Kubernetes object.
+type KubernetesObject struct {
+	Kind       string                 `json:"kind"`
+	APIVersion string                 `json:"apiVersion"`
+	Metadata   map[string]interface{} `json:"metadata"`
+	Spec       map[string]interface{} `json:"spec"`
+}
+
+// KubernetesSecret represents a Kubernetes secret
+type KubernetesSecret struct {
+	Kind       string            `json:"kind"`
+	APIVersion string            `json:"apiVersion"`
+	Metadata   map[string]string `json:"metadata"`
+	Data       map[string]string `json:"data,omitempty"`
+}
diff --git a/pkg/util/kubernetesutil.go b/pkg/util/kubernetesutil.go
index 581bd97a2..b1323a20d 100644
--- a/pkg/util/kubernetesutil.go
+++ b/pkg/util/kubernetesutil.go
@@ -26,7 +26,7 @@ import (
 
 // ParseKubernetesObject parses a Kubernetes API object in YAML format.
 func ParseKubernetesObject(object []byte) (*common.Resource, error) {
-	o := &common.KubernetesObject{}
+	o := &KubernetesObject{}
 	if err := yaml.Unmarshal(object, &o); err != nil {
 		return nil, fmt.Errorf("cannot unmarshal native kubernetes object (%#v)", err)
 	}