Skip to content
Snippets Groups Projects
Commit 3c582329 authored by Danny Auble's avatar Danny Auble
Browse files
parent 9cc782c8
No related branches found
No related tags found
No related merge requests found
...@@ -238,11 +238,11 @@ static int _job_modify(uint32_t jobid, char *bank_ptr, ...@@ -238,11 +238,11 @@ static int _job_modify(uint32_t jobid, char *bank_ptr,
hostlist_t hl; hostlist_t hl;
char *tasklist; char *tasklist;
if (!job_ptr->details) { if (!IS_JOB_PENDING(job_ptr) || !job_ptr->details) {
/* Job is done, nothing to reset */ /* Job is done, nothing to reset */
if (new_hostlist == '\0') if (new_hostlist == '\0')
goto host_fini; goto host_fini;
error("wiki: MODIFYJOB tasklist of non-pending " error("wiki: MODIFYJOB hostlist of non-pending "
"job %u", jobid); "job %u", jobid);
return ESLURM_DISABLED; return ESLURM_DISABLED;
} }
...@@ -292,12 +292,19 @@ host_fini: if (rc) { ...@@ -292,12 +292,19 @@ host_fini: if (rc) {
if (part_name_ptr) { if (part_name_ptr) {
struct part_record *part_ptr; struct part_record *part_ptr;
if (!IS_JOB_PENDING(job_ptr)) {
error("wiki: MODIFYJOB partition of non-pending "
"job %u", jobid);
return ESLURM_DISABLED;
}
part_ptr = find_part_record(part_name_ptr); part_ptr = find_part_record(part_name_ptr);
if (part_ptr == NULL) { if (part_ptr == NULL) {
error("wiki: MODIFYJOB has invalid partition %s", error("wiki: MODIFYJOB has invalid partition %s",
part_name_ptr); part_name_ptr);
return ESLURM_INVALID_PARTITION_NAME; return ESLURM_INVALID_PARTITION_NAME;
} }
info("wiki: change job %u partition %s", info("wiki: change job %u partition %s",
jobid, part_name_ptr); jobid, part_name_ptr);
xfree(job_ptr->partition); xfree(job_ptr->partition);
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment