diff --git a/NEWS b/NEWS index 24817d80622b7aa5b0ffa66285c185e2164d1695..ca848cb9707541927245489e24db572a915b93ca 100644 --- a/NEWS +++ b/NEWS @@ -64,8 +64,6 @@ documents those changes that are of interest to users and administrators. -- Make sure the slurmstepd blocks signals like SIGTERM correctly. -- When submitting a --test-only job respect the -M option. -- SPANK - When slurm_spank_init_post_opt() fails return error correctly. - -- When revoking a sibling job in the federation we want to send a start - message before purging the job record to get the uid of the revoked job. * Changes in Slurm 17.11.2 ========================== diff --git a/src/slurmctld/fed_mgr.c b/src/slurmctld/fed_mgr.c index 522250a6ff14316738166df0117dd8cdefdb1447..8b0f0747e7a1909d1390866b8f561c8085637987 100644 --- a/src/slurmctld/fed_mgr.c +++ b/src/slurmctld/fed_mgr.c @@ -4141,22 +4141,6 @@ extern int fed_mgr_job_revoke(struct job_record *job_ptr, bool job_complete, job_ptr->end_time = start_time; job_ptr->state_reason = WAIT_NO_REASON; xfree(job_ptr->state_desc); - - /* - * Since the job is purged/revoked quickly on the non-origin side it's - * possible that the job_start message has not been sent yet. Send it - * now so that the db record gets the uid set -- which the complete - * message doesn't send. - */ - if (!job_ptr->db_index && (origin_id != fed_mgr_cluster_rec->fed.id)) { - if (IS_JOB_FINISHED(job_ptr)) - jobacct_storage_g_job_start(acct_db_conn, job_ptr); - else - info("%s: job (%u) isn't finished and isn't an origin job (%u != %u) and doesn't have a db_index yet. We aren't sending a start message to the database.", - __func__, job_ptr->job_id, origin_id, - fed_mgr_cluster_rec->fed.id); - } - job_completion_logger(job_ptr, false); /* Don't remove the origin job */