Builds were marked retroactively as failures if any had to be retried
I noticed this on the MRs linked from the Rails 5 issue: https://gitlab.com/gitlab-org/gitlab-ce/issues/14286
Lots of them needed retries due to a bug with CI and are now marked as failures, even though all builds ended up succeeding. Previously they had been marked as successful. I think this is a regression from 8.8? I'm not sure if this only effects builds retroactively or if this would happen on all retried builds.