Show onward replication status on replicas#6224
Conversation
Hello maeldonn,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 |
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
|
7ca11e8 to
e34472f
Compare
❌ 1 Tests Failed:
View the full list of 1 ❄️ flaky test(s)
To view more test analytics, go to the Test Analytics Dashboard |
e34472f to
3195b2d
Compare
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
The following reviewers are expecting changes from the author, or must review again: |
Report the onward status (PENDING/PROCESSING/FAILED) instead of always REPLICA, and add x-amz-meta-replica to keep the replica flag visible. Issue: CLDSRV-931
3195b2d to
a6adc2d
Compare
SylvainSenechal
left a comment
There was a problem hiding this comment.
Maybe can ask Florent if he wants
responseMetaHeaders['x-amz-meta-scal-replica'] = 'false';
When the object is not a replica, or if current solution (returns nothing is enough)
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
|
|
@SylvainSenechal We omit this header to maintain consistency with the other headers. |
|
/approve |
|
I have successfully merged the changeset of this pull request
The following branches have NOT changed:
This pull request did not target the following hotfix branch(es) so they
Please check the status of the associated issue CLDSRV-931. Goodbye maeldonn. The following options are set: approve |
Report the onward status (PENDING/PROCESSING/FAILED) instead of always REPLICA, and add x-amz-meta-replica to keep the replica flag visible.
Issue: CLDSRV-931