Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[v16] [kube] add namespace/podName as SessionTracker hostname #44649

Merged
merged 1 commit into from
Jul 30, 2024

Conversation

tigrato
Copy link
Contributor

@tigrato tigrato commented Jul 25, 2024

Backport #44614 to branch/v16

This PR changes the `SessionTracker.Hostname` to include the namespace instead of just the podname.
The final result is `podNamespace/podName` as it uniquely identifies a pod within a Kubernetes cluster.

Helps #44496

Signed-off-by: Tiago Silva <[email protected]>
@tigrato tigrato added the no-changelog Indicates that a PR does not require a changelog entry label Jul 25, 2024
@tigrato tigrato enabled auto-merge July 25, 2024 18:00
@github-actions github-actions bot requested review from rudream and zmb3 July 25, 2024 18:00
@tigrato tigrato added this pull request to the merge queue Jul 30, 2024
@public-teleport-github-review-bot public-teleport-github-review-bot bot removed the request for review from rudream July 30, 2024 09:20
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Jul 30, 2024
@tigrato tigrato added this pull request to the merge queue Jul 30, 2024
Merged via the queue into branch/v16 with commit 4194ff0 Jul 30, 2024
38 of 39 checks passed
@tigrato tigrato deleted the bot/backport-44614-branch/v16 branch July 30, 2024 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport kubernetes-access no-changelog Indicates that a PR does not require a changelog entry size/sm
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants