Skip to content

Add support for joining Kubernetes sessions in the web ui#52993

Merged
rosstimothy merged 1 commit intomasterfrom
tross/kube_web_join
Mar 25, 2025
Merged

Add support for joining Kubernetes sessions in the web ui#52993
rosstimothy merged 1 commit intomasterfrom
tross/kube_web_join

Conversation

@rosstimothy
Copy link
Copy Markdown
Contributor

@rosstimothy rosstimothy commented Mar 12, 2025

Closes #52315

changelog: Enable support for joining Kubernetes sessions in the web UI.

@rosstimothy rosstimothy force-pushed the tross/terminal_focus branch from ccd7e0f to 9b897d2 Compare March 12, 2025 11:20
Base automatically changed from tross/terminal_focus to master March 12, 2025 15:21
@rosstimothy rosstimothy force-pushed the tross/kube_web_join branch 20 times, most recently from 1551f1e to a54bc3e Compare March 13, 2025 10:28
@rosstimothy rosstimothy marked this pull request as ready for review March 13, 2025 11:03
@github-actions github-actions Bot added size/md tsh tsh - Teleport's command line tool for logging into nodes running Teleport. ui labels Mar 13, 2025
@github-actions github-actions Bot requested review from gzdunek and ryanclark March 13, 2025 11:03
Copy link
Copy Markdown
Contributor

@gzdunek gzdunek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry for the delay, I'll test it tomorrow!

Comment thread lib/web/kube.go Outdated
Comment thread web/packages/teleport/src/Console/useTabRouting.ts Outdated
Comment on lines +111 to +115
if (doc.mode === undefined) {
setStatus('waiting-for-exec-data');
} else {
setStatus('initialized');
}
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we combine it with the condition above?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I would very much like to drop the else clause here and only set the initialized status in response to session data above. However when testing this last week, removing the else clause prevents things from working. @avatus and I couldn't really determine why that was the case.

@gzdunek
Copy link
Copy Markdown
Contributor

gzdunek commented Mar 20, 2025

I can't join a session for some reason.
I successfully start a session as user A, then try to join as user B as an observer, but I get the following error in the console:

dial tcp :443: connect: connection refused
disconnected

I don't see anything interesting in the teleport logs. Do you have any idea what happens?

@rosstimothy rosstimothy force-pushed the tross/kube_web_join branch 2 times, most recently from c871c0d to 123a1ff Compare March 21, 2025 23:37
@rosstimothy
Copy link
Copy Markdown
Contributor Author

I can't join a session for some reason. I successfully start a session as user A, then try to join as user B as an observer, but I get the following error in the console:

dial tcp :443: connect: connection refused
disconnected

I don't see anything interesting in the teleport logs. Do you have any idea what happens?

@gzdunek I think this should be resolved by 3282fe4.

Copy link
Copy Markdown
Contributor

@gzdunek gzdunek left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.
I tested all three joining modes and they work properly.

Adds the ability to join existing Kubernetes sessions in the web ui
as peers, observers, or modertators.

Closes #52315.
@rosstimothy rosstimothy force-pushed the tross/kube_web_join branch from 0b17ec2 to 1469c7b Compare March 25, 2025 13:39
@rosstimothy rosstimothy added this pull request to the merge queue Mar 25, 2025
Merged via the queue into master with commit be8fe5c Mar 25, 2025
47 checks passed
@rosstimothy rosstimothy deleted the tross/kube_web_join branch March 25, 2025 14:36
@backport-bot-workflows
Copy link
Copy Markdown
Contributor

@rosstimothy See the table below for backport results.

Branch Result
branch/v16 Failed
branch/v17 Failed

rosstimothy added a commit that referenced this pull request Mar 26, 2025
Adds the ability to join existing Kubernetes sessions in the web ui
as peers, observers, or modertators.

Closes #52315.
rosstimothy added a commit that referenced this pull request Mar 26, 2025
Adds the ability to join existing Kubernetes sessions in the web ui
as peers, observers, or modertators.

Closes #52315.
rosstimothy added a commit that referenced this pull request Mar 26, 2025
Adds the ability to join existing Kubernetes sessions in the web ui
as peers, observers, or modertators.

Closes #52315.
github-merge-queue Bot pushed a commit that referenced this pull request Mar 26, 2025
Adds the ability to join existing Kubernetes sessions in the web ui
as peers, observers, or modertators.

Closes #52315.
github-merge-queue Bot pushed a commit that referenced this pull request Mar 27, 2025
Adds the ability to join existing Kubernetes sessions in the web ui
as peers, observers, or modertators.

Closes #52315.
zmb3 added a commit that referenced this pull request Jun 5, 2025
We added support for joining kube sessions in the web UI with #52993
and just missed this docs update.
github-merge-queue Bot pushed a commit that referenced this pull request Jun 6, 2025
We added support for joining kube sessions in the web UI with #52993
and just missed this docs update.
backport-bot-workflows Bot pushed a commit that referenced this pull request Jun 6, 2025
We added support for joining kube sessions in the web UI with #52993
and just missed this docs update.
backport-bot-workflows Bot pushed a commit that referenced this pull request Jun 6, 2025
We added support for joining kube sessions in the web UI with #52993
and just missed this docs update.
backport-bot-workflows Bot pushed a commit that referenced this pull request Jun 6, 2025
We added support for joining kube sessions in the web UI with #52993
and just missed this docs update.
github-merge-queue Bot pushed a commit that referenced this pull request Jun 7, 2025
We added support for joining kube sessions in the web UI with #52993
and just missed this docs update.
github-merge-queue Bot pushed a commit that referenced this pull request Jun 7, 2025
We added support for joining kube sessions in the web UI with #52993
and just missed this docs update.
github-merge-queue Bot pushed a commit that referenced this pull request Jun 7, 2025
We added support for joining kube sessions in the web UI with #52993
and just missed this docs update.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport/branch/v17 size/md tsh tsh - Teleport's command line tool for logging into nodes running Teleport. ui

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Joining Kubernetes Session from Web UI

3 participants