]> git.apps.os.sepia.ceph.com Git - ceph.git/commit
projects / ceph.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: fa85dd0) | patch
osd/OSDMap: ignore xinfo if features == 0
authorSage Weil <sage@redhat.com>
Tue, 19 Sep 2017 22:25:56 +0000 (18:25 -0400)
committerBrad Hubbard <bhubbard@redhat.com>
Wed, 20 Sep 2017 03:11:10 +0000 (13:11 +1000)
commit8399833f3d221364a8ededb97cd4e1c5a258d924
treec6baaf2acd5298035395c5fdb15fe5e998442216tree | snapshot
parentfa85dd056580e32cc614a50712dc595f0b127618commit | diff
osd/OSDMap: ignore xinfo if features == 0

Some old bug (e.g., http://tracker.ceph.com/issues/20751) could
result in an UP+EXISTS osd having features==0.  If that happens,
we shouldn't crash the mon, which (reasonably) does

   if (osdmap.get_epoch()) {
     if (osdmap.get_num_up_osds() > 0) {
       assert(osdmap.get_up_osd_features() & CEPH_FEATURE_MON_STATEFUL_SUB);
       check_subs();
     }
   }

Signed-off-by: Sage Weil <sage@redhat.com>
src/osd/OSDMap.cc diff | blob | history
Unnamed repository; edit this file 'description' to name the repository.