scality / Arsenal

Common utilities for the open-source Scality S3 project components
Apache License 2.0
15 stars 19 forks source link

Improvement/arsn 423 bump dependencies #2266

Open benzekrimaha opened 1 month ago

benzekrimaha commented 1 month ago

Issue : ARSN-423

bert-e commented 1 month ago

Hello benzekrimaha,

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 commented 1 month ago

Conflict

There is a conflict between your branch improvement/ARSN-423-bump-dependencies and the destination branch development/8.2.

Please resolve the conflict on the feature branch (improvement/ARSN-423-bump-dependencies).

git fetch && \
git checkout origin/improvement/ARSN-423-bump-dependencies && \
git merge origin/development/8.2

Resolve merge conflicts and commit

git push origin HEAD:improvement/ARSN-423-bump-dependencies
bert-e commented 1 month ago

Waiting for approval

The following approvals are needed before I can proceed with the merge:

codecov[bot] commented 1 month ago

Codecov Report

Attention: Patch coverage is 68.60068% with 92 lines in your changes missing coverage. Please review.

Project coverage is 65.06%. Comparing base (fed5807) to head (03503cb).

Files with missing lines Patch % Lines
lib/policyEvaluator/RequestContext.ts 55.76% 23 Missing :warning:
lib/patches/locationConstraints.ts 81.81% 10 Missing :warning:
lib/auth/backends/ChainBackend.ts 27.27% 8 Missing :warning:
lib/network/kmsAWS/Client.ts 71.42% 8 Missing :warning:
lib/s3routes/routes/routeDELETE.ts 0.00% 6 Missing :warning:
lib/s3routes/routesUtils.ts 0.00% 5 Missing :warning:
lib/metrics/RedisClient.ts 0.00% 4 Missing :warning:
lib/s3middleware/azureHelpers/mpuUtils.ts 0.00% 4 Missing :warning:
lib/auth/auth.ts 57.14% 3 Missing :warning:
lib/clustering/ClusterRPC.ts 0.00% 3 Missing :warning:
... and 14 more
Additional details and impacted files ```diff @@ Coverage Diff @@ ## development/8.2 #2266 +/- ## =================================================== + Coverage 64.91% 65.06% +0.15% =================================================== Files 215 215 Lines 17299 17312 +13 Branches 3514 3574 +60 =================================================== + Hits 11229 11264 +35 + Misses 6055 6042 -13 + Partials 15 6 -9 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.


🚨 Try these New Features:

bert-e commented 1 week ago

Waiting for approval

The following approvals are needed before I can proceed with the merge:

benzekrimaha commented 1 week ago

/approve

benzekrimaha commented 1 week ago

/wait

bert-e commented 1 week ago

Waiting for approval

The following approvals are needed before I can proceed with the merge:

The following options are set: approve