update conditions on error for delete operation (#175)

Description of changes:
- Update conditions on delete in case of error. Right now the errors from delete are not reflected to the user
  - I have added a check to treat all errors as recoverable if the resource is being deleted as I am not sure what is the user experience if the resource hits a terminal condition after its deleted. Please advise

Testing:
Manually introduced an error from sdkDelete and verified it gets updated in conditions

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

Author: surajkota

templates/pkg/resource/manager.go.tpl

@@ -146,7 +146,18 @@ func (rm *resourceManager) Delete(
 		// Should never happen... if it does, it's buggy code.
 		panic("resource manager's Update() method received resource with nil CR object")
 	}
-	return rm.sdkDelete(ctx, r)
+	observed, err := rm.sdkDelete(ctx, r)
+	if err != nil {
+		if observed != nil {
+			return rm.onError(observed, err)
+		}
+		return rm.onError(r, err)
+	}
+
+	if observed != nil {
+		return rm.onSuccess(observed)
+	}
+	return rm.onSuccess(r)
 }
 
 // ARNFromName returns an AWS Resource Name from a given string name. This