From a3cbfde4b18fdbddf9c94be80acd4f0468a492b7 Mon Sep 17 00:00:00 2001
From: mbohlool <mehdy@google.com>
Date: Fri, 26 May 2017 17:29:22 -0700
Subject: [PATCH] Make OpenAPI GVK and Action extensions all lower-case

---
 pkg/kubectl/cmd/util/openapi/openapi.go                   | 6 +++---
 .../k8s.io/apimachinery/pkg/apis/meta/v1/group_version.go | 6 +++---
 staging/src/k8s.io/apiserver/pkg/endpoints/installer.go   | 8 ++++++--
 3 files changed, 12 insertions(+), 8 deletions(-)

diff --git a/pkg/kubectl/cmd/util/openapi/openapi.go b/pkg/kubectl/cmd/util/openapi/openapi.go
index 1ab98ee9e01..7ee40945c2a 100644
--- a/pkg/kubectl/cmd/util/openapi/openapi.go
+++ b/pkg/kubectl/cmd/util/openapi/openapi.go
@@ -370,15 +370,15 @@ func (o *Resources) getGroupVersionKind(s spec.Schema) (schema.GroupVersionKind,
 	if !ok {
 		return empty, fmt.Errorf("%s extension has unexpected type %T in %s", groupVersionKindExtensionKey, gvk, s.Extensions)
 	}
-	group, ok := gvkMap["Group"].(string)
+	group, ok := gvkMap["group"].(string)
 	if !ok {
 		return empty, fmt.Errorf("%s extension missing Group: %v", groupVersionKindExtensionKey, gvkMap)
 	}
-	version, ok := gvkMap["Version"].(string)
+	version, ok := gvkMap["version"].(string)
 	if !ok {
 		return empty, fmt.Errorf("%s extension missing Version: %v", groupVersionKindExtensionKey, gvkMap)
 	}
-	kind, ok := gvkMap["Kind"].(string)
+	kind, ok := gvkMap["kind"].(string)
 	if !ok {
 		return empty, fmt.Errorf("%s extension missing Kind: %v", groupVersionKindExtensionKey, gvkMap)
 	}
diff --git a/staging/src/k8s.io/apimachinery/pkg/apis/meta/v1/group_version.go b/staging/src/k8s.io/apimachinery/pkg/apis/meta/v1/group_version.go
index 8b6fdef5a40..bd0e9612a64 100644
--- a/staging/src/k8s.io/apimachinery/pkg/apis/meta/v1/group_version.go
+++ b/staging/src/k8s.io/apimachinery/pkg/apis/meta/v1/group_version.go
@@ -75,9 +75,9 @@ func (gk *GroupKind) String() string {
 //
 // +protobuf.options.(gogoproto.goproto_stringer)=false
 type GroupVersionKind struct {
-	Group   string `protobuf:"bytes,1,opt,name=group"`
-	Version string `protobuf:"bytes,2,opt,name=version"`
-	Kind    string `protobuf:"bytes,3,opt,name=kind"`
+	Group   string `json:"group" protobuf:"bytes,1,opt,name=group"`
+	Version string `json:"version" protobuf:"bytes,2,opt,name=version"`
+	Kind    string `json:"kind" protobuf:"bytes,3,opt,name=kind"`
 }
 
 func (gvk GroupVersionKind) String() string {
diff --git a/staging/src/k8s.io/apiserver/pkg/endpoints/installer.go b/staging/src/k8s.io/apiserver/pkg/endpoints/installer.go
index f7344bdb91a..f4e66bd6ad8 100644
--- a/staging/src/k8s.io/apiserver/pkg/endpoints/installer.go
+++ b/staging/src/k8s.io/apiserver/pkg/endpoints/installer.go
@@ -800,8 +800,12 @@ func (a *APIInstaller) registerResourceHandlers(path string, storage rest.Storag
 			return nil, fmt.Errorf("unrecognized action verb: %s", action.Verb)
 		}
 		for _, route := range routes {
-			route.Metadata(ROUTE_META_GVK, reqScope.Kind)
-			route.Metadata(ROUTE_META_ACTION, action.Verb)
+			route.Metadata(ROUTE_META_GVK, metav1.GroupVersionKind{
+				Group:   reqScope.Kind.Group,
+				Version: reqScope.Kind.Version,
+				Kind:    reqScope.Kind.Kind,
+			})
+			route.Metadata(ROUTE_META_ACTION, strings.ToLower(action.Verb))
 			ws.Route(route)
 		}
 		// Note: update GetAuthorizerAttributes() when adding a custom handler.