Skip to content
Snippets Groups Projects

Disable pipelines on default branch for non canonical projects

What does this MR do and why?

Disable pipelines on default branch for non canonical projects

We regularly see community contributors running out of pipeline minutes because they forgot to use -o ci.skip while pushing the updated master branch to their fork.

Currently we advise them to switch to the community fork to prevent them to use up their quota again, but that doesn't help them in the current month when they also use CI for other projects in their namespace.

We can disable these pipelines entirely as they serve no purpose on forks. This should help community contributors to not accidentially use up their entire pipeline minutes quota for the rest of the month.

Screenshots or screen recordings

Screenshots are required for UI changes, and strongly recommended for all other merge requests.

Before After

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

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
  • requested review from @nao.hashizume

  • @nao.hashizume, this Community contribution is ready for review.

    • Do you have capacity and domain expertise to review this? If not, find one or more reviewers and assign to them.
    • If you've reviewed it, add the workflowin dev label if these changes need more work before the next review.

    This message was generated automatically. You're welcome to improve it.

  • Allure report

    allure-report-publisher generated test report!

    e2e-test-on-gdk: :white_check_mark: test report for 8e390cd4

    expand test summary
    +-----------------------------------------------------------------------+
    |                            suites summary                             |
    +------------------+--------+--------+---------+-------+-------+--------+
    |                  | passed | failed | skipped | flaky | total | result |
    +------------------+--------+--------+---------+-------+-------+--------+
    | Verify           | 32     | 0      | 0       | 0     | 32    | ✅     |
    | Plan             | 55     | 0      | 0       | 0     | 55    | ✅     |
    | Create           | 40     | 0      | 6       | 0     | 46    | ✅     |
    | Data Stores      | 22     | 0      | 0       | 0     | 22    | ✅     |
    | Govern           | 48     | 0      | 0       | 0     | 48    | ✅     |
    | Package          | 0      | 0      | 1       | 0     | 1     | ➖     |
    | Monitor          | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Framework sanity | 0      | 0      | 1       | 0     | 1     | ➖     |
    | Manage           | 0      | 0      | 1       | 0     | 1     | ➖     |
    +------------------+--------+--------+---------+-------+-------+--------+
    | Total            | 201    | 0      | 9       | 0     | 210   | ✅     |
    +------------------+--------+--------+---------+-------+-------+--------+

    e2e-package-and-test: :x: test report for 8e390cd4

    expand test summary
    +-----------------------------------------------------------------------+
    |                            suites summary                             |
    +------------------+--------+--------+---------+-------+-------+--------+
    |                  | passed | failed | skipped | flaky | total | result |
    +------------------+--------+--------+---------+-------+-------+--------+
    | Create           | 539    | 0      | 101     | 0     | 640   | ✅     |
    | Plan             | 249    | 0      | 10      | 0     | 259   | ✅     |
    | Verify           | 155    | 1      | 9       | 0     | 165   | ❌     |
    | Fulfillment      | 8      | 0      | 69      | 0     | 77    | ✅     |
    | Govern           | 303    | 0      | 12      | 6     | 315   | ❗     |
    | Monitor          | 39     | 0      | 7       | 0     | 46    | ✅     |
    | Data Stores      | 117    | 0      | 3       | 0     | 120   | ✅     |
    | Manage           | 40     | 0      | 10      | 14    | 50    | ❗     |
    | Package          | 226    | 0      | 17      | 3     | 243   | ❗     |
    | Systems          | 8      | 0      | 0       | 0     | 8     | ✅     |
    | GitLab Metrics   | 2      | 0      | 1       | 0     | 3     | ✅     |
    | Configure        | 1      | 0      | 9       | 0     | 10    | ✅     |
    | Framework sanity | 0      | 0      | 5       | 0     | 5     | ➖     |
    | Release          | 15     | 0      | 3       | 0     | 18    | ✅     |
    | Analytics        | 7      | 0      | 0       | 0     | 7     | ✅     |
    | ModelOps         | 0      | 0      | 6       | 0     | 6     | ➖     |
    | Secure           | 6      | 0      | 3       | 0     | 9     | ✅     |
    | Growth           | 0      | 0      | 6       | 0     | 6     | ➖     |
    +------------------+--------+--------+---------+-------+-------+--------+
    | Total            | 1715   | 1      | 271     | 23    | 1987  | ❌     |
    +------------------+--------+--------+---------+-------+-------+--------+

    e2e-review-qa: :white_check_mark: test report for 8e390cd4

    expand test summary
    +-----------------------------------------------------------------------+
    |                            suites summary                             |
    +------------------+--------+--------+---------+-------+-------+--------+
    |                  | passed | failed | skipped | flaky | total | result |
    +------------------+--------+--------+---------+-------+-------+--------+
    | Govern           | 3      | 0      | 0       | 0     | 3     | ✅     |
    | Create           | 8      | 0      | 1       | 0     | 9     | ✅     |
    | Plan             | 3      | 0      | 1       | 0     | 4     | ✅     |
    | Monitor          | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Framework sanity | 0      | 0      | 1       | 0     | 1     | ➖     |
    | Package          | 0      | 0      | 1       | 0     | 1     | ➖     |
    | Data Stores      | 2      | 0      | 0       | 0     | 2     | ✅     |
    +------------------+--------+--------+---------+-------+-------+--------+
    | Total            | 20     | 0      | 4       | 0     | 24    | ✅     |
    +------------------+--------+--------+---------+-------+-------+--------+
    Edited by Ghost User
  • mentioned in issue #429121 (closed)

  • Nao Hashizume changed milestone to %16.6

    changed milestone to %16.6

  • Niklas van Schrick resolved all threads

    resolved all threads

  • added 1 commit

    • 9f204672 - Improve comment for workflow rule

    Compare with previous version

  • Nao Hashizume resolved all threads

    resolved all threads

  • Nao Hashizume approved this merge request

    approved this merge request

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading