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
RGW: Zipper - zonegroup, placement tier #45623
Conversation
This pull request can no longer be automatically merged: a rebase is needed and changes have to be manually resolved |
8ae0438
to
b0586de
Compare
jenkins test api |
b0586de
to
6d446ae
Compare
This fixes dbstore, which I inadvertently broke. |
Add ZoneGroup and PlacementTier to the Zipper API. This cleaned up a lot of issues in the Module branch. In addition, add cloud transition. Signed-off-by: Daniel Gryniewicz <dang@redhat.com>
6d446ae
to
92252d2
Compare
@cbodley Is this good to go now? |
std::unique_ptr<RGWRealmReloader> reloader; | ||
std::unique_ptr<RGWPeriodPusher> pusher; | ||
std::unique_ptr<RGWFrontendPauser> pauser; | ||
std::unique_ptr<RGWRealmWatcher> realm_watcher; | ||
if (store->get_name() == "rados") { | ||
// add a watcher to respond to realm configuration changes | ||
pusher = std::make_unique<RGWPeriodPusher>(&dp, store, null_yield); | ||
pauser = std::make_unique<RGWFrontendPauser>(fes, implicit_tenant_context, pusher.get()); | ||
reloader = std::make_unique<RGWRealmReloader>(store, service_map_meta, pauser.get()); |
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 like std::optional for conditional or deferred initialization, but this looks correct now 👍
(here during startup, it doesn't make much difference whether or not we allocate for unique_ptr)
Add ZoneGroup and PlacementTier to the Zipper API. This cleaned up a
lot of issues in the Module branch.
In addition, add cloud transition.
Signed-off-by: Daniel Gryniewicz dang@redhat.com
Checklist
Show available Jenkins commands
jenkins retest this please
jenkins test classic perf
jenkins test crimson perf
jenkins test signed
jenkins test make check
jenkins test make check arm64
jenkins test submodules
jenkins test dashboard
jenkins test dashboard cephadm
jenkins test api
jenkins test docs
jenkins render docs
jenkins test ceph-volume all
jenkins test ceph-volume tox
jenkins test windows