From: Sage Weil <sage@redhat.com>
Date: Fri, 29 Apr 2016 19:29:19 +0000 (-0400)
Subject: osd: remove useless set_role from split completion
X-Git-Tag: v11.0.0~662
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=efedfffe048999e60ccdbd29c2c5488675210b08;p=ceph.git

osd: remove useless set_role from split completion

This is done by PG::split_into().

Signed-off-by: Sage Weil <sage@redhat.com>
---

diff --git a/src/osd/OSD.cc b/src/osd/OSD.cc
index 77acdc9e74ee..de3de5a58589 100644
--- a/src/osd/OSD.cc
+++ b/src/osd/OSD.cc
@@ -2938,10 +2938,6 @@ void OSD::add_newly_split_pg(PG *pg, PG::RecoveryCtx *rctx)
   service.pg_add_epoch(pg->info.pgid, pg->get_osdmap()->get_epoch());
 
   dout(10) << "Adding newly split pg " << *pg << dendl;
-  vector<int> up, acting;
-  pg->get_osdmap()->pg_to_up_acting_osds(pg->info.pgid.pgid, up, acting);
-  int role = OSDMap::calc_pg_role(service.whoami, acting);
-  pg->set_role(role);
   pg->reg_next_scrub();
   pg->handle_loaded(rctx);
   pg->write_if_dirty(*(rctx->transaction));