Merge pull request #449 from sophieliu15/new_api_1

Support k8s.io/api/events/v1 API in Event Exporter.

Author: sophieliu15

event-exporter/sinks/stackdriver/log_entry_factory.go

@@ -22,7 +22,6 @@ import (
 
 	"github.com/golang/glog"
 	sd "google.golang.org/api/logging/v2"
-
 	corev1 "k8s.io/api/core/v1"
 	"k8s.io/apimachinery/pkg/runtime"
 	"k8s.io/apimachinery/pkg/runtime/serializer/json"
@@ -36,8 +35,6 @@ var (
 	fieldBlacklist = []string{
 		// Is unnecessary, because it's demuxed already
 		"count",
-		// Timestamp is in the logEntry's metadata
-		"lastTimestamp",
 		// Not relevant because of demuxing
 		"firstTimestamp",
 	}
@@ -66,10 +63,25 @@ func (f *sdLogEntryFactory) FromEvent(event *corev1.Event) *sd.LogEntry {
 
 	resource := f.resourceFactory.resourceFromEvent(event)
 
+	var timestamp string
+	if !event.LastTimestamp.IsZero() {
+		// The event was emitted using k8s.io/api/core/v1 library.
+		timestamp = event.LastTimestamp.Format(time.RFC3339Nano)
+	} else if event.Series != nil && !event.Series.LastObservedTime.IsZero() {
+		// The event was emitted using k8s.io/api/events/v1 library.
+		timestamp = event.Series.LastObservedTime.Format(time.RFC3339Nano)
+	} else if !event.EventTime.IsZero() {
+		// It is possible that either LastTimestamp or LastObservedTime is not set.
+		// In this case, EventTime is the next best choice if a log entry timestamp.
+		timestamp = event.EventTime.Format(time.RFC3339Nano)
+  } else {
+		timestamp = f.clock.Now().Format(time.RFC3339Nano)
+	}
+
 	return &sd.LogEntry{
 		JsonPayload: payload,
 		Severity:    f.detectSeverity(event),
-		Timestamp:   event.LastTimestamp.Format(time.RFC3339Nano),
+		Timestamp:   timestamp,
 		Resource:    resource,
 	}
 }

event-exporter/sinks/stackdriver/log_entry_factory_test.go

@@ -0,0 +1,141 @@
+package stackdriver
+
+import (
+	"testing"
+	"time"
+
+	"github.com/google/go-cmp/cmp"
+	sd "google.golang.org/api/logging/v2"
+	corev1 "k8s.io/api/core/v1"
+	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
+	"k8s.io/apimachinery/pkg/util/clock"
+)
+
+func TestFromEvent(t *testing.T) {
+	newTypesConfig := factoryConfig(newTypes)
+	monitoredResourceFactory := newMonitoredResourceFactory(newTypesConfig)
+	involvedObject := corev1.ObjectReference{Kind: node, Name: "test_node_name"}
+	wantedMonitoredResource := &sd.MonitoredResource{
+		Type: k8sNode,
+		Labels: map[string]string{
+			clusterName: newTypesConfig.clusterName,
+			location:    newTypesConfig.location,
+			projectID:   newTypesConfig.projectID,
+			nodeName:    "test_node_name",
+		},
+	}
+
+	time1 := time.Now()
+	time2 := time.Now()
+	time3 := time.Now()
+	time4 := time.Now()
+	lastTimestamp := metav1.NewTime(time1)
+	lastObservedTime := metav1.NewMicroTime(time2)
+	eventTime := metav1.NewMicroTime(time3)
+
+	tests := []struct {
+		desc      string
+		event  *corev1.Event
+		wanted *sd.LogEntry
+	}{
+		{
+			desc: "core/v1 event API",
+			event: &corev1.Event{
+				Type: "Warning",
+				InvolvedObject: involvedObject,
+				LastTimestamp: lastTimestamp,
+			},
+			wanted: &sd.LogEntry{
+				Timestamp: lastTimestamp.Format(time.RFC3339Nano),
+				Resource: wantedMonitoredResource,
+				Severity: "WARNING",
+			},
+		},
+		{
+			desc: "events/v1 event API",
+			event: &corev1.Event{
+				Type: "Warning",
+				InvolvedObject: involvedObject,
+				Series: &corev1.EventSeries{
+					Count:            1,
+					LastObservedTime: lastObservedTime,
+				},
+			},
+			wanted: &sd.LogEntry{
+				Timestamp: lastObservedTime.Format(time.RFC3339Nano),
+				Resource: wantedMonitoredResource,
+				Severity: "WARNING",
+			},
+		},
+		{
+			desc: "Only EventTime is set",
+			event: &corev1.Event{
+				Type: "Warning",
+				InvolvedObject: involvedObject,
+				EventTime: eventTime,
+			},
+			wanted: &sd.LogEntry{
+				Timestamp: eventTime.Format(time.RFC3339Nano),
+				Resource: wantedMonitoredResource,
+				Severity: "WARNING",
+			},
+		},
+		{
+			desc: "Timestamp not set",
+			event: &corev1.Event{
+				Type: "Warning",
+				InvolvedObject: involvedObject,
+			},
+			wanted: &sd.LogEntry{
+				Timestamp: time4.Format(time.RFC3339Nano),
+				Resource: wantedMonitoredResource,
+				Severity: "WARNING",
+			},
+		},
+		{
+			desc: "Event type is not set",
+			event: &corev1.Event{
+				InvolvedObject: involvedObject,
+				LastTimestamp: lastTimestamp,
+			},
+			wanted: &sd.LogEntry{
+				Timestamp: lastTimestamp.Format(time.RFC3339Nano),
+				Resource: wantedMonitoredResource,
+				Severity: "INFO",
+			},
+		},
+		{
+			desc: "Event type is not warning",
+			event: &corev1.Event{
+				Type: "Normal",
+				InvolvedObject: involvedObject,
+				LastTimestamp: lastTimestamp,
+			},
+			wanted: &sd.LogEntry{
+				Timestamp: lastTimestamp.Format(time.RFC3339Nano),
+				Resource: wantedMonitoredResource,
+				Severity: "INFO",
+			},
+		},
+	}
+
+	for _, test := range tests {
+		factory := newSdLogEntryFactory(clock.NewFakeClock(time4), monitoredResourceFactory)
+		got:= factory.FromEvent(test.event)
+    if diff := compareLogEntries(got, test.wanted); diff != "" {
+    	t.Errorf("Unexpected log entry from event %v, (-want +got): %s", test.event, diff)
+		}
+	}
+}
+
+func compareLogEntries(got, want *sd.LogEntry) string {
+	var ignore = map[string]bool{
+		"JsonPayload": true,
+	}
+	cmpIgnoreSomeFields := cmp.FilterPath(
+		func(p cmp.Path) bool {
+			return ignore[p.String()]
+		},
+		cmp.Ignore())
+	return cmp.Diff(got, want, cmpIgnoreSomeFields)
+}