Fix logger panic (#283)

* Fix logger panic * Changelog
giantswarm · Aug 6, 2024 · 18c1823 · 18c1823
1 parent 286a905
commit 18c1823
Show file tree

Hide file tree

Showing 2 changed files with 5 additions and 1 deletion.
diff --git a/CHANGELOG.md b/CHANGELOG.md
@@ -7,6 +7,10 @@ and this project adheres to [Semantic Versioning](https://semver.org/spec/v2.0.0
 
 ## [Unreleased]
 
+### Fixed
+
+- Fix panics in logging statements
+
 ## [0.29.2] - 2024-07-23
 
 ### Fixed

diff --git a/controllers/capa_controller.go b/controllers/capa_controller.go
@@ -252,7 +252,7 @@ func (r *CAPAClusterReconciler) removeAWSClusterFinalizer(ctx context.Context, l
 		// k8serrors package. We have to get the cluster again with the now
 		// removed finalizer(s) and try again.
 		if k8serrors.IsInvalid(err) && i < maxPatchRetries {
-			logger.Info("patching AWSCluster failed, trying again: %s", err.Error())
+			logger.Info("patching AWSCluster failed, trying again", "error", err.Error())
 			if err := r.Get(ctx, client.ObjectKeyFromObject(cluster), cluster); err != nil {
 				return microerror.Mask(err)
 			}