discovery restmapping should always prefer /v1

staging/src/k8s.io/client-go/discovery/restmapper.go

@@ -43,8 +43,9 @@ func NewRESTMapper(groupResources []*APIGroupResources, versionInterfaces meta.V
 	unionMapper := meta.MultiRESTMapper{}
 
 	var groupPriority []string
-	var resourcePriority []schema.GroupVersionResource
-	var kindPriority []schema.GroupVersionKind
+	// /v1 is special.  It should always come first
+	resourcePriority := []schema.GroupVersionResource{{Group: "", Version: "v1", Resource: meta.AnyResource}}
+	kindPriority := []schema.GroupVersionKind{{Group: "", Version: "v1", Kind: meta.AnyKind}}
 
 	for _, group := range groupResources {
 		groupPriority = append(groupPriority, group.Group.Name)

staging/src/k8s.io/client-go/discovery/restmapper_test.go

@@ -37,32 +37,33 @@ func TestRESTMapper(t *testing.T) {
 	resources := []*APIGroupResources{
 		{
 			Group: metav1.APIGroup{
+				Name: "extensions",
 				Versions: []metav1.GroupVersionForDiscovery{
-					{Version: "v1"},
-					{Version: "v2"},
+					{Version: "v1beta"},
 				},
-				PreferredVersion: metav1.GroupVersionForDiscovery{Version: "v1"},
+				PreferredVersion: metav1.GroupVersionForDiscovery{Version: "v1beta"},
 			},
 			VersionedResources: map[string][]metav1.APIResource{
-				"v1": {
-					{Name: "pods", Namespaced: true, Kind: "Pod"},
-				},
-				"v2": {
+				"v1beta": {
+					{Name: "jobs", Namespaced: true, Kind: "Job"},
 					{Name: "pods", Namespaced: true, Kind: "Pod"},
 				},
 			},
 		},
 		{
 			Group: metav1.APIGroup{
-				Name: "extensions",
 				Versions: []metav1.GroupVersionForDiscovery{
-					{Version: "v1beta"},
+					{Version: "v1"},
+					{Version: "v2"},
 				},
-				PreferredVersion: metav1.GroupVersionForDiscovery{Version: "v1beta"},
+				PreferredVersion: metav1.GroupVersionForDiscovery{Version: "v1"},
 			},
 			VersionedResources: map[string][]metav1.APIResource{
-				"v1beta": {
-					{Name: "jobs", Namespaced: true, Kind: "Job"},
+				"v1": {
+					{Name: "pods", Namespaced: true, Kind: "Pod"},
+				},
+				"v2": {
+					{Name: "pods", Namespaced: true, Kind: "Pod"},
 				},
 			},
 		},
@@ -76,6 +77,15 @@ func TestRESTMapper(t *testing.T) {
 	}{
 		{
 			input: schema.GroupVersionResource{
+				Resource: "pods",
+			},
+			want: schema.GroupVersionKind{
+				Version: "v1",
+				Kind:    "Pod",
+			},
+		},
+		{
+			input: schema.GroupVersionResource{
 				Version:  "v1",
 				Resource: "pods",
 			},
@@ -133,6 +143,15 @@ func TestRESTMapper(t *testing.T) {
 	}{
 		{
 			input: schema.GroupVersionResource{
+				Resource: "pods",
+			},
+			want: schema.GroupVersionResource{
+				Version:  "v1",
+				Resource: "pods",
+			},
+		},
+		{
+			input: schema.GroupVersionResource{
 				Version:  "v1",
 				Resource: "pods",
 			},