Skip to content

ci: remove sanity job as a prerequisite.

Helen Mae Koike Fornazier requested to merge helen.fornazier/mesa:sanity into main

The CI executes the sanity job as a prerequisite before proceeding with the remaining tasks. While this approach ensures that all tests starts with a sanity check, it also introduces the potential for rejections by Marge due to prolonged runner queue times.

From only two days of observation, this issue can be found on these cases:

Since this issue happens more often than a failure on sanity job, I propose to remove this requirement to increase Marge success rate.

In the future, expanding our runners pool or implementing job prioritization for Marge may allow us to return to the original order. However, for now, this change optimizes our workflow.

Please, let me know your thoughts.

I know this is not ideal, but Marge's queue is usually long, and it takes time to get things merged, so maybe we can improve a bit while we work on a proper solution.

cc @daniels @anholt @dh @gallo @eric

Edited by Helen Mae Koike Fornazier

Merge request reports