Skip to content
Snippets Groups Projects

Fix pipeline status update in pipeline editor

Merged Mireya Andres requested to merge pipeline-editor/fix-status-update into master
All threads resolved!

What does this MR do?

Related to #335074 (closed).

We refactored the pipeline editor to fetch the commitSha using graphql. This fixes a bug related to the refactor where the pipeline status does not update correctly when committing changes to the current branch.

When committing a new branch, it's possible that the pipeline data is not yet ready when the vue app fetches it, so we poll for the data until it is received. This is also the case for committing changes to the current branch (it seems we're fetching too early).

Since the bug fix is for a change that was released in the current milestone, no changelog trailers were added.

Screenshots or Screencasts

Committing to current branch

How to setup and validate locally (strongly suggested)

  1. Go to the Pipeline Editor in CI/CD > Editor,
  2. At the bottom of the page, submit a commit to the same branch you are in.
  3. Wait a few seconds and verify that the pipeline status is updated with the new pipeline and commit.

Does this MR meet the acceptance criteria?

Conformity

Availability and Testing

Edited by Mireya Andres

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Ezekiel Kigbo
  • Ezekiel Kigbo
  • Mireya Andres added 329 commits

    added 329 commits

    Compare with previous version

  • Ezekiel Kigbo approved this merge request

    approved this merge request

  • Ezekiel Kigbo resolved all threads

    resolved all threads

  • Ezekiel Kigbo enabled an automatic merge when the pipeline for 06a796fb succeeds

    enabled an automatic merge when the pipeline for 06a796fb succeeds

  • merged

  • Ezekiel Kigbo mentioned in commit 5bc7fffe

    mentioned in commit 5bc7fffe

  • added workflowcanary label and removed workflowstaging label

  • added workflowproduction label and removed workflowcanary label

  • Please register or sign in to reply
    Loading