Set synced when missing (#68)

Issue #, if available: aws-controllers-k8s/community#1133

Description of changes:
* Do not overwrite `ACK.ResourceSynced` condition, if it was being set by ReadOne/Create/Update hook code
* This change will restore existing functionality for asynchronous Create/Update handling until `AWS.ResourceManager.IsSynced` method is implemented. 
* revert the AWSResourceManager.IsSynced method, which can be added after this immediate patch
----

Tested locally with `iam-controller` and `rds-controller`

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Author: vijtrip2

mocks/pkg/types/aws_resource_manager.go

@@ -424,9 +424,12 @@ func (r *resourceReconciler) updateResource(
 		}
 		rlog.Info("updated resource")
 	} else {
-		// If there is no delta between desired state and latest state, it is
+		// If there is no delta between desired state and latest state, and
+		// ACK.ResourceSynced condition is not already set, it is
 		// safe to set ACK.ResourceSynced condition to true.
-		ackcondition.SetSynced(latest, corev1.ConditionTrue, nil, nil)
+		if ackcondition.Synced(latest) == nil {
+			ackcondition.SetSynced(latest, corev1.ConditionTrue, nil, nil)
+		}
 	}
 	return latest, nil
 }

pkg/runtime/reconciler.go

@@ -83,8 +83,6 @@ type AWSResourceManager interface {
 	// This method also adds/updates the ConditionTypeReferencesResolved for the
 	// AWSResource.
 	ResolveReferences(context.Context, client.Reader, AWSResource) (AWSResource, error)
-	// IsSynced returns true if a resource is synced.
-	IsSynced(context.Context, AWSResource) (bool, error)
 }
 
 // AWSResourceManagerFactory returns an AWSResourceManager that can be used to