Commit b581a585 authored Mar 01, 2017 by Ilya Dryomov

libceph: don't set weight to IN when OSD is destroyed

Since ceph.git commit 4e28f9e63644 ("osd/OSDMap: clear osd_info,
osd_xinfo on osd deletion"), weight is set to IN when OSD is deleted.
This changes the result of applying an incremental for clients, not
just OSDs.  Because CRUSH computations are obviously affected,
pre-4e28f9e63644 servers disagree with post-4e28f9e63644 clients on
object placement, resulting in misdirected requests.

Mirrors ceph.git commit a6009d1039a55e2c77f431662b3d6cc5a8e8e63f.

Fixes: 930c5328 ("libceph: apply new_state before new_up_client on incrementals")
Link: http://tracker.ceph.com/issues/19122


Signed-off-by: Ilya Dryomov <idryomov@gmail.com>
Reviewed-by: Sage Weil <sage@redhat.com>

parent 9afd30db

net/ceph/osdmap.c

+0 −1

Original line number	Diff line number	Diff line
		@@ -1379,7 +1379,6 @@ static int decode_new_up_state_weight(void *p, void end,
		if ((map->osd_state[osd] & CEPH_OSD_EXISTS) &&
		(xorstate & CEPH_OSD_EXISTS)) {
		pr_info("osd%d does not exist\n", osd);
		map->osd_weight[osd] = CEPH_OSD_IN;
		ret = set_primary_affinity(map, osd,
		CEPH_OSD_DEFAULT_PRIMARY_AFFINITY);
		if (ret)

Admin message