Actually reconcile service accounts (#324)

Signed-off-by: Jason Parraga <sovietaced@gmail.com>

Author: Sovietaced

internal/controller/install/common_helpers.go

@@ -152,6 +152,16 @@ func (cc *CommonComponents) ReconcileComponents(newComponents *CommonComponents)
 		cc.Service = nil
 	}
 
+	if newComponents.ServiceAccount != nil {
+		cc.ServiceAccount.Labels = newComponents.ServiceAccount.Labels
+		cc.ServiceAccount.Annotations = newComponents.ServiceAccount.Annotations
+		cc.ServiceAccount.ImagePullSecrets = newComponents.ServiceAccount.ImagePullSecrets
+		cc.ServiceAccount.Secrets = newComponents.ServiceAccount.Secrets
+		cc.ServiceAccount.AutomountServiceAccountToken = newComponents.ServiceAccount.AutomountServiceAccountToken
+	} else {
+		cc.ServiceAccount = nil
+	}
+
 	if newComponents.ClusterRole != nil {
 		cc.ClusterRole.Rules = newComponents.ClusterRole.Rules
 		cc.ClusterRole.Labels = newComponents.ClusterRole.Labels

internal/controller/install/common_helpers_test.go

@@ -759,6 +759,27 @@ func makeCommonComponents() CommonComponents {
 		},
 	}
 
+	automountServiceAccountToken := true
+	serviceAccount := corev1.ServiceAccount{
+		ObjectMeta: metav1.ObjectMeta{
+			Name:        "some-name",
+			Namespace:   "some-namespace",
+			Labels:      map[string]string{"some-label-key": "some-label-value"},
+			Annotations: map[string]string{"some-annotation-key": "some-annotation-value"},
+		},
+		ImagePullSecrets: []corev1.LocalObjectReference{
+			{
+				Name: "some-image-pull-secret",
+			},
+		},
+		Secrets: []corev1.ObjectReference{
+			{
+				Name: "some-secret",
+			},
+		},
+		AutomountServiceAccountToken: &automountServiceAccountToken,
+	}
+
 	pc := schedulingv1.PriorityClass{
 		Value: 1000,
 	}
@@ -768,6 +789,7 @@ func makeCommonComponents() CommonComponents {
 	}
 	return CommonComponents{
 		Deployment:      &deployment,
+		ServiceAccount:  &serviceAccount,
 		PriorityClasses: []*schedulingv1.PriorityClass{&pc},
 		Secret:          &secret,
 	}