-
Notifications
You must be signed in to change notification settings - Fork 240
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
keep storage class as cold for restored objects #5191
Conversation
Hello kerkesni,My role is to assist you with the merge of this 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 |
80a4eb1
to
0448287
Compare
0448287
to
6b7e56e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- i get this is missing some tests (in objectHead for exemple), to check he storage class is as expected when the object is 'restored
- there seems to be some extra processing of x-amz-storage-class for multipart : so we should ensure this change does not affect the behavior when restoring a multipart object (e.g. when cold backend would need to do a multipart put to restore)
x-amz-storage-class
seems to be used a bit everywhere: so i fear that in non-versionned (or version-suspended), the 'cold' storage class would remain set when the object is replaced... → should be tested as well?
6b7e56e
to
2cd5200
Compare
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:
|
Incorrect fix versionThe
Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:
Please check the |
e6a984d
to
aab13ce
Compare
To be compliant with the AWS S3 standard, the storage class of restored objects should be left as cold location Issue: CLDSRV-400
aab13ce
to
d26b8bc
Compare
We use the |
@nicolas2bert i've looked into backbeat and cloudserver, the only instances where i see the values of |
Incorrect fix versionThe
Considering where you are trying to merge, I ignored possible hotfix versions and I expected to find:
Please check the |
/approve |
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: 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:
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 |
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-400. Goodbye kerkesni. |
To be compliant with the AWS S3 standard, the storage class of restored objects should be left as cold location
Issue: CLDSRV-400