Project

General

Profile

Actions

Feature #17452

closed

Emit notifications on monmap updates

Added by John Spray over 7 years ago. Updated over 6 years ago.

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;       

Actions #1

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

Also available in: Atom PDF