Skip to content

[17.0][IMP] attachment_azure: add stack_info on ResourceExistsError #524

Merged
florentx merged 1 commit into
17.0from
azure_attachment-traceback
May 19, 2026
Merged

[17.0][IMP] attachment_azure: add stack_info on ResourceExistsError #524
florentx merged 1 commit into
17.0from
azure_attachment-traceback

Conversation

@vvrossem
Copy link
Copy Markdown

@vvrossem vvrossem commented Apr 1, 2026

The blob already exists, so the exception traceback (inside upload_blob)
is not useful on its own. Adding stack_info=True to _logger.exception()
also logs the call stack, revealing who triggered the duplicate write.

@vvrossem vvrossem changed the title [IMP] attachment_azure: format_stack() on ResourceExistsError (17.0][IMP] attachment_azure: format_stack() on ResourceExistsError Apr 1, 2026
@vvrossem vvrossem changed the title (17.0][IMP] attachment_azure: format_stack() on ResourceExistsError [17.0][IMP] attachment_azure: format_stack() on ResourceExistsError Apr 1, 2026
Comment thread attachment_azure/models/ir_attachment.py Outdated
The blob already exists, so the exception traceback (inside upload_blob)
is not useful on its own. Adding stack_info=True to _logger.exception()
also logs the call stack, revealing who triggered the duplicate write.
@vvrossem vvrossem force-pushed the azure_attachment-traceback branch from 552d575 to d0fcf01 Compare April 14, 2026 13:23
@vvrossem vvrossem changed the title [17.0][IMP] attachment_azure: format_stack() on ResourceExistsError [17.0][IMP] attachment_azure: add stack_info on ResourceExistsError Apr 14, 2026
@vvrossem
Copy link
Copy Markdown
Author

vvrossem commented May 4, 2026

Hello @vrenaville, @florentx, could you please merge this PR ?

@florentx florentx merged commit b40cfce into 17.0 May 19, 2026
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants