Merge pull request #95 from draychev/simplify-2

Simplify IAM Controller's Update fn
keikoproj · May 7, 2022 · 32e2109 · 32e2109
2 parents a7120cf + b86acef
commit 32e2109
Showing 1 changed file with 1 addition and 4 deletions.
diff --git a/controllers/iamrole_controller.go b/controllers/iamrole_controller.go
@@ -366,10 +366,7 @@ func (StatusUpdatePredicate) Update(e event.UpdateEvent) bool {
 	oldObj := e.ObjectOld.(*iammanagerv1alpha1.Iamrole)
 	newObj := e.ObjectNew.(*iammanagerv1alpha1.Iamrole)
 
-	if oldObj.Status != newObj.Status {
-		return false
-	}
-	return true
+	return oldObj.Status == newObj.Status
 }
 
 //SetupWithManager sets up manager with controller