From eddf21f42cca8da092a21af18dcf44dd4af648b4 Mon Sep 17 00:00:00 2001
From: Bo-Chun Louis Chen <louistw@uab.edu>
Date: Wed, 22 Jan 2025 13:54:20 -0600
Subject: [PATCH] style: use variable instead of hardcoded

---
 etc/slurm/job_submit.lua | 7 +++++--
 1 file changed, 5 insertions(+), 2 deletions(-)

diff --git a/etc/slurm/job_submit.lua b/etc/slurm/job_submit.lua
index 94e6ce2..ebe16d9 100644
--- a/etc/slurm/job_submit.lua
+++ b/etc/slurm/job_submit.lua
@@ -26,6 +26,9 @@ local grp = require("posix").grp
 function slurm_job_submit(job_desc, part_list, submit_uid)
 
    local target_group = "gpfs5"
+   local target_feature = "gpfs5"
+   local default_feature = "gpfs4"
+
    local username = pwd.getpwuid(submit_uid).pw_name
    local group_member = grp.getgrnam(target_group).gr_mem
    local found = false
@@ -35,12 +38,12 @@ function slurm_job_submit(job_desc, part_list, submit_uid)
    for i,member in ipairs(group_member) do
       if member == username then
          found = true
-         job_desc.features = "gpfs5"
+         job_desc.features = target_feature
          break
       end
    end
    if not found then
-      job_desc.features = "gpfs4"
+      job_desc.features = default_feature
    end
 
    slurm.log_info("username:" .. username)
-- 
GitLab