rgw: update new name after converting region to zonegroup

Signed-off-by: Orit Wasserman <owasserm@redhat.com>

diff --git a/src/rgw/rgw_rados.cc b/src/rgw/rgw_rados.cc
index 6fe8c8a0014269648ca12d9022dd880dccb102da..4bc57e3f4f4a216bf77d419cb2e7dc26467af605 100644 (file)
--- a/src/rgw/rgw_rados.cc
+++ b/src/rgw/rgw_rados.cc
@@ -1176,7 +1176,9 @@ const string& RGWZoneParams::get_predefined_name(CephContext *cct) {
 
 int RGWZoneParams::init(CephContext *cct, RGWRados *store, bool setup_obj, bool old_format)
 {
-  name = cct->_conf->rgw_zone;
+  if (name.empty()) {
+    name = cct->_conf->rgw_zone;
+  }
 
   int ret = RGWSystemMetaObj::init(cct, store, setup_obj, old_format);
   if (ret < 0) {
@@ -2865,19 +2867,25 @@ int RGWRados::replace_region_with_zonegroup()
     derr << "create zonegroup " << *iter << dendl;
     /* read region info default has no data */
     if (*iter != default_zonegroup_name){
-      RGWZoneGroup zonegroup(*iter);   
+      RGWZoneGroup zonegroup(*iter);
       int ret = zonegroup.init(cct, this, true, true);
       if (ret < 0) {
        lderr(cct) << "failed init zonegroup: ret "<< ret << " " << cpp_strerror(-ret) << dendl;
        return ret;
       }
-      derr << "create zonegroup: store_info " << *iter << dendl;    
+      derr << "create zonegroup: name " << *iter << dendl;
       ret = zonegroup.update();
       if (ret < 0 && ret != -EEXIST) {
        lderr(cct) << "failed to store zonegroup " << *iter << ": ret "<< ret << " " << cpp_strerror(-ret)
                   << dendl;
        return ret;
       }
+      ret = zonegroup.update_name();
+      if (ret < 0 && ret != -EEXIST) {
+       lderr(cct) << "failed to store zonegroup " << *iter << ": ret "<< ret << " " << cpp_strerror(-ret)
+                  << dendl;
+       return ret;
+      }
       if (zonegroup.get_name() == default_region) {
        ret = zonegroup.set_as_default();
        if (ret < 0) {
@@ -2893,8 +2901,6 @@ int RGWRados::replace_region_with_zonegroup()
                   << dendl;
        return ret;
       }
-      lderr(cct) << "delete region " << *iter << ": ret "<< ret << " " << cpp_strerror(-ret)
-                  << dendl;
     }
   }
 
@@ -2977,6 +2983,8 @@ int RGWRados::init_complete()
       lderr(cct) << "failed reading zonegroup info: ret "<< ret << " " << cpp_strerror(-ret) << dendl;
       return ret;
     } else if (ret == -ENOENT) {
+      derr << "zonegroup create_default " << dendl;
+
       ret = zonegroup.create_default();
       if (ret < 0) {
        lderr(cct) << "failure in zonegroup create_default: ret "<< ret << " " << cpp_strerror(-ret)

diff --git a/src/rgw/rgw_rados.h b/src/rgw/rgw_rados.h
index 30ce15f9df4ba59abe831ac77629199989c7d190..befdb2aabe5249a5fdb759efb0f57c113493acce 100644 (file)
--- a/src/rgw/rgw_rados.h
+++ b/src/rgw/rgw_rados.h
@@ -791,6 +791,7 @@ public:
   int delete_obj(bool old_format = false);
   int rename(const string& new_name);
   int update() { return store_info(false);}
+  int update_name() { return store_name(false);}
   int read();
   int write(bool exclusive);