Skip to content

Enrich error/success status spans with failure context#253

Open
morgan-wowk wants to merge 1 commit into
masterfrom
execution-tracing-error-data
Open

Enrich error/success status spans with failure context#253
morgan-wowk wants to merge 1 commit into
masterfrom
execution-tracing-error-data

Conversation

@morgan-wowk
Copy link
Copy Markdown
Collaborator

@morgan-wowk morgan-wowk commented May 22, 2026

Enrich execution trace spans with error details and root span status

FAILED span: Attaches error.message from orchestration_error_message in extra_data and execution.exit_code from the associated ContainerExecution when available.

SYSTEM_ERROR span: Attaches error.message from the system error exception message and exception.stacktrace from the full exception traceback stored in extra_data.

SUCCEEDED span: Attaches execution.exit_code from the associated ContainerExecution when available.

Root span: Calls set_status(ERROR) when the terminal status is FAILED or SYSTEM_ERROR, leaving the root span unmarked for successful executions.

Screenshots

Screenshot 2026-05-22 at 8.14.49 PM.png

@morgan-wowk morgan-wowk force-pushed the execution-tracing-core branch from ebcd9a2 to e9fe6e0 Compare May 23, 2026 00:28
@morgan-wowk morgan-wowk force-pushed the execution-tracing-error-data branch from 6581758 to c73509f Compare May 23, 2026 00:31
@morgan-wowk morgan-wowk force-pushed the execution-tracing-core branch from e9fe6e0 to 059c47b Compare May 23, 2026 02:57
@morgan-wowk morgan-wowk force-pushed the execution-tracing-error-data branch from c73509f to 057544f Compare May 23, 2026 02:57
@morgan-wowk morgan-wowk marked this pull request as ready for review May 23, 2026 03:22
@morgan-wowk morgan-wowk changed the base branch from execution-tracing-core to graphite-base/253 May 26, 2026 23:16
@morgan-wowk morgan-wowk force-pushed the execution-tracing-error-data branch from 057544f to 04630ed Compare May 26, 2026 23:27
@morgan-wowk morgan-wowk requested a review from Ark-kun as a code owner May 26, 2026 23:27
@morgan-wowk morgan-wowk changed the base branch from graphite-base/253 to execution-tracing-core May 26, 2026 23:48
@morgan-wowk morgan-wowk changed the base branch from execution-tracing-core to graphite-base/253 May 27, 2026 00:21
@morgan-wowk morgan-wowk force-pushed the execution-tracing-error-data branch from 04630ed to 2ac38c8 Compare May 27, 2026 00:21
@graphite-app graphite-app Bot changed the base branch from graphite-base/253 to master May 27, 2026 00:22
FAILED span: error.message from orchestration_error_message, exit_code
SYSTEM_ERROR span: error.message + exception.stacktrace from extra_data
SUCCEEDED span: exit_code from ContainerExecution
Root span: set_status(ERROR) for FAILED and SYSTEM_ERROR terminals
@morgan-wowk morgan-wowk force-pushed the execution-tracing-error-data branch from 2ac38c8 to 2025a32 Compare May 27, 2026 00:22
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.

1 participant