Fix rendering pipelines for a new merge request

What does this MR do?

This MR fixes a bug with invalid pipelines API call, which leads to raising an exception.

Does this MR meet the acceptance criteria?

• Changelog entry added
• Added tests for this feature/bug

What are the relevant issue numbers?

See https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/8844#note_22930365

marked as a Work In Progress

added 1 commit

• 25c1f051 - Fix rendering pipelines for a new merge request

Compare with previous version

changed title from WIP: Pass proper endpoint path to MR pipelines partial to WIP: Fix rendering pipelines for a new merge request

marked the task Added tests for this feature/bug as completed

added 1 commit

• 1c73b764 - Add feature spec for a new merge request with pipeline

Compare with previous version

added 1 commit

• 5d25871c - Add Changelog entry for new merge request pipelines fix

Compare with previous version

unmarked as a Work In Progress

@ruben @felipe_artur @selfup FYI - we'll need this before tagging 8.17

marked the task Changelog entry added as completed

added 1 commit

• edebf538 - Update _pipelines.html.haml

Compare with previous version

mentioned in merge request !8844 (merged)

added 1 commit

• 1d0820c7 - Fix broken specs from the Frontend side

Compare with previous version

added 1 commit

• 3ea1f91e - Fix merge request controller specs for new action

Compare with previous version

@grzesiek I pushed a commit that fixes some related broken tests. //cc @mikegreiling

mentioned in merge request !9038 (merged)

@grzesiek @filipa should this MR be assigned to someone?

changed milestone to %8.17

@jameslopez It should! But not sure which endboss is available now. /cc @rspeicher @rymai @ayufan

@ayufan can you please review this? :)

assigned to @ayufan

added 1 commit

• b40f7147 - fix broken frontend tests

Compare with previous version

The frontend portions LGTM

I committed a quick fix for another broken frontend test which @filipa can review for me. After that it should be ready for merge.

Thank you @mikegreiling for this fixes! it LGTM! :)

I believe Frontend is good to go! :)

Make sure if the 8.17-rc1 stable branch has already been made that we ~"Pick into Stable" for this one. I don't want broken frontend tests going out in this release. I also have !9038 (merged) waiting to go in which depends on this being merged first.

resolved all discussions

added 1 commit

• bf6a0e47 - Remove changelog entry; Changes after review

Compare with previous version

@fatihacet can you please review? Thank you!

Backend changes were already reviewed :)

assigned to @fatihacet

enabled an automatic merge when the pipeline for bf6a0e47 succeeds

Enabled auto merge, thanks @grzesiek @filipa and @mikegreiling

mentioned in issue #27252 (closed)

added 1 commit

• d410c937 - fix lint error

Compare with previous version

enabled an automatic merge when the pipeline for d410c937 succeeds

merged

mentioned in commit 71c80375

resolved all discussions

mentioned in issue #27866 (closed)

resolved all discussions