ARSN-607: Skip head-check for scality backends (HF S3C 9.5.2)#2663
Conversation
MultipleBackendGateway.head() feature-detects the location client's head() method and then passes it the whole objectGetInfo object. sproxydclient 8.2.0+ added a head(key) method that asserts a 40-char string key, so once arsenal bumped to it (ARSN-606) the head-check run during object GET crashed the worker with an uncaught AssertionError ('object' !== 'string') on multiple-backend sproxyd locations. Before 8.2.0 sproxydclient had no head(), so the feature-detect skipped it for sproxyd and this path stayed dormant. The head-check only verifies external cloud backend data state and sproxyd was never head-checked; skip scality clients, mirroring the clientType === 'scality' handling in get()/delete(). Issue: ARSN-607 (cherry picked from commit 5fde2d1)
Hello bourgoismickael,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
|
Codecov Report❌ Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## hotfix/8.2.48 #2663 +/- ##
=================================================
+ Coverage 72.00% 72.04% +0.04%
=================================================
Files 222 222
Lines 17954 17955 +1
Branches 3738 3717 -21
=================================================
+ Hits 12927 12935 +8
+ Misses 5023 5016 -7
Partials 4 4 ☔ View full report in Codecov by Harness. 🚀 New features to boost your workflow:
|
|
/approve |
|
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
Please check the status of the associated issue ARSN-607. Goodbye bourgoismickael. The following options are set: approve |
Cherry-pick from PR #2662 for S3C 9.5.2 HF