Actions
Feature #17452
closedEmit notifications on monmap updates
Status:
Resolved
Priority:
Normal
Assignee:
-
Category:
ceph-mgr
Target version:
-
% Done:
0%
Source:
other
Tags:
Backport:
Reviewed:
Affected Versions:
Pull request ID:
Description
From Mgr::ms_dispatch:
424 case CEPH_MSG_MON_MAP: 425 // FIXME: we probably never get called here because MonClient 426 // has consumed the message. For consuming OSDMap we need 427 // to be the tail dispatcher, but to see MonMap we would 428 // need to be at the head. 429 // Result is that ClusterState has access to monmap (it reads 430 // from monclient anyway), but we don't see notifications. Hook 431 // into MonClient to get notifications instead of messing 432 // with message delivery to achieve it? 433 assert(0); 434 435 py_modules.notify_all("mon_map", ""); 436 break;
Updated by John Spray over 6 years ago
- Status changed from New to Resolved
This was fixed in:
commit 8d803e0483d797fc87c0e58de28009ba296b7dfb Author: Greg Farnum <gfarnum@redhat.com> Date: Tue Mar 7 15:30:55 2017 -0800 MonClient: support "passthrough" of MonMap messages so other Dispatchers can see them Use this in the Manager so it can notify of new monitor maps. Signed-off-by: Greg Farnum <gfarnum@redhat.com>
Actions