diff --git a/NEWS b/NEWS index 4c295ca402b5784d973f9409fb55338b861e3aff..0003d2efbd2014ae53116b64c71883b88bb87fe5 100644 --- a/NEWS +++ b/NEWS @@ -207,6 +207,8 @@ documents those changes that are of interest to users and admins. -- With default configuration on non-Cray systems, enable salloc to be spawned as a background process. Based upon work by Don Albert (Bull) and Gerrit Renker (CSCS). + -- Fixed Regression from 2.2.4 in accounting where an inherited limit + would not be set correctly in the added child association. * Changes in SLURM 2.2.6 ======================== diff --git a/src/plugins/accounting_storage/mysql/as_mysql_assoc.c b/src/plugins/accounting_storage/mysql/as_mysql_assoc.c index 15d97df59d4c7d1b772107ff1175b468dc274914..3b696b704b6ce0a871f33dc51fa50560b8e37b88 100644 --- a/src/plugins/accounting_storage/mysql/as_mysql_assoc.c +++ b/src/plugins/accounting_storage/mysql/as_mysql_assoc.c @@ -643,24 +643,26 @@ static int _set_assoc_limits_for_add( if (!(row = mysql_fetch_row(result))) goto end_it; - if (row[ASSOC2_REQ_DEF_QOS] && assoc->def_qos_id == NO_VAL) + if (row[ASSOC2_REQ_DEF_QOS] && assoc->def_qos_id == INFINITE) assoc->def_qos_id = slurm_atoul(row[ASSOC2_REQ_DEF_QOS]); - else if (assoc->def_qos_id == NO_VAL) + else if (assoc->def_qos_id == INFINITE) assoc->def_qos_id = 0; - if (row[ASSOC2_REQ_MCMPJ] && assoc->max_cpu_mins_pj == (uint64_t)NO_VAL) + if (row[ASSOC2_REQ_MCMPJ] + && assoc->max_cpu_mins_pj == (uint64_t)INFINITE) assoc->max_cpu_mins_pj = slurm_atoull(row[ASSOC2_REQ_MCMPJ]); - if (row[ASSOC2_REQ_MCRM] && assoc->max_cpu_run_mins == (uint64_t)NO_VAL) + if (row[ASSOC2_REQ_MCRM] + && assoc->max_cpu_run_mins == (uint64_t)INFINITE) assoc->max_cpu_run_mins = slurm_atoull(row[ASSOC2_REQ_MCRM]); - if (row[ASSOC2_REQ_MCPJ] && assoc->max_cpus_pj == NO_VAL) + if (row[ASSOC2_REQ_MCPJ] && assoc->max_cpus_pj == INFINITE) assoc->max_cpus_pj = slurm_atoul(row[ASSOC2_REQ_MCPJ]); - if (row[ASSOC2_REQ_MJ] && assoc->max_jobs == NO_VAL) + if (row[ASSOC2_REQ_MJ] && assoc->max_jobs == INFINITE) assoc->max_jobs = slurm_atoul(row[ASSOC2_REQ_MJ]); - if (row[ASSOC2_REQ_MNPJ] && assoc->max_nodes_pj == NO_VAL) + if (row[ASSOC2_REQ_MNPJ] && assoc->max_nodes_pj == INFINITE) assoc->max_nodes_pj = slurm_atoul(row[ASSOC2_REQ_MNPJ]); - if (row[ASSOC2_REQ_MSJ] && assoc->max_submit_jobs == NO_VAL) + if (row[ASSOC2_REQ_MSJ] && assoc->max_submit_jobs == INFINITE) assoc->max_submit_jobs = slurm_atoul(row[ASSOC2_REQ_MSJ]); - if (row[ASSOC2_REQ_MWPJ] && assoc->max_wall_pj == NO_VAL) + if (row[ASSOC2_REQ_MWPJ] && assoc->max_wall_pj == INFINITE) assoc->max_wall_pj = slurm_atoul(row[ASSOC2_REQ_MWPJ]); if (assoc->qos_list) {