Merge pull request #492 from cmoussa1/add.return.to.exc

`job.state.inactive`: add `return -1` to exception
flux-framework · Oct 1, 2024 · b07f36c · b07f36c
2 parents 921b4ef + 3a61b6d
commit b07f36c
Showing 1 changed file with 3 additions and 0 deletions.
diff --git a/src/plugins/mf_priority.cpp b/src/plugins/mf_priority.cpp
@@ -1138,8 +1138,11 @@ static int inactive_cb (flux_plugin_t *p,
         if (flux_jobtap_dependency_remove (p,
                                            jobid,
                                            "max-running-jobs-user-limit") < 0)
+        {
             flux_jobtap_raise_exception (p, jobid, "mf_priority",
                                          0, "failed to remove job dependency");
+            return -1;
+        }
 
         b->held_jobs.erase (b->held_jobs.begin ());
     }