Skip to content

Commit 80beef6

Browse files
Benjamin Withamgaijin03
authored andcommitted
Fix sprio regression not handling comma separated list of jobids
sprio was unable to parse a comma-separated list of jobs despite documentation stating that this was possible. The entire job list was passed to the unfmt_job_id_string() function, but this function is unable to parse job lists. Switching to only pass the single job to unfmt_job_id_string() fixes this issue. Regression introduced in 4f21612. Changelog: Fix sprio regression not handling comma separated list of jobids. Ticket: 24625
1 parent 9cf3d4c commit 80beef6

1 file changed

Lines changed: 1 addition & 1 deletion

File tree

src/sprio/opts.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -466,7 +466,7 @@ static list_t *_build_job_list(char *str)
466466
job = strtok_r(my_job_list, ",", &tmp_char);
467467
while (job) {
468468
slurm_selected_step_t sel_step;
469-
if (unfmt_job_id_string(str, &sel_step, NO_VAL)) {
469+
if (unfmt_job_id_string(job, &sel_step, NO_VAL)) {
470470
error("Invalid job id: %s", job);
471471
exit(1);
472472
} else if (sel_step.het_job_offset != NO_VAL)

0 commit comments

Comments
 (0)