From c56c4c87f74d80766fdb42785de4934d105512ee Mon Sep 17 00:00:00 2001 From: AdoHe Date: Sun, 28 Feb 2016 00:59:01 -0500 Subject: [PATCH] kubectl add AGE column to job, ingress and daemonset --- pkg/kubectl/resource_printer.go | 21 +++++++++++++-------- pkg/kubectl/resource_printer_test.go | 8 ++++---- 2 files changed, 17 insertions(+), 12 deletions(-) diff --git a/pkg/kubectl/resource_printer.go b/pkg/kubectl/resource_printer.go index 717ad47cef6..5d58ebb6eb4 100644 --- a/pkg/kubectl/resource_printer.go +++ b/pkg/kubectl/resource_printer.go @@ -398,12 +398,12 @@ var podColumns = []string{"NAME", "READY", "STATUS", "RESTARTS", "AGE"} var podTemplateColumns = []string{"TEMPLATE", "CONTAINER(S)", "IMAGE(S)", "PODLABELS"} var replicationControllerColumns = []string{"NAME", "DESIRED", "CURRENT", "AGE"} var replicaSetColumns = []string{"NAME", "DESIRED", "CURRENT", "AGE"} -var jobColumns = []string{"NAME", "DESIRED", "SUCCESSFUL"} +var jobColumns = []string{"NAME", "DESIRED", "SUCCESSFUL", "AGE"} var serviceColumns = []string{"NAME", "CLUSTER-IP", "EXTERNAL-IP", "PORT(S)", "AGE"} -var ingressColumns = []string{"NAME", "RULE", "BACKEND", "ADDRESS"} +var ingressColumns = []string{"NAME", "RULE", "BACKEND", "ADDRESS", "AGE"} var endpointColumns = []string{"NAME", "ENDPOINTS", "AGE"} var nodeColumns = []string{"NAME", "STATUS", "AGE"} -var daemonSetColumns = []string{"NAME", "DESIRED", "CURRENT", "NODE-SELECTOR"} +var daemonSetColumns = []string{"NAME", "DESIRED", "CURRENT", "NODE-SELECTOR", "AGE"} var eventColumns = []string{"FIRSTSEEN", "LASTSEEN", "COUNT", "NAME", "KIND", "SUBOBJECT", "TYPE", "REASON", "SOURCE", "MESSAGE"} var limitRangeColumns = []string{"NAME", "AGE"} var resourceQuotaColumns = []string{"NAME", "AGE"} @@ -839,18 +839,20 @@ func printJob(job *extensions.Job, w io.Writer, options PrintOptions) error { selector, _ := unversioned.LabelSelectorAsSelector(job.Spec.Selector) if job.Spec.Completions != nil { - if _, err := fmt.Fprintf(w, "%s\t%d\t%d", + if _, err := fmt.Fprintf(w, "%s\t%d\t%d\t%s", name, *job.Spec.Completions, job.Status.Succeeded, + translateTimestamp(job.CreationTimestamp), ); err != nil { return err } } else { - if _, err := fmt.Fprintf(w, "%s\t%s\t%d", + if _, err := fmt.Fprintf(w, "%s\t%s\t%d\t%s", name, "", job.Status.Succeeded, + translateTimestamp(job.CreationTimestamp), ); err != nil { return err } @@ -1002,11 +1004,13 @@ func printIngress(ingress *extensions.Ingress, w io.Writer, options PrintOptions } } - if _, err := fmt.Fprintf(w, "%s\t%v\t%v\t%v", + if _, err := fmt.Fprintf(w, "%s\t%v\t%v\t%v\t%s", name, "-", backendStringer(ingress.Spec.Backend), - loadBalancerStatusStringer(ingress.Status.LoadBalancer)); err != nil { + loadBalancerStatusStringer(ingress.Status.LoadBalancer), + translateTimestamp(ingress.CreationTimestamp), + ); err != nil { return err } if _, err := fmt.Fprint(w, appendLabels(ingress.Labels, options.ColumnLabels)); err != nil { @@ -1078,11 +1082,12 @@ func printDaemonSet(ds *extensions.DaemonSet, w io.Writer, options PrintOptions) // this shouldn't happen if LabelSelector passed validation return err } - if _, err := fmt.Fprintf(w, "%s\t%d\t%d\t%s", + if _, err := fmt.Fprintf(w, "%s\t%d\t%d\t%s\t%s", name, desiredScheduled, currentScheduled, labels.FormatLabels(ds.Spec.Template.Spec.NodeSelector), + translateTimestamp(ds.CreationTimestamp), ); err != nil { return err } diff --git a/pkg/kubectl/resource_printer_test.go b/pkg/kubectl/resource_printer_test.go index 61e93cb1e63..57accde49d0 100644 --- a/pkg/kubectl/resource_printer_test.go +++ b/pkg/kubectl/resource_printer_test.go @@ -1326,7 +1326,7 @@ func TestPrintDaemonSet(t *testing.T) { DesiredNumberScheduled: 3, }, }, - "test1\t3\t2\t\n", + "test1\t3\t2\t\t0s\n", }, } @@ -1359,13 +1359,13 @@ func TestPrintJob(t *testing.T) { Succeeded: 1, }, }, - "job1\t2\t1\n", + "job1\t2\t1\t0s\n", }, { extensions.Job{ ObjectMeta: api.ObjectMeta{ Name: "job2", - CreationTimestamp: unversioned.Time{Time: time.Now().Add(1.9e9)}, + CreationTimestamp: unversioned.Time{Time: time.Now().AddDate(-10, 0, 0)}, }, Spec: extensions.JobSpec{ Completions: nil, @@ -1374,7 +1374,7 @@ func TestPrintJob(t *testing.T) { Succeeded: 0, }, }, - "job2\t\t0\n", + "job2\t\t0\t10y\n", }, }