Skip to content

Conversation

makkes
Copy link
Member

@makkes makkes commented Jul 4, 2022

When a cluster is created in a namespace that's different from the
default one ("default") then downloading the kubeconfig failed because
the UI didn't provide the cluster's namespace to the server.

With this change, the namespace is provided as a query parameter.

@makkes makkes requested review from AlinaGoaga and ahussein3 July 4, 2022 14:56
@makkes makkes added bug Something isn't working area/ui labels Jul 4, 2022
@makkes makkes self-assigned this Jul 4, 2022
@makkes makkes force-pushed the kubeconfig-provide-ns branch from 6025a22 to 38b7fcb Compare July 4, 2022 14:58
@makkes makkes removed the bug Something isn't working label Jul 4, 2022
Copy link
Collaborator

@foot foot left a comment

Choose a reason for hiding this comment

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

image

Nice catch! Works great for me. LGTM

@makkes makkes changed the title UI: provide cluster namespace when fetching kbueconfig UI: provide cluster namespace when fetching kubeconfig Jul 4, 2022
When a cluster is created in a namespace that's different from the
default one ("default") then downloading the kubeconfig failed because
the UI didn't provide the cluster's namespace to the server.

With this change, the namespace is provided as a query parameter.
@makkes makkes force-pushed the kubeconfig-provide-ns branch from 38b7fcb to 6cb261b Compare July 5, 2022 08:53
@makkes makkes merged commit 14aee7c into main Jul 5, 2022
@makkes makkes deleted the kubeconfig-provide-ns branch July 5, 2022 09:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants