CLDSRV-961 Fix lifecycle listings stuck on PHD master keys - #6310
Conversation
Pin arsenal to 8.4.26 (8.5.17 on 9.4), which fixes lifecycle listings (DelimiterNonCurrent, DelimiterOrphanDeleteMarker) getting stuck forever on a run of dangling PHD master keys longer than the scan cap. bucketd runs the actual lifecycle listing, not cloudserver, so also repin the MetaData image to the released ghcr.io/scality/metadata:9.17.0-standalone tag (MD-1359): the s3c legs need the fix there too, not only in the pinned arsenal. Add a functional suite proving the real stack -- an actual metadata backend that writes PHD masters and races a repair timer, through the real backbeat route -- makes forward progress across such a desert. The listing algorithm itself is exhaustively unit-tested in Arsenal against both v0 and v1 (scality/Arsenal#2685); this suite intentionally does not re-derive those cases and stays scoped to end-to-end wiring. (cherry picked from commit d14b326)
Hello nicolas2bert,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Incorrect fix versionThe
Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:
Please check the |
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 |
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/bugfix/CLDSRV-961/phd-hf origin/development/9.4
git merge origin/bugfix/CLDSRV-961/phd-hf
# <intense conflict resolution>
git commit
git push -u origin w/9.4/bugfix/CLDSRV-961/phd-hfThe following options are set: create_integration_branches |
Integration data createdI have created the integration data for the additional destination branches.
The following branches will NOT be impacted:
You can set option 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 |
|
@bert-e approve |
Build failedThe build for commit did not succeed in branch bugfix/CLDSRV-961/phd-hf The following options are set: approve, create_integration_branches |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files
@@ Coverage Diff @@
## development/9.3 #6310 +/- ##
===================================================
- Coverage 85.32% 85.28% -0.04%
===================================================
Files 206 206
Lines 13435 13435
===================================================
- Hits 11463 11458 -5
- Misses 1972 1977 +5
Flags with carried forward coverage won't be shown. Click here to find out more. 🚀 New features to boost your workflow:
|
Build failedThe build for commit did not succeed in branch w/9.4/bugfix/CLDSRV-961/phd-hf The following options are set: approve, create_integration_branches |
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 |
|
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue CLDSRV-961. Goodbye nicolas2bert. |
Pin arsenal to 8.4.26 (8.5.17 on 9.4), which fixes lifecycle listings (DelimiterNonCurrent,
DelimiterOrphanDeleteMarker) getting stuck forever on a run of dangling PHD
master keys longer than the scan cap. bucketd runs the actual lifecycle
listing, not cloudserver, so also repin the MetaData image to the released
ghcr.io/scality/metadata:9.17.0-standalone tag (MD-1359): the s3c legs need
the fix there too, not only in the pinned arsenal.
Add a functional suite proving the real stack -- an actual metadata backend
that writes PHD masters and races a repair timer, through the real backbeat
route -- makes forward progress across such a desert. The listing algorithm
itself is exhaustively unit-tested in Arsenal against both v0 and v1
(scality/Arsenal#2685); this suite intentionally does not re-derive those
cases and stays scoped to end-to-end wiring.
(cherry picked from commit d14b326)