From 0a690f06a4062afa77835462911ccbd5361477f9 Mon Sep 17 00:00:00 2001 From: Daniel Perrefort Date: Wed, 20 Dec 2023 15:58:42 -0500 Subject: [PATCH] Fixes bug where slurm users are terminated --- shinigami/utils.py | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/shinigami/utils.py b/shinigami/utils.py index 1f843a5..74bad2b 100755 --- a/shinigami/utils.py +++ b/shinigami/utils.py @@ -143,9 +143,11 @@ async def terminate_errant_processes( process_df = await get_remote_processes(conn) # Filter process data by various whitelist/blacklist criteria + # Outputs from each filter function call are passed to the next filter + # so the order of the function calls matter significantly + process_df = exclude_active_slurm_users(process_df) process_df = include_orphaned_processes(process_df) process_df = include_user_whitelist(process_df, uid_whitelist) - process_df = exclude_active_slurm_users(process_df) for _, row in process_df.iterrows(): # pragma: nocover logging.info(f'[{node}] Marking for termination {dict(row)}')