Improvement/cldsrv 966/support anon req limiting - #6249
Conversation
Hello tmacro,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Request integration branchesWaiting for integration branch creation to be requested by the user. To request integration branches, please comment on this pull request with the following command: Alternatively, the |
|
/create_integration_branches |
1130021 to
780528d
Compare
|
/reset |
Codecov Report❌ Patch coverage is
Additional details and impacted files
... and 2 files with indirect coverage changes @@ Coverage Diff @@
## development/9.3 #6249 +/- ##
===================================================
- Coverage 85.20% 85.18% -0.02%
===================================================
Files 206 206
Lines 13424 13434 +10
===================================================
+ Hits 11438 11444 +6
- Misses 1986 1990 +4
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
The following options are set: create_integration_branches |
![github-advanced-security[bot]](https://avatars.githubusercontent.com/in/57789?s=60&v=4){kind=small}
55e4579 to
649e967
Compare
History mismatchMerge commit #803a6f1c6e2932a9b1b7fef322318037162703e2 on the integration branch It is likely due to a rebase of the branch Please use the The following options are set: create_integration_branches |
|
/reset |
Lossy reset warningThere seems to be manual commits on integration branches (e.g. conflict You can use the The following options are set: create_integration_branches |
History mismatchMerge commit #803a6f1c6e2932a9b1b7fef322318037162703e2 on the integration branch It is likely due to a rebase of the branch Please use the The following options are set: create_integration_branches |
|
/force_reset |
Reset completeI have successfully deleted this pull request's integration branches. The following options are set: create_integration_branches |
ConflictA conflict has been raised during the creation of I have not created the integration branch. Here are the steps to resolve this conflict: git fetch
git checkout -B w/9.4/improvement/CLDSRV-966/support_anon_req_limiting origin/development/9.4
git merge origin/improvement/CLDSRV-966/support_anon_req_limiting
# <intense conflict resolution>
git commit
git push -u origin w/9.4/improvement/CLDSRV-966/support_anon_req_limitingThe following options are set: create_integration_branches |
ConflictA conflict has been raised during the update of Please resolve the conflict on the integration branch ( Here are the steps to resolve this conflict: git fetch
git checkout w/9.4/improvement/CLDSRV-966/support_anon_req_limiting
git pull # or "git reset --hard origin/w/9.4/improvement/CLDSRV-966/support_anon_req_limiting"
git merge origin/development/9.4
# <intense conflict resolution>
git commit
git merge origin/improvement/CLDSRV-966/support_anon_req_limiting
# <intense conflict resolution>
git commit
git push -u origin w/9.4/improvement/CLDSRV-966/support_anon_req_limitingThe following options are set: create_integration_branches |
|
/reset |
Reset completeI have successfully deleted this pull request's integration branches. The following options are set: create_integration_branches |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
The following options are set: create_integration_branches |
|
/approve |
In the queueThe changeset has received all authorizations and has been added to the The changeset will be merged in:
The following branches will NOT be impacted:
This pull request does not target the following hotfix branch(es) so they
There is no action required on your side. You will be notified here once IMPORTANT Please do not attempt to modify this pull request.
If you need this pull request to be removed from the queue, please contact a The following options are set: approve, create_integration_branches |
Queue build failedThe corresponding build for the queue failed:
Remove the pull request from the queue
|
5 participants
Support rate limiting anonymous requests & cross-account requests when there is a cold bucketOwner cache.
If the account rate limit config has not been retrieved from Vault during request auth the an extra call to Vault is done to retrieve the bucket owner's rate limit config.
Another small improvement is included (CLDSRV-969) to allow setting burst capacity to
0.