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

octopus: rgwlc: remove lc entry on bucket delete #44730

Merged
merged 1 commit into from
Jan 26, 2022

Conversation

cfsnyder
Copy link
Contributor

backport tracker: https://tracker.ceph.com/issues/53850


backport of #36308
parent tracker: https://tracker.ceph.com/issues/46728

this backport was staged using ceph-backport.sh version 16.0.0.6848
find the latest version at https://github.com/ceph/ceph/blob/master/src/script/ceph-backport.sh

Buckets with lifecycle policies installed have a state entry that
must also be deleted when the bucket is removed.

Fixes: https://tracker.ceph.com/issues/46728

N.b., should really be generic, not specific to the RADOS store, but
there doesn't seem to be a clean model for implementing generic side
effects in Zipper, currently.

Signed-off-by: Matt Benjamin <mbenjamin@redhat.com>
(cherry picked from commit cc1e812)

Conflicts:
	src/rgw/rgw_sal_rados.cc

Cherry-pick notes:
- Code from rgw_sal_rados.cc existed in rgw_sal.cc in Octopus
@ljflores
Copy link
Contributor

http://pulpito.front.sepia.ceph.com/yuriw-2022-01-24_18:01:47-rados-wip-yuri10-testing-2022-01-24-0810-octopus-distro-default-smithi/

Failures, unrelated:
https://tracker.ceph.com/issues/50280
https://tracker.ceph.com/issues/45318

Details:
Bug_#50280: cephadm: RuntimeError: uid/gid not found - Ceph
Bug_#45318: octopus: Health check failed: 2/6 mons down, quorum b,a,c,e (MON_DOWN)" in cluster log running tasks/mon_clock_no_skews.yaml

@ljflores
Copy link
Contributor

jenkins test api

@cbodley
Copy link
Contributor

cbodley commented Jan 26, 2022

ty @cfsnyder!

@yuriw
Copy link
Contributor

yuriw commented Jan 26, 2022

jenkins test api

@yuriw yuriw merged commit a28cf56 into ceph:octopus Jan 26, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants