]> git.apps.os.sepia.ceph.com Git - ceph.git/commitdiff
rgw: cleanup some unnecessary function called in RGWZoneGroup::add_zone
authorEnming Zhang <enming.zhang@umcloud.com>
Mon, 6 Nov 2017 08:38:18 +0000 (16:38 +0800)
committerEnming Zhang <enming.zhang@umcloud.com>
Mon, 6 Nov 2017 08:38:18 +0000 (16:38 +0800)
Signed-off-by: Enming Zhang <enming.zhang@umcloud.com>
src/rgw/rgw_rados.cc

index 452dce2ad10dd0d2bf07ca3a1455467bfdb4fa68..13d3fc91217d7e5e34e16574d19dc09bfbe0b1ea 100644 (file)
@@ -374,18 +374,18 @@ int RGWZoneGroup::add_zone(const RGWZoneParams& zone_params, bool *is_master, bo
 
   if (is_master) {
     if (*is_master) {
-      if (!master_zone.empty() && master_zone != zone_params.get_id()) {
+      if (!master_zone.empty() && master_zone != zone_id) {
         ldout(cct, 0) << "NOTICE: overriding master zone: " << master_zone << dendl;
       }
-      master_zone = zone_params.get_id();
-    } else if (master_zone == zone_params.get_id()) {
+      master_zone = zone_id;
+    } else if (master_zone == zone_id) {
       master_zone.clear();
     }
   }
 
-  RGWZone& zone = zones[zone_params.get_id()];
-  zone.name = zone_params.get_name();
-  zone.id = zone_params.get_id();
+  RGWZone& zone = zones[zone_id];
+  zone.name = zone_name;
+  zone.id = zone_id;
   if (!endpoints.empty()) {
     zone.endpoints = endpoints;
   }