github/client-go
1
0
Fork 0
mirror of https://github.com/kubernetes/client-go.git synced 2025-07-20 10:00:23 +00:00
Code Issues Projects Releases Wiki Activity

discovery restmapping should always prefer /v1

Browse Source 
Kubernetes-commit: 49f8a37c6488c40c6533cc908467b05038c7ef24
This commit is contained in:
deads2k 2017-02-28 14:31:31 -05:00 committed by Kubernetes Publisher
parent 1f37358b93
commit 7395acc6da
2 changed files with 36 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
discovery/restmapper.go
View File

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

47
discovery/restmapper_test.go
View File

@ -35,6 +35,21 @@ import (
func TestRESTMapper(t *testing.T) { func TestRESTMapper(t *testing.T) {
resources := []*APIGroupResources{ resources := []*APIGroupResources{
{
Group: metav1.APIGroup{
Name: "extensions",
Versions: []metav1.GroupVersionForDiscovery{
{Version: "v1beta"},
},
PreferredVersion: metav1.GroupVersionForDiscovery{Version: "v1beta"},
},
VersionedResources: map[string][]metav1.APIResource{
"v1beta": {
{Name: "jobs", Namespaced: true, Kind: "Job"},
{Name: "pods", Namespaced: true, Kind: "Pod"},
},
},
},
{ {
Group: metav1.APIGroup{ Group: metav1.APIGroup{
Versions: []metav1.GroupVersionForDiscovery{ Versions: []metav1.GroupVersionForDiscovery{
@ -52,20 +67,6 @@ func TestRESTMapper(t *testing.T) {
}, },
}, },
}, },
{
Group: metav1.APIGroup{
Name: "extensions",
Versions: []metav1.GroupVersionForDiscovery{
{Version: "v1beta"},
},
PreferredVersion: metav1.GroupVersionForDiscovery{Version: "v1beta"},
},
VersionedResources: map[string][]metav1.APIResource{
"v1beta": {
{Name: "jobs", Namespaced: true, Kind: "Job"},
},
},
},
} }
restMapper := NewRESTMapper(resources, nil) restMapper := NewRESTMapper(resources, nil)
@ -74,6 +75,15 @@ func TestRESTMapper(t *testing.T) {
input schema.GroupVersionResource input schema.GroupVersionResource
want schema.GroupVersionKind want schema.GroupVersionKind
}{ }{
{
input: schema.GroupVersionResource{
Resource: "pods",
},
want: schema.GroupVersionKind{
Version: "v1",
Kind: "Pod",
},
},
{ {
input: schema.GroupVersionResource{ input: schema.GroupVersionResource{
Version: "v1", Version: "v1",
@ -131,6 +141,15 @@ func TestRESTMapper(t *testing.T) {
input schema.GroupVersionResource input schema.GroupVersionResource
want schema.GroupVersionResource want schema.GroupVersionResource
}{ }{
{
input: schema.GroupVersionResource{
Resource: "pods",
},
want: schema.GroupVersionResource{
Version: "v1",
Resource: "pods",
},
},
{ {
input: schema.GroupVersionResource{ input: schema.GroupVersionResource{
Version: "v1", Version: "v1",