Skip to content

Commit

Permalink
Fix the seq mapper
Browse files Browse the repository at this point in the history
Just move on if the node is full - we'll catch any
oversubscription if/when we try to place another
process on that node.

Signed-off-by: Ralph Castain <[email protected]>
  • Loading branch information
rhc54 committed Nov 20, 2023
1 parent 9bd351f commit ee5ca2e
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion src/mca/rmaps/seq/rmaps_seq.c
Original file line number Diff line number Diff line change
Expand Up @@ -359,7 +359,8 @@ static int prte_rmaps_seq_map(prte_job_t *jdata,
goto error;
}
rc = prte_rmaps_base_check_oversubscribed(jdata, app, node, options);
if (PRTE_SUCCESS != rc) {
if (PRTE_SUCCESS != rc &&
PRTE_ERR_TAKE_NEXT_OPTION != rc) {
PMIX_RELEASE(proc);
goto error;
}
Expand Down

0 comments on commit ee5ca2e

Please sign in to comment.