impr(ARSN-475): Add bucketOwnerId field to ObjectMD model#2337
Conversation
Hello tmacro,My role is to assist you with the merge of this Available options
Available commands
Status report is not available. |
ghost
left a comment
There was a problem hiding this comment.
You might want to bump arsenal as well before merging 🙂
| // object is a placeholder and is not up to date. | ||
| export const mdModelVersion = 6; | ||
| // Version 7 adds the "bucketOwnerId" attribute to the object metadata. | ||
| // This is set when the owner of the bucket is different from the owner-id of the object. |
There was a problem hiding this comment.
Shall we add a bit more context here? To explain why we need it.
There was a problem hiding this comment.
at least describe that owner-id is the requester account id for cross account put object
Waiting for approvalThe following approvals are needed before I can proceed with the merge:
|
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## development/8.2 #2337 +/- ##
===================================================
+ Coverage 69.41% 69.42% +0.01%
===================================================
Files 217 217
Lines 17402 17409 +7
Branches 3606 3606
===================================================
+ Hits 12080 12087 +7
Misses 5318 5318
Partials 4 4 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
dde44eb to
a01688c
Compare
|
/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-475. Goodbye tmacro. The following options are set: approve |
No description provided.