From 1f012a2bb3e07a53b5b28bb9bc8781667508faeb Mon Sep 17 00:00:00 2001
From: Clayton Coleman <ccoleman@redhat.com>
Date: Mon, 4 Apr 2016 23:02:37 -0400
Subject: [PATCH] Implement a simpler unversioned.Time serialization

Time is the number of seconds since the unix epoch and is consistent
with our JSON fields.
---
 pkg/api/unversioned/time.go       |  3 ++-
 pkg/api/unversioned/time_proto.go | 28 ++++++++++------------------
 2 files changed, 12 insertions(+), 19 deletions(-)

diff --git a/pkg/api/unversioned/time.go b/pkg/api/unversioned/time.go
index 1180e6bd14f..df94bbe72c8 100644
--- a/pkg/api/unversioned/time.go
+++ b/pkg/api/unversioned/time.go
@@ -28,8 +28,9 @@ import (
 // of the factory methods that the time package offers.
 //
 // +protobuf.options.marshal=false
+// +protobuf.as=Timestamp
 type Time struct {
-	time.Time `protobuf:"Timestamp,1,req,name=time"`
+	time.Time `protobuf:"-"`
 }
 
 // NewTime returns a wrapped instance of the provided time
diff --git a/pkg/api/unversioned/time_proto.go b/pkg/api/unversioned/time_proto.go
index 6d6fe5d4e7b..5ca0edcdf05 100644
--- a/pkg/api/unversioned/time_proto.go
+++ b/pkg/api/unversioned/time_proto.go
@@ -22,15 +22,9 @@ import (
 	"time"
 )
 
-// ProtoTime is a struct that is equivalent to Time, but intended for
+// Timestamp is a struct that is equivalent to Time, but intended for
 // protobuf marshalling/unmarshalling. It is generated into a serialization
 // that matches Time. Do not use in Go structs.
-type ProtoTime struct {
-	// Represents the time of an event.
-	Timestamp Timestamp `json:"timestamp"`
-}
-
-// Timestamp is a protobuf Timestamp compatible representation of time.Time
 type Timestamp struct {
 	// Represents seconds of UTC time since Unix epoch
 	// 1970-01-01T00:00:00Z. Must be from from 0001-01-01T00:00:00Z to
@@ -39,20 +33,18 @@ type Timestamp struct {
 	// Non-negative fractions of a second at nanosecond resolution. Negative
 	// second values with fractions must still have non-negative nanos values
 	// that count forward in time. Must be from 0 to 999,999,999
-	// inclusive.
+	// inclusive. This field may be limited in precision depending on context.
 	Nanos int32 `json:"nanos"`
 }
 
-// ProtoTime returns the Time as a new ProtoTime value.
-func (m *Time) ProtoTime() *ProtoTime {
+// Timestamp returns the Time as a new Timestamp value.
+func (m *Time) ProtoTime() *Timestamp {
 	if m == nil {
-		return &ProtoTime{}
+		return &Timestamp{}
 	}
-	return &ProtoTime{
-		Timestamp: Timestamp{
-			Seconds: m.Time.Unix(),
-			Nanos:   int32(m.Time.Nanosecond()),
-		},
+	return &Timestamp{
+		Seconds: m.Time.Unix(),
+		Nanos:   int32(m.Time.Nanosecond()),
 	}
 }
 
@@ -61,11 +53,11 @@ func (m *Time) Size() (n int) { return m.ProtoTime().Size() }
 
 // Reset implements the protobuf marshalling interface.
 func (m *Time) Unmarshal(data []byte) error {
-	p := ProtoTime{}
+	p := Timestamp{}
 	if err := p.Unmarshal(data); err != nil {
 		return err
 	}
-	m.Time = time.Unix(p.Timestamp.Seconds, int64(p.Timestamp.Nanos))
+	m.Time = time.Unix(p.Seconds, int64(p.Nanos))
 	return nil
 }