rgw/multisite: Add testcases for multisite sync policy#38238
rgw/multisite: Add testcases for multisite sync policy#38238soumyakoduri wants to merge 2 commits intoceph:mainfrom
Conversation
56451cb to
e3fdf4c
Compare
|
@soumyakoduri can you report the issues you're currently seeing? |
c15da58 to
b242960
Compare
b242960 to
8e58169
Compare
8e58169 to
183f123
Compare
src/test/rgw/rgw_multi/tests.py
Outdated
|
|
||
| return | ||
|
|
||
| def test_sync_pipe_multiple_buckets_to_single(): |
There was a problem hiding this comment.
@yehudasa ..I tested this (and the below test case - test_sync_pipe_single_bucket_to_multiple) configuration manually. It doesn't seem to be working. The data is not synced. Could you please take a look and let me know if I missed out anything.
There was a problem hiding this comment.
@soumyakoduri looking at it now, you should be creating different pipes to different source buckets. See here: yehudasa@40a7511
Also, it's better if you set a wildcard 'allowed' policy at the zonegroup level, and create the bucket sync policy on the bucket. So at the zonegroup level just specify '*' as source and dest buckets, and for this test create two separate pipes on the (preferably target) bucket policy itself. Bucket policy changes don't require period update, only ones that are on the zonegroup.
There was a problem hiding this comment.
@soumyakoduri also see here for the second part: yehudasa@d993fa0
There was a problem hiding this comment.
@soumyakoduri looking at it now, you should be creating different pipes to different source buckets. See here: yehudasa@40a7511
Also, it's better if you set a wildcard 'allowed' policy at the zonegroup level, and create the bucket sync policy on the bucket. So at the zonegroup level just specify '*' as source and dest buckets, and for this test create two separate pipes on the (preferably target) bucket policy itself. Bucket policy changes don't require period update, only ones that are on the zonegroup.
Thanks. That worked. So that means instead of creating comma separated bucket list, there should be different pipe created for each <source,dest> bucket combination.. right?
For the second part, I will add them as separate test-cases. Most of the test cases added here are to test functionality at zonegroup level, except for the last one. Will now create corresponding bucket-level cases for each of them.
183f123 to
1ca0f84
Compare
1ca0f84 to
b675f8f
Compare
b675f8f to
21c10f2
Compare
21c10f2 to
f785a17
Compare
|
retest this please |
f785a17 to
fa79e35
Compare
|
This pull request has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs for another 30 days. |
|
unstale |
|
This pull request has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs for another 30 days. |
|
This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved |
Fixes: https://tracker.ceph.com/issues/48416 bucket was passed in without bucket_id, now reading entrypoint info if needed. Signed-off-by: Yehuda Sadeh <yehuda@redhat.com>
Added test cases for the various use-cases of multisite sync policy feature listed in https://docs.ceph.com/en/latest/radosgw/multisite-sync-policy/ Signed-off-by: Soumya Koduri <skoduri@redhat.com>
|
jenkins test make check |
|
This pull request has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs for another 30 days. |
|
This pull request has been automatically closed because there has been no activity for 90 days. Please feel free to reopen this pull request (or open a new one) if the proposed change is still appropriate. Thank you for your contribution! |
|
This pull request has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs for another 30 days. |
2 similar comments
|
This pull request has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs for another 30 days. |
|
This pull request has been automatically marked as stale because it has not had any activity for 60 days. It will be closed if no further activity occurs for another 30 days. |
|
This pull request has been automatically closed because there has been no activity for 90 days. Please feel free to reopen this pull request (or open a new one) if the proposed change is still appropriate. Thank you for your contribution! |
Added few cases to test multisite sync policy configuration & its basic functionality -
https://docs.ceph.com/en/latest/radosgw/multisite-sync-policy/
This PR also includes the fix from Yehuda for the issue#https://tracker.ceph.com/issues/48416, needed for successful run of the tests.
Fixes: https://tracker.ceph.com/issues/48416
Fixes: https://tracker.ceph.com/issues/55812
Signed-off-by: Soumya Koduri skoduri@redhat.com