From: Casey Bodley Date: Sat, 13 Apr 2019 15:34:21 +0000 (-0400) Subject: rgw: RGWPeriodPusher uses zone system key for inter-zonegroup messages X-Git-Tag: v13.2.7~200^2 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F29256%2Fhead;p=ceph.git rgw: RGWPeriodPusher uses zone system key for inter-zonegroup messages RGWPeriodPusher was using an empty RGWAccessKey for inter-zonegroup messages, which were rejected as an anonymous user with 403 Forbidden. this prevented multi-zonegroup configurations from converging on the same period configuration Fixes: http://tracker.ceph.com/issues/39287 Signed-off-by: Casey Bodley (cherry picked from commit f97be8cb6e025442607042d9defc01542049d472) Conflicts: src/rgw/rgw_period_pusher.cc - mimic has store where master has store->svc.zone (in the argument list) --- diff --git a/src/rgw/rgw_period_pusher.cc b/src/rgw/rgw_period_pusher.cc index ec676c23943..c5f2fbefef5 100644 --- a/src/rgw/rgw_period_pusher.cc +++ b/src/rgw/rgw_period_pusher.cc @@ -245,7 +245,7 @@ void RGWPeriodPusher::handle_notify(RGWZonesNeedPeriod&& period) hint = conns.emplace_hint( hint, std::piecewise_construct, std::forward_as_tuple(zonegroup.get_id()), - std::forward_as_tuple(cct, store, zonegroup.get_id(), zonegroup.endpoints, RGWAccessKey())); + std::forward_as_tuple(cct, store, zonegroup.get_id(), zonegroup.endpoints)); } }