/retest should only re-run failed pipelines

[pierDipi]

If a few pipelines are failing, there is no way for us to re-test only the failed ones.

Restarting all of them is a waste of resources and increases the chance of getting transient errors on pipelines (registry temporarily down, etc)

This would be similar to how Prow works: https://github.com/kubernetes/community/blob/master/contributors/guide/pull-requests.md

[chmouel]

That's a good point but unfortunately this ship has sailed since we can't change the default behavior, we can tho introduce a /retest failed (assuming there is no PR named failed)

[pierDipi]

/retest failed (assuming there is no PR named failed)

to overcome the problem with potential names I guess a new comment like /retest-failed would solve that?

[chmouel]

added this jira issue https://issues.redhat.com/browse/SRVKP-7236 so we can track this internally

[chmouel]

we can keep #1959

[pierDipi]

@chmouel is this issue actually the same as that one?

This one is more about re-running failed pipelines without re-running them all or listing each individually while the other issue is more about an expected behavior / bug