cloudserver icon indicating copy to clipboard operation
cloudserver copied to clipboard

fix concurrent mpu completions

Open SylvainSenechal opened this issue 5 months ago • 3 comments

Issue: CLDSRV-687

Related : https://github.com/scality/Arsenal/pull/2459

SylvainSenechal avatar Jul 22 '25 13:07 SylvainSenechal

Hello sylvainsenechal,

My role is to assist you with the merge of this pull request. Please type @bert-e help to get information on this process, or consult the user documentation.

Available options
name description privileged authored
/after_pull_request Wait for the given pull request id to be merged before continuing with the current one.
/bypass_author_approval Bypass the pull request author's approval :star:
/bypass_build_status Bypass the build and test status :star:
/bypass_commit_size Bypass the check on the size of the changeset TBA :star:
/bypass_incompatible_branch Bypass the check on the source branch prefix :star:
/bypass_jira_check Bypass the Jira issue check :star:
/bypass_peer_approval Bypass the pull request peers' approval :star:
/bypass_leader_approval Bypass the pull request leaders' approval :star:
/approve Instruct Bert-E that the author has approved the pull request. :writing_hand:
/create_pull_requests Allow the creation of integration pull requests.
/create_integration_branches Allow the creation of integration branches.
/no_octopus Prevent Wall-E from doing any octopus merge and use multiple consecutive merge instead
/unanimity Change review acceptance criteria from one reviewer at least to all reviewers
/wait Instruct Bert-E not to run until further notice.
Available commands
name description privileged
/help Print Bert-E's manual in the pull request.
/status Print Bert-E's current status in the pull request TBA
/clear Remove all comments from Bert-E from the history TBA
/retry Re-start a fresh build TBA
/build Re-start a fresh build TBA
/force_reset Delete integration branches & pull requests, and restart merge process from the beginning.
/reset Try to remove integration branches unless there are commits on them which do not appear on the source branch.

Status report is not available.

bert-e avatar Jul 22 '25 13:07 bert-e

Incorrect fix version

The Fix Version/s in issue CLDSRV-687 contains:

  • None

Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:

  • 9.1.0

Please check the Fix Version/s of CLDSRV-687, or the target branch of this pull request.

bert-e avatar Jul 22 '25 13:07 bert-e

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 75.91%. Comparing base (615f831) to head (429693e). Report is 25 commits behind head on development/9.1.

:white_check_mark: All tests successful. No failed tests found.

Additional details and impacted files

Impacted file tree graph

Files with missing lines Coverage Δ
lib/services.js 83.38% <ø> (ø)
@@               Coverage Diff                @@
##           development/9.1    #5879   +/-   ##
================================================
  Coverage            75.91%   75.91%           
================================================
  Files                  188      188           
  Lines                11976    11976           
================================================
  Hits                  9091     9091           
  Misses                2885     2885           
Flag Coverage Δ
ceph-backend-test 45.44% <ø> (ø)
file-ft-tests 47.33% <ø> (ø)
kmip-ft-tests 26.99% <ø> (ø)
mongo-v0-ft-tests 47.77% <ø> (ø)
mongo-v1-ft-tests 47.78% <ø> (ø)
multiple-backend 33.98% <ø> (ø)
sur-tests 35.47% <ø> (+0.83%) :arrow_up:
sur-tests-inflights 36.55% <ø> (ø)
unit 66.93% <ø> (ø)
utapi-v2-tests 33.34% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:rocket: New features to boost your workflow:
  • :package: JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

codecov[bot] avatar Jul 22 '25 13:07 codecov[bot]