Skip to content

Add k8s pod/cluster attributes to PENDING status span#254

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

Add k8s pod/cluster attributes to PENDING status span#254
morgan-wowk wants to merge 1 commit into
execution-tracing-error-datafrom
execution-tracing-launcher-data

Conversation

@morgan-wowk
Copy link
Copy Markdown
Collaborator

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

Add Kubernetes pod/cluster attributes to PENDING execution spans

k8s.pod.name, k8s.namespace.name, k8s.cluster.url sourced from
ContainerExecution.launcher_data. Handles kubernetes, kubernetes_job,
and non-k8s launchers (SkyPilot) gracefully via early-return guards.

The new _launcher_pod_attrs helper only emits attributes when the span
status is PENDING and launcher_data is present, so spans for other
statuses or non-Kubernetes launchers are unaffected.

Screenshots

Screenshot 2026-05-22 at 8.13.33 PM.png

Copy link
Copy Markdown
Collaborator Author

k8s.pod.name, k8s.namespace.name, k8s.cluster.url sourced from
ContainerExecution.launcher_data. Handles kubernetes, kubernetes_job,
and non-k8s launchers (SkyPilot) gracefully via early-return guards.
@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 force-pushed the execution-tracing-launcher-data branch from 3e85d85 to 239a70c Compare May 23, 2026 02:57
@morgan-wowk morgan-wowk marked this pull request as ready for review May 23, 2026 03: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