diff --git a/CHANGELOG.md b/CHANGELOG.md index 71c532f69e..38bb5dd2fc 100644 --- a/CHANGELOG.md +++ b/CHANGELOG.md @@ -12,6 +12,7 @@ - The command `operator-sdk olm-catalog gen-csv --csv-version= --update-crds` would fail to copy over CRD manifests into `deploy/olm-catalog` for manifests whose name didn't end with a `_crd.yaml` suffix. This has been fixed so `gen-csv` now copies all CRD manifests specified by `deploy/olm-catalog/csv_config.yaml` by checking the type of the manifest rather than the filename suffix. ([#2015](https://github.com/operator-framework/operator-sdk/pull/2015)) - Fixed an issue in `operator-sdk olm-catalog gen-csv` where the generated CSV is missing the expected set of owned CRDs. ([#2017](https://github.com/operator-framework/operator-sdk/pull/2017)) +- Fixed invalid usage of `logr.Logger.Info()` in the Ansible-based operator implementation, which caused unnecessary operator panics. ([#2031](https://github.com/operator-framework/operator-sdk/pull/2031)) ## v0.10.0 diff --git a/pkg/ansible/controller/reconcile.go b/pkg/ansible/controller/reconcile.go index 1a26424a73..8c9dc2ece2 100644 --- a/pkg/ansible/controller/reconcile.go +++ b/pkg/ansible/controller/reconcile.go @@ -271,7 +271,7 @@ func (r *AnsibleOperatorReconciler) markError(u *unstructured.Unstructured, name // Get the latest resource to prevent updating a stale status err := r.Client.Get(context.TODO(), namespacedName, u) if apierrors.IsNotFound(err) { - logger.Info("Resource not found, assuming it was deleted", err) + logger.Info("Resource not found, assuming it was deleted") return nil } if err != nil { @@ -310,7 +310,7 @@ func (r *AnsibleOperatorReconciler) markDone(u *unstructured.Unstructured, names // Get the latest resource to prevent updating a stale status err := r.Client.Get(context.TODO(), namespacedName, u) if apierrors.IsNotFound(err) { - logger.Info("Resource not found, assuming it was deleted", err) + logger.Info("Resource not found, assuming it was deleted") return nil } if err != nil {