ci_run_n_monitor: fix handling of optional jobs again
Looks like a recent rework broke this again, so let's add the existence check back. Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/27874>
This commit is contained in:

committed by
Marge Bot

parent
e4d45c582b
commit
656e329fa2
@@ -229,7 +229,7 @@ def traverse_dag_needs(jobs_metadata: Dag) -> None:
|
|||||||
partial = True
|
partial = True
|
||||||
|
|
||||||
while partial:
|
while partial:
|
||||||
next_depth: set[str] = {n for dn in final_needs for n in jobs_metadata[dn]["needs"]}
|
next_depth: set[str] = {n for dn in final_needs if dn in jobs_metadata for n in jobs_metadata[dn]["needs"]}
|
||||||
partial: bool = not final_needs.issuperset(next_depth)
|
partial: bool = not final_needs.issuperset(next_depth)
|
||||||
final_needs = final_needs.union(next_depth)
|
final_needs = final_needs.union(next_depth)
|
||||||
|
|
||||||
|
Reference in New Issue
Block a user