fix and enhance logging message on policy deletion

Author: graindcafe

internal/controller/user/reconcile.go

@@ -377,7 +377,7 @@ func (r *S3UserReconciler) handleUpdate(
 		policyFound := slices.Contains(userResource.Spec.Policies, policy)
 		if !policyFound {
 			logger.Info(
-				fmt.Sprintf("S3User policy definition doesn't contain policy %s", policy),
+				fmt.Sprintf("S3User has unexpected policy not in definition: %s", policy),
 				"userResource",
 				userResource.Name,
 				"NamespacedName",
@@ -391,7 +391,7 @@ func (r *S3UserReconciler) handleUpdate(
 		policyFound := slices.Contains(userPolicies, policy)
 		if !policyFound {
 			logger.Info(
-				fmt.Sprintf("S3User policy definition must contain policy %s", policy),
+				fmt.Sprintf("S3User is missing policy from definition: %s", policy),
 				"userResource",
 				userResource.Name,
 				"NamespacedName",

internal/s3/client/impl/minioS3Client.go

@@ -539,7 +539,7 @@ func (minioS3Client *MinioS3Client) RemovePoliciesFromUser(
 		}
 		s3Logger.Error(
 			err,
-			"an error occurred when attaching a policy to the user",
+			"an error occurred when detaching a policy to the user",
 			"code",
 			errAsResp.Code,
 		)