Follow-up from "Resolve "Notifications for when pipelines are fixed""
The following discussion from !16951 (merged) should be addressed:
-
@jacopo-beschi started a discussion: (+5 comments) @godfat @dosuken123 I think I found an issue with this, looks like the
>=
is not exact: when we retry a build might happen thatlast_updated_by_pipeline_id == pipeline.id
and in this case we won't update theci_ref.status
thus don't send the notification. I think this should be updated to use>
instead of>=
.wdyt?