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

[BUG] Can't run Node shell #2281

Closed
vb0om opened this issue Nov 9, 2023 · 2 comments · Fixed by #2282
Closed

[BUG] Can't run Node shell #2281

vb0om opened this issue Nov 9, 2023 · 2 comments · Fixed by #2282
Labels
bug Something isn't working

Comments

@vb0om
Copy link

vb0om commented Nov 9, 2023




Hello! Thank's for K9s. It's really awesome :)

Describe the bug
Can't run Node shell. When try to run Node shell using hotkey "S", nothing happens. I checked that the pod with name like "node-shell-XXXX" - was started and running, but after about 9 seconds, it terminated without error(code 0). When I changed bin of K9s to version 0.27.4 - everything worked as expected.

To Reproduce
Steps to reproduce the behavior:

  1. Choose Node and push "S" button

Expected behavior
Running node shell well

@vb0om vb0om changed the title Can't run Node shell [BUG] Can't run Node shell Nov 9, 2023
@derailed derailed added the bug Something isn't working label Nov 10, 2023
@jtackaberry
Copy link

jtackaberry commented Nov 11, 2023

I have this problem too, even in 0.28.1 where this was supposedly fixed. (I suppose the change log is in error, since the PR hasn't yet been merged?)

k9s log says:

6:46PM DBG Shell Config &config.ShellPod{Image:"busybox:1.35.0", Command:[]string{}, Args:[]string{}, Namespace:"default", Limits:config.Limits{"cpu":"100m", "memory":"100Mi"}, Labels:map[string]string{}}
6:46PM DBG Checking shell pod [0] Pending
6:46PM DBG Checking shell pod [1] Running
6:46PM ERR SSH Failed error="os detect failed: no os information available"

"SSH failed" is confusing. Wouldn't k9s just exec into the pod?

@derailed
Copy link
Owner

@jtackaberry Thanks for the heads up Jason! Thought I had merge it so that's on me ;(

Let see if we're happier on v0.28.2....

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants