From bc846e03cc9e1661a2b080c1ab9be48a5700de3c Mon Sep 17 00:00:00 2001
From: Jacob Floyd <cognifloyd@gmail.com>
Date: Wed, 4 May 2022 13:24:07 -0500
Subject: [PATCH] tests: test inspectContainerEvent edge cases

---
 runtime/kubernetes/container_test.go  | 119 ++++++++++++++++++++++++++
 runtime/kubernetes/kubernetes_test.go |  16 +++-
 2 files changed, 134 insertions(+), 1 deletion(-)

diff --git a/runtime/kubernetes/container_test.go b/runtime/kubernetes/container_test.go
index a92799edd..e0423430e 100644
--- a/runtime/kubernetes/container_test.go
+++ b/runtime/kubernetes/container_test.go
@@ -17,6 +17,7 @@ import (
 	"github.com/sirupsen/logrus"
 	"golang.org/x/sync/errgroup"
 	v1 "k8s.io/api/core/v1"
+	metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"
 )
 
 func TestKubernetes_InspectContainer(t *testing.T) {
@@ -885,6 +886,124 @@ func Test_podTracker_inspectContainerEvent(t *testing.T) {
 		imagePullError bool
 		event          *v1.Event
 	}{
+		{
+			name:           "pod scheduled event ignored",
+			trackedPod:     "test/github-octocat-1",
+			ctnName:        "step-github-octocat-1-clone",
+			ctnImage:       "target/vela-git:v0.4.0",
+			imagePulled:    false,
+			imagePullError: false,
+			event: &v1.Event{
+				ObjectMeta: metav1.ObjectMeta{
+					Name:      _pod.ObjectMeta.Name + ".16ea333d810392b7",
+					Namespace: _pod.ObjectMeta.Namespace,
+				},
+				Action:              "Binding",
+				Reason:              "Scheduled", // reasonScheduled,
+				Message:             "Successfully assigned test/github-octocat-1 to k8s-worker",
+				ReportingController: "default-scheduler",
+				ReportingInstance:   "default-scheduler-k8s-master",
+				Source:              v1.EventSource{}, // empty
+			},
+		},
+		{
+			name:           "container created event is ignored",
+			trackedPod:     "test/github-octocat-1",
+			ctnName:        "step-github-octocat-1-clone",
+			ctnImage:       "target/vela-git:v0.4.0",
+			imagePulled:    false,
+			imagePullError: false,
+			event: mockContainerEvent(
+				_pod,
+				"step-github-octocat-1-clone",
+				"Created", // reasonCreated,
+				"Created container step-github-octocat-1-clone",
+			),
+		},
+		{
+			name:           "container started event is ignored",
+			trackedPod:     "test/github-octocat-1",
+			ctnName:        "step-github-octocat-1-clone",
+			ctnImage:       "target/vela-git:v0.4.0",
+			imagePulled:    false,
+			imagePullError: false,
+			event: mockContainerEvent(
+				_pod,
+				"step-github-octocat-1-clone",
+				"Started", // reasonStarted,
+				"Started container step-github-octocat-1-clone",
+			),
+		},
+		{
+			name:           "container killing event is ignored",
+			trackedPod:     "test/github-octocat-1",
+			ctnName:        "step-github-octocat-1-clone",
+			ctnImage:       "target/vela-git:v0.4.0",
+			imagePulled:    false,
+			imagePullError: false,
+			event: mockContainerEvent(
+				_pod,
+				"step-github-octocat-1-clone",
+				"Killing", //reasonKilling,
+				"Container step-github-octocat-1-clone definition changed, will be restarted",
+			),
+		},
+		{
+			name:           "generic image pull failed event is ignored",
+			trackedPod:     "test/github-octocat-1",
+			ctnName:        "step-github-octocat-1-clone",
+			ctnImage:       "target/vela-git:v0.4.0",
+			imagePulled:    false,
+			imagePullError: false,
+			event: mockContainerEvent(
+				_pod,
+				"step-github-octocat-1-clone",
+				reasonFailed,
+				"ErrImagePull",
+			),
+		},
+		{
+			name:           "generic image pull back-off is ignored",
+			trackedPod:     "test/github-octocat-1",
+			ctnName:        "step-github-octocat-1-clone",
+			ctnImage:       "target/vela-git:v0.4.0",
+			imagePulled:    false,
+			imagePullError: false,
+			event: mockContainerEvent(
+				_pod,
+				"step-github-octocat-1-clone",
+				reasonBackOff,
+				"ImagePullBackOff",
+			),
+		},
+		{
+			name:           "event for unnamed container ignored",
+			trackedPod:     "test/github-octocat-1",
+			ctnName:        "step-github-octocat-1-clone",
+			ctnImage:       "target/vela-git:v0.4.0",
+			imagePulled:    false,
+			imagePullError: false,
+			event: mockContainerEvent(
+				_pod,
+				"",
+				reasonFailed,
+				"Failed to pull image \"target/vela-git:v0.4.0\": containerd message",
+			),
+		},
+		{
+			name:           "event for unknown container ignored",
+			trackedPod:     "test/github-octocat-1",
+			ctnName:        "step-github-octocat-1-clone",
+			ctnImage:       "target/vela-git:v0.4.0",
+			imagePulled:    false,
+			imagePullError: false,
+			event: mockContainerEvent(
+				_pod,
+				"admissions-controller-injected-container",
+				reasonFailed,
+				"Failed to pull image \"target/vela-git:v0.4.0\": containerd message",
+			),
+		},
 		{
 			name:           "image pull failed",
 			trackedPod:     "test/github-octocat-1",
diff --git a/runtime/kubernetes/kubernetes_test.go b/runtime/kubernetes/kubernetes_test.go
index 71969df9d..0c1d3c176 100644
--- a/runtime/kubernetes/kubernetes_test.go
+++ b/runtime/kubernetes/kubernetes_test.go
@@ -567,15 +567,29 @@ var (
 )
 
 func mockContainerEvent(pod *v1.Pod, ctnName, reason, message string) *v1.Event {
+	var fieldPath string
+	if ctnName != "" {
+		fieldPath = fmt.Sprintf("spec.containers{%s}", ctnName)
+	} else {
+		fieldPath = "spec.containers[2]"
+	}
+
 	return &v1.Event{
 		ObjectMeta: metav1.ObjectMeta{
+			Name:      _pod.ObjectMeta.Name + ".16ea333d810392b7",
 			Namespace: pod.ObjectMeta.Namespace,
 		},
 		InvolvedObject: v1.ObjectReference{
 			Kind:      pod.TypeMeta.Kind,
 			Name:      pod.ObjectMeta.Name,
 			Namespace: pod.ObjectMeta.Namespace,
-			FieldPath: fmt.Sprintf("spec.containers{%s}", ctnName),
+			FieldPath: fieldPath,
+		},
+		ReportingController: "",
+		ReportingInstance:   "",
+		Source: v1.EventSource{
+			Component: "kubelet",
+			Host:      "k8s-worker",
 		},
 		Reason:  reason,
 		Message: message,