Skip to content
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

Object locking update #291

Merged
merged 5 commits into from
Jun 12, 2024
Merged

Object locking update #291

merged 5 commits into from
Jun 12, 2024

Conversation

fghaas
Copy link
Contributor

@fghaas fghaas commented Jun 11, 2024

This is a follow-up from the accidentally closed PR #289.

revij added 2 commits June 11, 2024 16:00
…bucket

In our object locking guide, we reference the "put-object-retention" commmand
instead of the "put-object-lock-configuration" which is used to set the default
object lock configuration for newly uploaded objects - without having the user
to set it on a per-object level.
We previously referred the use of "put-legal-hold" aws cli command
which is wrong. The correct command to use is "put-object-legal-hold"
since you can only apply a legal hold to a single object at a time.
Also included an alternative way to set the legal hold for multiple
objects using the aws cli.
@fghaas
Copy link
Contributor Author

fghaas commented Jun 11, 2024

@revij This looks good to me now, but I would appreciate an extra pair of eyes from @colder-is-better which is why I have requested a review from him as well.

--default compliance 30d \
<region>/<bucket>
```
> `--default` sets the default object lock settings for new objects and
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just an insignificant micro-nit:

Suggested change
> `--default` sets the default object lock settings for new objects and
> The `--default` parameter sets the default object lock settings for new objects and

Copy link
Contributor

@colder-is-better colder-is-better left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (though I have this one minor micro-nit).

Our object locking guide includes the steps neccesary to set the
default object lock mode for a bucket but does not include the
commands that one can use to set/update the retention mode/period
of a single object in a "lock enabled" bucket.

Co-authored-by: Florian Haas <[email protected]>
@fghaas fghaas force-pushed the object-locking-update branch from 33dafed to 6685902 Compare June 12, 2024 08:16
revij added 2 commits June 12, 2024 10:21
Various syntax fixes for all the references of the mc cli command,
including a short notice explaining how to remove the legal hold
from an object.
radosgw behaviour sets a legal-hold to the latest version of an object
if a version-id is not defined. Updated the aws command example to
include --version-id and added a short notice.
@fghaas fghaas force-pushed the object-locking-update branch from 6685902 to 301367c Compare June 12, 2024 08:21
@fghaas fghaas requested a review from colder-is-better June 12, 2024 08:22
@fghaas fghaas merged commit 301367c into main Jun 12, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants