Skip to content
Snippets Groups Projects

Set AutoMergeProcessWorker data consistency to sticky

Merged drew stachon requested to merge set-auto-merge-process-worker-to-sticky into master
All threads resolved!

What does this MR do and why?

Set AutoMergeProcessWorker data consistency to sticky

There's not a lot of immediate benefit to doing this, but we are preparing some other changes that will move a very high-volume read query out of the PipelineProcessWorker and into this worker, where it will hit replicas before the merge process actually starts.

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.

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
  • Marc Shaw removed review request for @marc_shaw

    removed review request for @marc_shaw

  • drew stachon added 626 commits

    added 626 commits

    Compare with previous version

  • drew stachon added 1 commit

    added 1 commit

    • 82cce8c1 - Use feature flag for AutoMergeProcessWorker consistency

    Compare with previous version

  • drew stachon requested review from @marc_shaw

    requested review from @marc_shaw

  • mentioned in issue #496567 (closed)

  • drew stachon added 32 commits

    added 32 commits

    • 82cce8c1...a4678d1e - 30 commits from branch master
    • 183de654 - Set AutoMergeProcessWorker data consistency to sticky
    • 7f8c2160 - Use feature flag for AutoMergeProcessWorker consistency

    Compare with previous version

  • drew stachon added 62 commits

    added 62 commits

    • 7f8c2160...b40fdd42 - 60 commits from branch master
    • 6e51077d - Set AutoMergeProcessWorker data consistency to sticky
    • 318cbf66 - Use feature flag for AutoMergeProcessWorker consistency

    Compare with previous version

  • Marc Shaw requested review from @alejandro

    requested review from @alejandro

  • Marc Shaw approved this merge request

    approved this merge request

  • Marc Shaw removed review request for @marc_shaw

    removed review request for @marc_shaw

  • Before you set this MR to auto-merge

    This merge request will progress on pipeline tiers until it reaches the last tier: pipelinetier-3. We will trigger a new pipeline for each transition to a higher tier.

    Before you set this MR to auto-merge, please check the following:

    • You are the last maintainer of this merge request
    • The latest pipeline for this merge request is pipelinetier-3 (You can find which tier it is in the pipeline name)
    • This pipeline is recent enough (created in the last 8 hours)

    If all the criteria above apply, please set auto-merge for this merge request.

    See pipeline tiers and merging a merge request for more details.

  • E2E Test Result Summary

    allure-report-publisher generated test report!

    e2e-test-on-gdk: :white_check_mark: test report for ba7ad56b

    expand test summary
    +------------------------------------------------------------------+
    |                          suites summary                          |
    +-------------+--------+--------+---------+-------+-------+--------+
    |             | passed | failed | skipped | flaky | total | result |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Create      | 128    | 0      | 18      | 0     | 146   | ✅     |
    | Plan        | 76     | 0      | 0       | 1     | 76    | ✅     |
    | Govern      | 73     | 0      | 0       | 0     | 73    | ✅     |
    | Verify      | 90     | 0      | 4       | 0     | 94    | ✅     |
    | Package     | 24     | 0      | 11      | 0     | 35    | ✅     |
    | Monitor     | 8      | 0      | 0       | 0     | 8     | ✅     |
    | Data Stores | 33     | 0      | 1       | 0     | 34    | ✅     |
    | Release     | 5      | 0      | 0       | 0     | 5     | ✅     |
    | Analytics   | 2      | 0      | 0       | 0     | 2     | ✅     |
    | Secure      | 4      | 0      | 0       | 0     | 4     | ✅     |
    | Manage      | 1      | 0      | 1       | 0     | 2     | ✅     |
    | Fulfillment | 2      | 0      | 0       | 0     | 2     | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+
    | Total       | 446    | 0      | 35      | 1     | 481   | ✅     |
    +-------------+--------+--------+---------+-------+-------+--------+

    e2e-test-on-omnibus: :white_check_mark: test report for 318cbf66

    expand test summary
    +-------------------------------------------------------------+
    |                       suites summary                        |
    +--------+--------+--------+---------+-------+-------+--------+
    |        | passed | failed | skipped | flaky | total | result |
    +--------+--------+--------+---------+-------+-------+--------+
    | Verify | 100    | 0      | 30      | 2     | 130   | ✅     |
    +--------+--------+--------+---------+-------+-------+--------+
    | Total  | 100    | 0      | 30      | 2     | 130   | ✅     |
    +--------+--------+--------+---------+-------+-------+--------+
  • drew stachon added 275 commits

    added 275 commits

    • 318cbf66...6a920f22 - 273 commits from branch master
    • 902e78b7 - Set AutoMergeProcessWorker data consistency to sticky
    • 2c3f73f8 - Use feature flag for AutoMergeProcessWorker consistency

    Compare with previous version

  • drew stachon added 2 commits

    added 2 commits

    • 183de654 - Set AutoMergeProcessWorker data consistency to sticky
    • 3002d382 - Use feature flag for AutoMergeProcessWorker consistency

    Compare with previous version

  • drew stachon reset approvals from @marc_shaw by pushing to the branch

    reset approvals from @marc_shaw by pushing to the branch

  • drew stachon added 1 commit

    added 1 commit

    • 17817553 - Use feature flag for AutoMergeProcessWorker consistency

    Compare with previous version

  • drew stachon added 335 commits

    added 335 commits

    • 17817553...6a920f22 - 333 commits from branch master
    • ff6138bd - Set AutoMergeProcessWorker data consistency to sticky
    • 3d0be075 - Use feature flag for AutoMergeProcessWorker consistency

    Compare with previous version

  • drew stachon removed review request for @alejandro

    removed review request for @alejandro

  • drew stachon requested review from @alejandro and @marc_shaw

    requested review from @alejandro and @marc_shaw

  • drew stachon removed review request for @marc_shaw

    removed review request for @marc_shaw

  • drew stachon added 24 commits

    added 24 commits

    • 3d0be075...4327e7d6 - 22 commits from branch master
    • 59d99527 - Set AutoMergeProcessWorker data consistency to sticky
    • ba7ad56b - Use feature flag for AutoMergeProcessWorker consistency

    Compare with previous version

  • Alejandro Rodríguez approved this merge request

    approved this merge request

  • Alejandro Rodríguez resolved all threads

    resolved all threads

  • Alejandro Rodríguez enabled automatic add to merge train when checks pass

    enabled automatic add to merge train when checks pass

  • Alejandro Rodríguez added this merge request to the merge train at position 2

    added this merge request to the merge train at position 2

  • mentioned in commit f8e1d3d5

  • added workflowstaging label and removed workflowcanary label

  • mentioned in issue #483008 (closed)

  • drew stachon mentioned in merge request !168123 (merged)

    mentioned in merge request !168123 (merged)

  • drew stachon mentioned in merge request !177639 (merged)

    mentioned in merge request !177639 (merged)

  • Please register or sign in to reply
    Loading