Suspend k9s using CTRL-Z

[arajski]

Description

Implements unix-like suspend functionality mentioned in #2013

At the moment, k9s supports Ctrl-C, which exits the application, however it's not possible to suspend the process and send it to the background. This PR implements that functionality, which is common among other tools, like vim.
User can now send the process to the background by pressing Ctrl-Z.
Once suspended, it can be resumed by either using fg or selecting a specific id from the jobs list (for example %1) in case there are multiple ones.

This behaviour can be enabled by setting allowSuspend to true.

Changes

• remapped "Toggle Faults" action from Ctrl-Z to Ctrl-T (first one available that kind of made sense to me)
• added suspend functionality to Ctrl-Z, which sends SIGTSTP signal to the current k9s process
• added allowSuspend config to opt in to this behaviour
• updated tests

TODO:

• test on other platforms

Notes

This is my first contribution to this project, so happy to update things if there's anything missing/required. Thanks!

[derailed]

@arajski Thanks for this update Artur!
Biggest concern with this behavior is to make sure we're cool all supported platforms.

[derailed]

@arajski Thank you for the updates Artur! LGTM
I think we need to test this out on linux, windows flavors, macOs.

[arajski]

@derailed seems like go doesn't support os signals on Windows without some hacky workarounds.
I was thinking about spawning a shell for anything other than linux and darwin OS.
From a user perspective it's going to look almost the same, except in order to return to k9s, you'll need to exit the shell instead of resuming the process from the background. Let me know if that makes sense.
Otherwise we can go with the subshell approach for everyone.

[derailed]

@arajski Thank you for the research Artur! I think for non nix users likely Ctrtl-z does not make much sense. I don't use windows much these days but isn't Ctrl-z equivalent to undo?
If so might be best to detect the os and disable that functionality while in windows land?

[arajski]

Done 👍 Tested on macOS Sonoma and Debian 12

[derailed]

@arajski Thanks for the checks and updates Artur.

[derailed]

@arajski Looks much better. Thank you! Looks like we need to resolve a conflict...

[arajski]

@derailed sorry it took a while, conflict has been resolved. I noticed that you moved the quitCmd to the view package.
I didn't want to do anything out of context, but happy to also move this one as well if you think it's now more suitable.

[mghesh-yseop]

Hi @arajski and @derailed, still some conflicts ongoing. I'd love to see this feature merged 😄

[derailed]

@arajski Can you take another peek (conflicts). Thank you!

[arajski]

@derailed all up to date 👍🏻

[derailed]

@arajski Thank you for these updates!

[github-actions]

This PR is stale because it has been open for 30 days with no activity.