Skip to content

fix: emit errors with source chains#47

Merged
SuperFluffy merged 2 commits intomainfrom
log-errors-with-source-chains
Aug 11, 2025
Merged

fix: emit errors with source chains#47
SuperFluffy merged 2 commits intomainfrom
log-errors-with-source-chains

Conversation

@SuperFluffy
Copy link
Copy Markdown
Contributor

The logging situation around errors was a bit chaotic. In too many instances the error source was cut off entirely, in a few other places debug formatting was used, in some the error was a field, in others the error was flattened into the tracing event message.

This patch crates a utility function error_field in a new crate tempo-telemetry-util that helps us streamline both typed errors and dynamic (eyre) errors.

@SuperFluffy SuperFluffy force-pushed the log-errors-with-source-chains branch from b73fc2c to 45f8082 Compare August 8, 2025 14:56
@SuperFluffy SuperFluffy merged commit 9d2ede9 into main Aug 11, 2025
8 of 12 checks passed
@SuperFluffy SuperFluffy deleted the log-errors-with-source-chains branch August 11, 2025 07:31
Zygimantass pushed a commit that referenced this pull request Sep 5, 2025
* feat: utility to emit error fields with source chain

* fix: emit all errors with their full source chain
legion2002 pushed a commit that referenced this pull request Oct 9, 2025
* feat: utility to emit error fields with source chain

* fix: emit all errors with their full source chain
unbalancedparentheses pushed a commit to unbalancedparentheses/tempo that referenced this pull request Feb 23, 2026
* feat: utility to emit error fields with source chain

* fix: emit all errors with their full source chain
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.

2 participants