update log message when requeueing with no error (#99)

Description of changes:
* When a resource was being requeued without any error, the debug log message reads `requeue needed after error {... , "error": null, "after": "10h0m0s"...}` , which is a red herring for users when debugging controller issues.
* This PR updates the log message correctly to not say `requeue needed after error`, when the resource is being requeued with no error.
* Resource can be requeued without any error for drift remediation.

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

pkg/runtime/reconciler.go

@@ -949,20 +949,28 @@ func (r *resourceReconciler) HandleReconcileError(
 	var requeueNeededAfter *requeue.RequeueNeededAfter
 	if errors.As(err, &requeueNeededAfter) {
 		after := requeueNeededAfter.Duration()
-		rlog.Debug(
-			"requeue needed after error",
-			"error", requeueNeededAfter.Unwrap(),
-			"after", after,
-		)
+		if wrappedError := requeueNeededAfter.Unwrap(); wrappedError != nil {
+			rlog.Debug(
+				"requeue needed after error",
+				"error", wrappedError,
+				"after", after,
+			)
+		} else {
+			rlog.Debug("requeueing", "after", after)
+		}
 		return ctrlrt.Result{RequeueAfter: after}, nil
 	}
 
 	var requeueNeeded *requeue.RequeueNeeded
 	if errors.As(err, &requeueNeeded) {
-		rlog.Debug(
-			"requeue needed error",
-			"error", requeueNeeded.Unwrap(),
-		)
+		if wrappedError := requeueNeeded.Unwrap(); wrappedError != nil {
+			rlog.Debug(
+				"requeue needed after error",
+				"error", wrappedError,
+			)
+		} else {
+			rlog.Debug("requeueing immediately")
+		}
 		return ctrlrt.Result{Requeue: true}, nil
 	}