Skip to content

Next

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
    • Help
    • Support
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
GitLab FOSS
GitLab FOSS
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
    • Cycle Analytics
    • Insights
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
    • Charts
    • Locked Files
  • Issues 61
    • Issues 61
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge Requests 0
    • Merge Requests 0
  • CI / CD
    • CI / CD
    • Pipelines
    • Jobs
    • Schedules
    • Charts
  • Security & Compliance
    • Security & Compliance
    • Dependency List
  • Packages
    • Packages
    • Container Registry
  • Snippets
    • Snippets
  • Members
    • Members
  • Collapse sidebar
  • Activity
  • Graph
  • Charts
  • Create a new issue
  • Jobs
  • Commits
  • Issue Boards
  • GitLab.org
  • GitLab FOSSGitLab FOSS
  • Issues
  • #34823

Closed
Open
Opened Jul 07, 2017 by Kamil Trzciński@ayufan🔴
  • Report abuse
  • New issue
Report abuse New issue

Handle missing pipeline in merge request widget

Resources

FE

We had to revert this MR as it doesn't work nice with External CI Services: https://gitlab.com/gitlab-org/gitlab-ce/merge_requests/12127#note_33854371.

We should get back to it and re-implement handling of ci_status for merge request widget, to properly support internal, external pipelines, and legacy old CI services. Currently we derive this message from various sources:

  • has_ci=true, status=running and head_pipeline=nil is yet another valid case => it indicates external CI service using polling mode,

  • has_ci=true, status=running and head_pipeline!=nil => it indicates either external or internal CI pipeline, the CI permissions doesn't apply here,

  • if has_ci=false then head_pipeline will always be nil => it indicates that pipelines are disabled,

  • we don't have a way to check if external pipeline should be created if GitLab CI is disabled.

We should instead have a single source of trouth, probably detailed CI merge request status that would be handled by frontend and provided by backend.

This status would represent exactly the states:

  • cannot connect to CI services: if this is external legacy CI service,
  • waiting for Pipeline: if this is internal GitLab CI,
  • (no information/or showing waiting): if this is external CI service (Jenkins, using Commit Status).
Edited Jul 17, 2017 by Filipa Lacerda

Related issues

  • Discussion
  • Designs
Assignee
Assign to
Backlog
Milestone
Backlog
Assign milestone
Time tracking
None
Due date
None
13
Labels
Accepting merge requests Create [DEPRECATED] P4 S3 Verify [DEPRECATED] backend bug default-priority default-severity devops::verify frontend merge requests technical debt
Assign labels
  • View project labels
Reference: gitlab-org/gitlab-foss#34823