radosgw-admin: 'zone set' won't overwrite existing default-placement

Fixes: https://tracker.ceph.com/issues/63829
Signed-off-by: Casey Bodley <cbodley@redhat.com>
(cherry picked from commit d785add11c38411a0b2f5933cc9581c00cc51ac0)

diff --git a/src/rgw/driver/rados/rgw_zone.cc b/src/rgw/driver/rados/rgw_zone.cc
index ed09f24f6c24b1c9e25d070edd0a5fa23a1ee732..5c3f55b3dd88303c32c93b446e060adaf15ce3d6 100644 (file)
--- a/src/rgw/driver/rados/rgw_zone.cc
+++ b/src/rgw/driver/rados/rgw_zone.cc
@@ -1005,10 +1005,12 @@ int create_zone(const DoutPrefixProvider* dpp, optional_yield y,
   }
 
   // add default placement with empty pool name
+  RGWZonePlacementInfo placement;
   rgw_pool pool;
-  auto& placement = info.placement_pools["default-placement"];
   placement.storage_classes.set_storage_class(
       RGW_STORAGE_CLASS_STANDARD, &pool, nullptr);
+  // don't overwrite if it already exists
+  info.placement_pools.emplace("default-placement", std::move(placement));
 
   // build a set of all pool names used by other zones
   std::set<rgw_pool> pools;