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

k9s 0.24.8 edit needs an extra keystroke to process #1109

Closed
MrLuje opened this issue Apr 30, 2021 · 10 comments
Closed

k9s 0.24.8 edit needs an extra keystroke to process #1109

MrLuje opened this issue Apr 30, 2021 · 10 comments
Labels
bug Something isn't working

Comments

@MrLuje
Copy link
Contributor

MrLuje commented Apr 30, 2021




Describe the bug
Hitting "E" for the edit view works weirdly, it needed an extra keypress to really open the editor

A clear and concise description of what the bug is.

To Reproduce
Steps to reproduce the behavior:

  • Open k9s and go to any resources, like pods
  • Hit "e" for the edit view
    nothing happen here, I'm still on the pods view... but if I press another key like space or enter, it opens the editor

Expected behavior
I should be in my editor opened on the selected pod after hitting "Edit"

Versions (please complete the following information):

  • OS: Windows WSL2 + Windows Terminal
  • K9s: 0.24.8
  • K8s: 1.17.17

I reverted to 0.24.7 to confirm and it properly works on 0.24.7 but not on 0.24.8

@MrLuje
Copy link
Contributor Author

MrLuje commented Apr 30, 2021

Just noticed CTRL+C also need an extra keypress to exit

@dyipon
Copy link

dyipon commented May 2, 2021

Shell needs extra keypress too

@derailed derailed added the help wanted Extra attention is needed label May 2, 2021
@derailed
Copy link
Owner

derailed commented May 2, 2021

@MrLuje @dyipon - I can't seem to repro this on my loaner box. Running windows 10 + powerhell. Pod edit + shell in are working as expected on v0.24.8 ie no additional input.
How did you guys install k9s?
Any windows pro here care to shade some light? My window fu is pretty much tapped out ;(

@dyipon
Copy link

dyipon commented May 2, 2021

@derailed - me @ debian buster, and k9s installed manually with wget. It worked until this release

@MrLuje
Copy link
Contributor Author

MrLuje commented May 2, 2021

@derailed Installed with homebrew
I also tested with regular cmd.exe (as I'm using Windows Terminal usually) and I still hit the issue

@MrLuje
Copy link
Contributor Author

MrLuje commented May 2, 2021

@derailed A few more tests :

  • compiled k9s from source 0.24.8 on windows, no bug on the windows binary
  • compiled k9s from source 0.24.8 on wsl2, bug again on the linux binary
  • compiled on wsl2 an older version of k9s (tried every minor from 0.24.7 to 0.24.5) also bug (even though your binaries for the same versions are fine 🤔)

@derailed derailed added bug Something isn't working and removed help wanted Extra attention is needed labels May 4, 2021
@derailed
Copy link
Owner

derailed commented May 4, 2021

@MrLuje Once again Thank you so much for the extra details! I am very much appreciating your help and support tracking this down. Honestly, I am not sure I've got this?? Crossing fingers and toes on this one.
Let's give v0.24.9 a spin and see if we're happier... Please reopen if we've got a dud. Thank you!!

@derailed derailed closed this as completed May 4, 2021
@dyipon
Copy link

dyipon commented May 4, 2021

Extra keystroke doesn't need anymore, so ctrl-c and shell works again. But if I enter into a shellpod, an extra '?' key presented in the prompt. Not a big deal, just have to press a backspace. Tried on debian linux.

@MrLuje
Copy link
Contributor Author

MrLuje commented May 4, 2021

Toes did the trick 😁
As stated by @dyipon, there are remaining stuff when opening a shell (^ or ? depending on the previous action) but it's much better already
image
image

@derailed
Copy link
Owner

derailed commented May 4, 2021

@MrLuje @dyipon Serenity now! Thank you both for reporting back. Will track down this last issue. Tx!

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

No branches or pull requests

3 participants