Pipeline not updated in MR widget when the latest one is skipped
Description
In this merge request pipeline has not been updated in the merge request widget. The latest one has been skipped, so maybe this is the problem. See screenshot blow.
/cc @ayufan
Designs
- Show closed items
Activity
-
Newest first Oldest first
-
Show all activity Show comments only Show history only
- Author Maintainer
Testing this in https://gitlab.com/grzesiek/ci-cd-sandbox/merge_requests/4.
- Author Maintainer
This bug is still present and I was able to reproduce it on gitlab.com.
- Author Maintainer
- Grzegorz Bizon assigned to @grzesiek
assigned to @grzesiek
- Grzegorz Bizon changed milestone to %10.1
changed milestone to %10.1
- Grzegorz Bizon added Deliverable In dev labels
added Deliverable In dev labels
- Author Maintainer
@bikebilly @bikebilly I just added a Deliverable label, this will be part of https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/14482.
- Grzegorz Bizon mentioned in merge request !14482 (merged)
mentioned in merge request !14482 (merged)
- Grzegorz Bizon added In review and removed In dev labels
- Grzegorz Bizon mentioned in commit 057a8b70
mentioned in commit 057a8b70
- Author Maintainer
@bikebilly I think that if MWPS is set, then
skipped
pipeline is clearly not successful, so we should not merge. - Author Maintainer
This means that when someone clicks "MWPS" a green pipeline is expected, not a skipped one, to merge.
- Author Maintainer
@bikebilly I think it is important to show latest pipeline in the widget, regardless of a status. Currently it should not affect MWPS.
- Author Maintainer
Note that this is In review already.
- Author Maintainer
My comment was to say: should we consider that for MWPS?
We do, @bikebilly. Only a successful pipeline can trigger MWPS, and it has be to be a pipeline associated with the latest commit in the repository.
- Kamil Trzcińśki closed via merge request !14482 (merged)
closed via merge request !14482 (merged)
- Kamil Trzcińśki mentioned in commit 0bba522f
mentioned in commit 0bba522f
- Nikola Milojevic mentioned in merge request !14785
mentioned in merge request !14785