From 542ee95c36fd60af25e574a965d70803ff827160 Mon Sep 17 00:00:00 2001 From: Vincent Van Rossem Date: Tue, 31 Mar 2026 15:31:19 +0200 Subject: [PATCH] [IMP] attachment_azure: downgrade ResourceExistsError log from ERROR to INFO Using _logger.exception for a ResourceExistsError is misleading: the operation is harmless since blobs are content-addressed (SHA1) and filename is returned correctly regardless. Switch to _logger.info to avoid false-positive ERROR noise. --- attachment_azure/models/ir_attachment.py | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/attachment_azure/models/ir_attachment.py b/attachment_azure/models/ir_attachment.py index ee0741fc..9bddf0fb 100644 --- a/attachment_azure/models/ir_attachment.py +++ b/attachment_azure/models/ir_attachment.py @@ -177,8 +177,8 @@ def _store_file_write(self, key, bin_data): try: blob_client.upload_blob(file, blob_type="BlockBlob") except ResourceExistsError: - _logger.exception( - "Trying to re create an existing resource %s" % filename + _logger.info( + "Resource already exists, skipping upload for %s", filename ) except HttpResponseError as error: # log verbose error from azure, return short message for user