kills resident users when del option is used #69

Closed

btatu wants to merge 1 commit from master-fix-48033 into master

btatu commented

2020-02-10 11:55:38 +01:00

Member

This merge request is ready to fix this error :

16:54 < reg> userdel: user FOO is currently used by process 5350
16:54 < reg> à killer pour web-add del !

This merge request is ready to fix this error : ~~~ 16:54 < reg> userdel: user FOO is currently used by process 5350 16:54 < reg> à killer pour web-add del ! ~~~

gcolpart commented

2020-02-10 16:20:01 +01:00

Owner

no need to have a condition
why using killall and not kill ?

* no need to have a condition * why using killall and not kill ?

btatu commented

2020-02-10 18:11:22 +01:00

Author

Member

I've adopted the logic of this script : It exit when a command return 1.
So, we can kill all users with an exit 0 with :

pkill -u $user || true

But, It's two methods. I prefer respect the first logical used. Otherwise, which other solutions ?

I've adopted the logic of this script : It exit when a command return 1. So, we can kill all users with an exit 0 with : ~~~ pkill -u $user || true ~~~ But, It's two methods. I prefer respect the first logical used. Otherwise, which other solutions ?

bwaegeneire commented

2021-09-22 10:37:53 +02:00

Owner

After a quick talk eith gcolpart, no need to kill the processes. Just force the deletion of the account as e3dc97d6d4 does.

We don't need this PR anymore.

After a quick talk eith gcolpart, no need to kill the processes. Just force the deletion of the account as e3dc97d6d4b2e262cd046d83790268799842645f does. We don't need this PR anymore.