Skip to content

App. Explorer "hangs" on extension loading if logged out from a cluster #3987

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.

Sign up for GitHub

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

Jump to bottom
Closed
vrubezhny opened this issue Mar 15, 2024 · 1 comment · Fixed by #4059
Closed

App. Explorer "hangs" on extension loading if logged out from a cluster #3987

vrubezhny opened this issue Mar 15, 2024 · 1 comment · Fixed by #4059
Assignees
@vrubezhny
Milestone
1.14.0

Comments

@vrubezhny
Copy link
Contributor

When Kube config's current-context points to a cluster which is available, but a user is not logged in to, the extension fails to load the App. explorer initial content permanently staying in "loading..." state.

Steps to reproduce:

  • Log in to "http:/api-rhamilto-devcluster-openshift-com:6443" cluster, see the default project and its contents loaded and shown in App. Explorer
  • Log out from the cluster, see "Login" button appeared instead of the App. Explorer tree
  • Exit VSCode Openshift Tools
  • Stat again, Fail: see the "infinite" loading of App. Explorer

This probably happens because the kubectl utilities, when invoked, are "permanently" waiting for user input:

image

$ kubectl cluster-info
Please enter Username: 

$ kubectl api-versions
Please enter Username:

The issue was highly likely introduced by this commit

@mohitsuman mohitsuman added this to the 1.14.0 milestone Apr 2, 2024
@vrubezhny
Copy link
Contributor Author

Depends on #3990

@vrubezhny vrubezhny moved this to 🏗 In progress in IDE Cloudaptors Apr 5, 2024
vrubezhny added a commit to vrubezhny/vscode-openshift-tools that referenced this issue Apr 9, 2024
@vrubezhny
Add 'kubectl' binary to the distribution
2f9a5d6 
Partially fixes redhat-developer#3987 by taking control over the 'kubectl' binary execution

Issue: redhat-developer#3987

Signed-off-by: Victor Rubezhny <[email protected]>
vrubezhny added a commit to vrubezhny/vscode-openshift-tools that referenced this issue Apr 9, 2024
@vrubezhny
Add 'kubectl' binary to the distribution
33f91d9 
Partially fixes redhat-developer#3987 by taking control over the 'kubectl' binary execution

Issue: redhat-developer#3987

Signed-off-by: Victor Rubezhny <[email protected]>
@vrubezhny vrubezhny mentioned this issue Apr 9, 2024
Closed
vrubezhny added a commit to vrubezhny/vscode-openshift-tools that referenced this issue Apr 11, 2024
@vrubezhny
Get rid of using k8s extension provided 'kubectl' binary
a28adbf 
The 'oc' binary is used instead in non-blocking manner, especially when getting 'api-versions' values.
Partially fixes redhat-developer#3987.

Issue: redhat-developer#3987

Signed-off-by: Victor Rubezhny <[email protected]>
vrubezhny added a commit to vrubezhny/vscode-openshift-tools that referenced this issue Apr 11, 2024
@vrubezhny
Get rid of using k8s extension provided 'kubectl' binary
6ae507d 
The 'oc' binary is used instead in non-blocking manner, especially when getting 'api-versions' values.
Partially fixes redhat-developer#3987.

Issue: redhat-developer#3987

Signed-off-by: Victor Rubezhny <[email protected]>
@vrubezhny vrubezhny mentioned this issue Apr 11, 2024
Merged
vrubezhny added a commit to vrubezhny/vscode-openshift-tools that referenced this issue Apr 12, 2024
@vrubezhny
Get rid of using k8s extension provided 'kubectl' binary
016fc66 
The 'oc' binary is used instead in non-blocking manner, especially when getting 'api-versions' values.
Partially fixes redhat-developer#3987.

Issue: redhat-developer#3987

Signed-off-by: Victor Rubezhny <[email protected]>
@vrubezhny vrubezhny moved this from 🏗 In progress to 👀 In review in IDE Cloudaptors Apr 12, 2024
datho7561 pushed a commit that referenced this issue Apr 15, 2024
@vrubezhny @datho7561
Get rid of using k8s extension provided 'kubectl' binary
c3a9b91 
The 'oc' binary is used instead in non-blocking manner, especially when getting 'api-versions' values.
Partially fixes #3987.

Issue: #3987

Signed-off-by: Victor Rubezhny <[email protected]>
@github-project-automation github-project-automation bot moved this from 👀 In review to ✅ Done in IDE Cloudaptors Apr 15, 2024
@vrubezhny vrubezhny mentioned this issue Apr 24, 2024
Closed
@vrubezhny vrubezhny mentioned this issue May 29, 2024
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vrubezhny vrubezhny

Labels
None yet
Projects
IDE Cloudaptors
Status: ✅ Done
Milestone
1.14.0
2 participants
@vrubezhny @mohitsuman