[Fix] : returned both a non-zero result and a non-nil error

[drivebyer]

fix log with following message:

{"level":"info","ts":"2023-11-21T21:42:03+08:00","msg":"Warning: Reconciler returned both a non-zero result and a non-nil error. The result will always be ignored if the error is non-nil and the non-nil error causes reqeueuing with exponential backoff. For more details, see: https://pkg.go.dev/sigs.k8s.io/controller-runtime/pkg/reconcile#Reconciler","controller":"rediscluster","controllerGroup":"redis.redis.opstreelabs.in","controllerKind":"RedisCluster","RedisCluster":{"name":"redis-cluster-v1beta2","namespace":"kuttl-test-social-bluejay"},"namespace":"kuttl-test-social-bluejay","name":"redis-cluster-v1beta2","reconcileID":"97a22714-ba92-41d6-a76a-0dbb67bf677d"}

The result will be ignore when error is non-nil.

https://github.com/kubernetes-sigs/controller-runtime/blob/5f8d96b043ef8ccadc544535415395be336630e8/pkg/reconcile/reconcile.go#L93-L95

[codecov]

Codecov Report

Attention: 14 lines in your changes are missing coverage. Please review.

Comparison is base (a597c39) 27.79% compared to head (afb4582) 27.79%.

Files	Patch %	Lines
controllers/rediscluster_controller.go	0.00%	14 Missing ⚠️

Additional details and impacted files

@@           Coverage Diff           @@
##           master     #705   +/-   ##
=======================================
  Coverage   27.79%   27.79%           
=======================================
  Files          18       18           
  Lines        3123     3123           
=======================================
  Hits          868      868           
  Misses       2228     2228           
  Partials       27       27           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.