Skip to content

Commit

Permalink
rgw: RGWPeriodPusher uses zone system key for inter-zonegroup messages
Browse files Browse the repository at this point in the history
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 <cbodley@redhat.com>
(cherry picked from commit f97be8c)
  • Loading branch information
cbodley authored and Vicente-Cheng committed May 3, 2019
1 parent 76c4555 commit 7fae242
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion src/rgw/rgw_period_pusher.cc
Expand Up @@ -250,7 +250,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->svc.zone, zonegroup.get_id(), zonegroup.endpoints, RGWAccessKey()));
std::forward_as_tuple(cct, store->svc.zone, zonegroup.get_id(), zonegroup.endpoints));
}
}

Expand Down

0 comments on commit 7fae242

Please sign in to comment.