Reorganize the output of "kubectl get -o json"

"resourceVersion" and "selfLink" are members of "metadata" rather than the list. This change reorganize the struct to provide consistency with other places. Before this change: "kind": "List", "metadata": {}, "resourceVersion": "", "selfLink": "" After this change: "kind": "List", "metadata": { "resourceVersion": "", "selfLink": "" } Reference: https://github.com/kubernetes/kubernetes/blob/master/staging/src/k8s.io/apimachinery/pkg/apis/meta/v1/types.go#L57

Reorganize the output of "kubectl get -o json"
a6db26d5 · Cao Shufeng · b4c76c6a · a6db26d5 · a6db26d5
Commit a6db26d5 authored Mar 31, 2017 by Cao Shufeng
Hide whitespace changes
Inline Side-by-side

Showing with 5 additions and 3 deletions

get.go pkg/kubectl/cmd/get.go +4 -2

get_test.go pkg/kubectl/cmd/get_test.go +1 -1

No files found.
--- a/pkg/kubectl/cmd/get.go
+++ b/pkg/kubectl/cmd/get.go
@@ -362,8 +362,10 @@ func RunGet(f cmdutil.Factory, out, errOut io.Writer, cmd *cobra.Command, args [
 				},
 			}
 			if listMeta, err := meta.ListAccessor(obj); err == nil {
-				list.Object["selfLink"] = listMeta.GetSelfLink()
+				list.Object["metadata"] = map[string]interface{}{
-				list.Object["resourceVersion"] = listMeta.GetResourceVersion()
+					"selfLink":        listMeta.GetSelfLink(),
+					"resourceVersion": listMeta.GetResourceVersion(),
+				}
 			}
 			for _, item := range items {

--- a/pkg/kubectl/cmd/get_test.go
+++ b/pkg/kubectl/cmd/get_test.go
@@ -603,7 +603,7 @@ func TestGetMultipleTypeObjectsAsList(t *testing.T) {
 	}
 	expected := &unstructured.UnstructuredList{
-		Object: map[string]interface{}{"kind": "List", "apiVersion": "v1", "metadata": map[string]interface{}{}, "selfLink": "", "resourceVersion": ""},
+		Object: map[string]interface{}{"kind": "List", "apiVersion": "v1", "metadata": map[string]interface{}{"selfLink": "", "resourceVersion": ""}},
 		Items: []unstructured.Unstructured{
 			fn(&pods.Items[0]),
 			fn(&pods.Items[1]),