rgw: multisite: enable slave lc if lc is enabled from master#34553
rgw: multisite: enable slave lc if lc is enabled from master#34553IlsooByun wants to merge 1 commit intoceph:masterfrom
Conversation
Signed-off-by: Ilsoo Byun <ilsoobyun@linecorp.com>
|
Hi @IlsooByun I think this is a valid improvement, but I think we need to handle the special case where we are syncing to an archive zone. In that case, the current consensus is that ordinary lifecycle policy must not take effect on the archive zone--otherwise, data would tend to be eliminated there precisely when it was expired on the active sites, preventing recovery from the archive. We think it -does- make sense to enable explicit policy for data reduction/expiration in archive zone, but we thought a) such policy should need to be applied explicitly in the archive zone; and b) possibly needs to be introduced as an extension to the lifecycle grammar so that ordinary policies can't be applied by mistake. |
|
@mattbenjamin Thank you for a detailed explanation. |
|
Hi @IlsooByun Sorry, let me clarify: there is no other design, I agree with your analysis. I'm suggesting that I think this change needs to protect archive zone from modification, in addition to what it currently does. |
|
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! |
Fixes: https://tracker.ceph.com/issues/44268
Related PR: #33524
Signed-off-by: Ilsoo Byun ilsoobyun@linecorp.com
Checklist
Show available Jenkins commands
jenkins retest this pleasejenkins test crimson perfjenkins test signedjenkins test make checkjenkins test make check arm64jenkins test submodulesjenkins test dashboardjenkins test dashboard backendjenkins test docsjenkins render docsjenkins test ceph-volume alljenkins test ceph-volume tox