Skip to content

[v18] Improve tsh kubectl traces#63762

Merged
rosstimothy merged 1 commit intobranch/v18from
bot/backport-63652-branch/v18
Feb 16, 2026
Merged

[v18] Improve tsh kubectl traces#63762
rosstimothy merged 1 commit intobranch/v18from
bot/backport-63652-branch/v18

Conversation

@rosstimothy
Copy link
Contributor

Backport #63652 to branch/v18

changelog: Improved tracing support via tsh --trace kubectl.

The tracing support for tsh kubectl was in a semi-working, but mostly
not functional state. The spans were disjoint instead of all being
associated with a singular SpanContext partly because we didn't pass
the SpanContext from parent to child process and partly due to kubectl
limitations. This addresess those limitations by passing the SpanContext
as an environment variable from the parent tsh process to the child. This
gets all Teleport originated spans unified in the same Span, however, the
kubectl originated spans - by way of us setting a custom http.RoundTripper
were not grouped because kubectl uses a context.TODO for all requests. To
work around this the custom http.RoundTripper was updated to take a
parent context and inject it into the http.Request before it is passed
along to the tracing machinery.
@rosstimothy rosstimothy marked this pull request as ready for review February 12, 2026 17:15
@github-actions github-actions bot added backport size/sm tsh tsh - Teleport's command line tool for logging into nodes running Teleport. labels Feb 12, 2026
@rosstimothy rosstimothy added this pull request to the merge queue Feb 16, 2026
Merged via the queue into branch/v18 with commit 4225f37 Feb 16, 2026
44 of 45 checks passed
@rosstimothy rosstimothy deleted the bot/backport-63652-branch/v18 branch February 16, 2026 14:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport size/sm tsh tsh - Teleport's command line tool for logging into nodes running Teleport.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

Comments