Skip to content
Snippets Groups Projects

Ensure only one deployment run at once on release-environment

1 unresolved thread

What does this MR do and why?

To prevent simultaneous execution of deploy and qa jobs in a release environment, the deploy and qa jobs triggered by the first pipeline will take precedence. Meanwhile, the deploy and qa jobs of subsequent pipelines will remain Waiting until the corresponding jobs in the first pipeline are completed.

Related Issue: gitlab-com/gl-infra/delivery#20008 (comment 1896145520)

MR acceptance checklist

Please evaluate this MR against the MR acceptance checklist. It helps you analyze changes to reduce risks in quality, performance, reliability, security, and maintainability.

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.

Edited by Mawreen Dela Cruz

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
  • Dat Tang requested changes

    requested changes

  • added 1 commit

    • b27eb9b5 - Use workflow:auto_cancel:on_new_commit instead of interruptible to control...

    Compare with previous version

  • requested review from @dat.tang.gitlab

  • Dat Tang approved this merge request

    approved this merge request

  • Looks good to me :tada: Thanks for the work @madelacruz . We have learned a lot about GitLab CI :laughing:

  • added pipelinetier-2 label and removed pipelinetier-1 label

  • Mawreen Dela Cruz requested review from @rymai

    requested review from @rymai

  • E2E Test Result Summary

    allure-report-publisher generated test report!

    e2e-test-on-gdk: :white_check_mark: test report for 553846f9

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Package     | 19     | 0      | 12      | 19    | 31    | ✅     |
    | Analytics   | 2      | 0      | 0       | 1     | 2     | ✅     |
    | Verify      | 31     | 0      | 1       | 30    | 32    | ✅     |
    | Create      | 101    | 0      | 9       | 94    | 110   | ✅     |
    | Plan        | 51     | 0      | 2       | 47    | 53    | ✅     |
    | Govern      | 66     | 0      | 0       | 43    | 66    | ✅     |
    | Manage      | 0      | 0      | 1       | 0     | 1     | ➖     |
    | Release     | 5      | 0      | 0       | 5     | 5     | ✅     |
    | Monitor     | 7      | 0      | 0       | 7     | 7     | ✅     |
    | Data Stores | 31     | 0      | 0       | 22    | 31    | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 313    | 0      | 25      | 268   | 338   | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
  • Rémy Coutable
  • Rémy Coutable
  • Rémy Coutable removed review request for @rymai

    removed review request for @rymai

  • added 1 commit

    • e7c9bb87 - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • Mawreen Dela Cruz reset approvals from @dat.tang.gitlab by pushing to the branch

    reset approvals from @dat.tang.gitlab by pushing to the branch

  • Hi @dat.tang.gitlab, I have added a small change to this MR following @rymai review suggestions. Could you take a look as well? Thank you!

    Edited by Mawreen Dela Cruz
  • added 1 commit

    • 8d57effb - Apply 1 suggestion(s) to 1 file(s)

    Compare with previous version

  • added pipelinetier-1 label and removed pipelinetier-2 label

  • added 1 commit

    Compare with previous version

  • Dat Tang requested review from @rymai

    requested review from @rymai

  • Dat Tang approved this merge request

    approved this merge request

  • added pipelinetier-2 label and removed pipelinetier-1 label

  • Hi @rymai, for your review and approval. Thank you!

    Edited by Mawreen Dela Cruz
  • Mawreen Dela Cruz requested review from @splattael and removed review request for @rymai

    requested review from @splattael and removed review request for @rymai

  • Mawreen Dela Cruz requested review from @rymai and removed review request for @splattael

    requested review from @rymai and removed review request for @splattael

  • Rémy Coutable approved this merge request

    approved this merge request

  • added pipelinetier-3 label and removed pipelinetier-2 label

  • Rémy Coutable resolved all threads

    resolved all threads

  • Rémy Coutable enabled an automatic merge when the pipeline for 553846f9 succeeds

    enabled an automatic merge when the pipeline for 553846f9 succeeds

  • @rymai, did you forget to run a pipeline before you merged this work? Based on our code review process, if the latest pipeline was created more than 4 hours ago, you should:

    1. Ensure the merge request is not in Draft status.
    2. Start a pipeline (especially important for Community contribution merge requests).
    3. Set the merge request to auto-merge.

    This is a guideline, not a rule. Please consider replying to this comment for transparency.

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

  • Rémy Coutable mentioned in commit c17ecf23

    mentioned in commit c17ecf23

  • changed milestone to %17.1

  • Dat Tang mentioned in commit 43955841

    mentioned in commit 43955841

  • Dat Tang mentioned in merge request !153104 (merged)

    mentioned in merge request !153104 (merged)

  • added workflowstaging label and removed workflowcanary label

  • Dat Tang mentioned in commit 8cd1745e

    mentioned in commit 8cd1745e

  • Mawreen Dela Cruz mentioned in merge request !153370 (merged)

    mentioned in merge request !153370 (merged)

  • Please register or sign in to reply
    Loading