41978 2022-02-17 02:00:22 +0300 Impossible to exit hsh-shell if processes left in background 2022-02-20 01:26:54 +0300 1 1 4 Development Sisyphus hasher unstable x86 Linux NEW P5 normal --- 1 vt ldv at glebfm iv ldv placeholder qa-sisyphus oldest_to_newest 207910 0 vt 2022-02-17 02:00:22 +0300 When developing or debugging something in the hasher (as in best practices) some process could be inadvertently left in background, causing failure exiting hsh-shell and working terminal hang. Example: ``` $ hsh-shell builder@x86_64:/.in$ sleep 1234 & [1] 3978262 builder@x86_64:/.in$ exit logout ``` Hangs there. It does not respond to ^C or ^Z, user cannot kill stray process, because it's different user. Would be more rational if hasher kills such processes on exit. (Possible after user y/n request). Or I'm missing something? 208012 1 ldv 2022-02-19 19:09:35 +0300 If hasher kills all lingering processes after the main process exit, some data may be lost. If hasher doesn't kill all lingering processes after the main process exit, the wait may take too long. Somebody is going to be unhappy whatever approach is chosen. Maybe you want an option. 208013 2 vt 2022-02-19 19:49:27 +0300 I suggest user confirmation - kill (y/n) or wait longer. Also, what data could be lost in the hasher? 208014 3 ldv 2022-02-19 20:03:16 +0300 (In reply to Vitaly Chikunov from comment #2) > I suggest user confirmation - kill (y/n) or wait longer. Snatch terminal from remaining hasher processes? > Also, what data could be lost in the hasher? Any kind of data processed inside hasher. For example, if the main process fails to wait for its child processes properly, the killing spree would terminate these processes prematurely. 208015 4 vt 2022-02-19 20:09:19 +0300 (Ответ для Dmitry V. Levin на комментарий #3) > (In reply to Vitaly Chikunov from comment #2) > > I suggest user confirmation - kill (y/n) or wait longer. > > Snatch terminal from remaining hasher processes? Design is such that user cannot be asked? > > Also, what data could be lost in the hasher? > > Any kind of data processed inside hasher. > > For example, if the main process fails to wait for its child processes > properly, > the killing spree would terminate these processes prematurely. Thanks. But, how would you resolve this situation otherwise? (Without kill from root). Terminal is hanged, you don't kill processes, waited process user is different (and sometimes user do not have root to even kill). 208016 5 ldv 2022-02-19 20:40:31 +0300 (In reply to Vitaly Chikunov from comment #4) > (Ответ для Dmitry V. Levin на комментарий #3) > > (In reply to Vitaly Chikunov from comment #2) > > > I suggest user confirmation - kill (y/n) or wait longer. > > > > Snatch terminal from remaining hasher processes? > > Design is such that user cannot be asked? If there is a terminal, everything is theoretically possible, but ... > > > Also, what data could be lost in the hasher? > > > > Any kind of data processed inside hasher. > > > > For example, if the main process fails to wait for its child processes > > properly, > > the killing spree would terminate these processes prematurely. > > Thanks. But, how would you resolve this situation otherwise? (Without kill > from root). Terminal is hanged, you don't kill processes, waited process > user is different (and sometimes user do not have root to even kill). ... root is not needed, the user can kill its hasher-priv processes at this stage, they have the same uid as the user and they are visible via ps. 208025 6 vt 2022-02-20 00:59:31 +0300 > ... root is not needed, the user can kill its hasher-priv processes at this > stage, they have the same uid as the user and they are visible via ps. IC. It's worked. This is certainly non-obvious. But, the background process also got SIGKILL and terminal is messed up so I got to run `stty sane`. So this solution, while exists and I didn't think of it, still as good as premature kill (and it's SIGKILL, not even SIGINT/SIGTERM which careful process could handle so save data). 208026 7 ldv 2022-02-20 01:26:54 +0300 (In reply to Vitaly Chikunov from comment #6) > > ... root is not needed, the user can kill its hasher-priv processes at this > > stage, they have the same uid as the user and they are visible via ps. > > IC. It's worked. This is certainly non-obvious. But, the background process > also got SIGKILL and terminal is messed up so I got to run `stty sane`. That's because hasher-priv doesn't handle deadly signals, maybe it should handle them and restore the terminal. > So this solution, while exists and I didn't think of it, still as good as > premature kill (and it's SIGKILL, not even SIGINT/SIGTERM which careful > process could handle so save data). Those who kill hasher-priv this way should make sure the data is safe. Alternatively, one can terminate chrootuid2.sh first. But all these methods are hacks, I'd prefer to have an option for hsh-shell to wait for the specified number of seconds before calling it a day, or an option to specify an escape sequence similar to "ssh -e".