-
Notifications
You must be signed in to change notification settings - Fork 595
mirror:support version #5149
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
base: master
Are you sure you want to change the base?
mirror:support version #5149
Conversation
It seems to me that this would be a problem if mirroring between 2 versioned S3 sites, preventing non-current versions to be deleted. I can see the idea with this, but I'm not sure if this is the solution. |
Yes, I am considering this point, I am working on the version control capability between two s3 sites, the scope of the modification is relatively large, still in verification. |
…to fix_mirror_remove * 'fix_mirror_remove' of https://github.com/dormanze/mc: Mirror should delete on delete marker creation + ILM (minio#5148) fix: use tags package instead of s3utils (minio#5146) fix empty ilm tier list output (minio#5145)
@klauspost I've finished my pr. Please take time to review it. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed after 21 days if no further activity occurs. Thank you for your contributions. |
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 think the documentation for this change needs some revision to clearly explain the difference between operations with and without the --with-versioning
flag.
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 haven't tested these changes since I don't have a testing environment for both of the use cases this supports, but it looks good and it seems dormanze has already tested this in February.
Community Contribution License
All community contributions in this pull request are licensed to the project maintainers
under the terms of the Apache 2 license.
By creating this pull request I represent that I have the right to license the
contributions to the project maintainers under the Apache 2 license.
Description
1.Supports version during bucket-to-bucket mirroring.
2.Check whether objects of other versions exist before remove.
Motivation and Context
fix #5139
How to test this PR?
case1:
1.Creating Buckets and Enabling Versioning Control
2.Start the mirror process.
3.Upload the object to the bucket twice.
4.Check whether the versions of test.txt at the local and peer ends are consistent.
5.Delete an object based on the version number.
Check whether the corresponding version is deleted from the peer site.
case2:
1.Creating a Bucket and Enabling Versioning Control
2.Start the mirror process.
3.Upload the object to the bucket twice.
4.Delete an object based on the version number.
Check whether local files are deleted.
Types of changes
Checklist:
commit-id
orPR #
here)