Merge pull request #41 from inteon/fix_predicate

Also reconcile when the linked issuer receives an update

controllers/predicates.go

@@ -121,6 +121,7 @@ func (CertificateSigningRequestPredicate) Update(e event.UpdateEvent) bool {
 // In these cases we want to trigger:
 // - the Ready condition was added/ removed
 // - the Ready condition's Status property changed
+// - the Ready condition's observed generation changed
 type LinkedIssuerPredicate struct {
 	predicate.Funcs
 }
@@ -156,7 +157,7 @@ func (LinkedIssuerPredicate) Update(e event.UpdateEvent) bool {
 		return readyOld != nil || readyNew != nil
 	}
 
-	return readyNew.Status != readyOld.Status
+	return readyNew.Status != readyOld.Status || readyNew.ObservedGeneration != readyOld.ObservedGeneration
 }
 
 // Predicate for Issuer events that should trigger the Issuer reconciler

controllers/predicates_test.go

@@ -506,6 +506,31 @@ func TestLinkedIssuerPredicate(t *testing.T) {
 				),
 			},
 		},
+		{
+			name:            "ready-condition-identical-new-observed-generation",
+			shouldReconcile: true,
+			event: event.UpdateEvent{
+				ObjectOld: testutil.SimpleIssuerFrom(issuer1,
+					testutil.SetSimpleIssuerStatusCondition(
+						fakeClock,
+						cmapi.IssuerConditionReady,
+						cmmeta.ConditionFalse,
+						"reason1",
+						"message1",
+					),
+				),
+				ObjectNew: testutil.SimpleIssuerFrom(issuer1,
+					testutil.SetSimpleIssuerGeneration(2),
+					testutil.SetSimpleIssuerStatusCondition(
+						fakeClock,
+						cmapi.IssuerConditionReady,
+						cmmeta.ConditionFalse,
+						"reason2",
+						"message2",
+					),
+				),
+			},
+		},
 		{
 			name:            "ready-condition-changed",
 			shouldReconcile: true,