From 487ec3f9ea4a916a2e57a087fad6f26f6fd2b36b Mon Sep 17 00:00:00 2001 From: Joao Eduardo Luis Date: Wed, 29 Nov 2023 14:05:25 +0000 Subject: [PATCH] rgw/sfs: don't care about location constraints We don't deal with zones, all zones are welcome zones. So, when someone wants to store something on a given location, welcome them by allowing all zones. Fixes: aquarist-labs/s3gw#848 Signed-off-by: Joao Eduardo Luis --- src/rgw/driver/sfs/zone.cc | 2 +- src/rgw/driver/sfs/zone.h | 3 ++- 2 files changed, 3 insertions(+), 2 deletions(-) diff --git a/src/rgw/driver/sfs/zone.cc b/src/rgw/driver/sfs/zone.cc index befab23d8d5bf..c3e637c9ebba3 100644 --- a/src/rgw/driver/sfs/zone.cc +++ b/src/rgw/driver/sfs/zone.cc @@ -42,7 +42,7 @@ bool SFSZone::get_redirect_endpoint(std::string* /*endpoint*/) { } bool SFSZone::has_zonegroup_api(const std::string& /*api*/) const { - return false; + return true; } const std::string& SFSZone::get_current_period_id() { diff --git a/src/rgw/driver/sfs/zone.h b/src/rgw/driver/sfs/zone.h index 9f45b24c5bb0f..601a067c8c34b 100644 --- a/src/rgw/driver/sfs/zone.h +++ b/src/rgw/driver/sfs/zone.h @@ -43,7 +43,8 @@ class SFSZoneGroup : public StoreZoneGroup { return !!group->placement_targets.count(target); } virtual bool is_master_zonegroup() const override { - return group->is_master_zonegroup(); + // we are a master zonegroup, because we are just the one running. + return true; } virtual const std::string& get_api_name() const override { return group->api_name;