]> git.apps.os.sepia.ceph.com Git - ceph-ci.git/commitdiff
crimson/osd: pass octopus to PeeringCtx's constructor
authorKefu Chai <kchai@redhat.com>
Fri, 13 Sep 2019 05:14:02 +0000 (13:14 +0800)
committerKefu Chai <kchai@redhat.com>
Fri, 13 Sep 2019 08:14:13 +0000 (16:14 +0800)
to accommodate the change of 77ecc327

classic OSD started to use single-pg peering ops since ce05c172, and it
switched over to these ops since octopus. and it's assumed that
crimson-osd won't be GA until octopus+2 release, so it's not needed to
support pre-octopus releases. hence "octopus" is passed to PeeringCtx's
constructor.

Signed-off-by: Kefu Chai <kchai@redhat.com>
src/crimson/osd/osd.cc
src/crimson/osd/osd_operations/peering_event.h

index 6b2468d440cd63382f89c54d22bbd2739478ffa4..b0586d2f02d41b144f01b3ddb7a5282d3ea90478 100644 (file)
@@ -691,7 +691,7 @@ seastar::future<Ref<PG>> OSD::handle_pg_create_info(
          if (!pg)
            return seastar::make_ready_future<Ref<PG>>(Ref<PG>());
         return store->create_new_collection(coll_t(info->pgid)).then([this, &info, startmap, pg] (auto coll) {
-         PeeringCtx rctx;
+         PeeringCtx rctx{ceph_release_t::octopus};
          const pg_pool_t* pp = startmap->get_pg_pool(info->pgid.pool());
 
          int up_primary, acting_primary;
@@ -1005,7 +1005,8 @@ seastar::future<> OSD::consume_map(epoch_t epoch)
   auto &pgs = pg_map.get_pgs();
   return seastar::parallel_for_each(pgs.begin(), pgs.end(), [=](auto& pg) {
     return shard_services.start_operation<PGAdvanceMap>(
-      *this, pg.second, pg.second->get_osdmap_epoch(), epoch).second;
+      *this, pg.second, pg.second->get_osdmap_epoch(), epoch,
+      PeeringCtx{ceph_release_t::octopus}, false).second;
   }).then([epoch, this] {
     osdmap_gate.got_map(epoch);
     return seastar::make_ready_future();
index 62ecccfbe729fd108ade35c9b25c21731af8f469..7f32953a9e928eb350f46349376f6b2ab033e6ce 100644 (file)
@@ -64,6 +64,7 @@ public:
     ShardServices &shard_services, const pg_shard_t &from, const spg_t &pgid,
     Args&&... args) :
     shard_services(shard_services),
+    ctx{ceph_release_t::octopus},
     from(from),
     pgid(pgid),
     evt(std::forward<Args>(args)...)