[asterisk-bugs] [Zaptel 0012405]: [patch] bad master selection (2 different ways)

[noreply at bugs.digium.com]

A NOTE has been added to this issue. 
====================================================================== 
http://bugs.digium.com/view.php?id=12405 
====================================================================== 
Reported By:                biohumanoid
Assigned To:                
====================================================================== 
Project:                    Zaptel
Issue ID:                   12405
Category:                   zaptel (the module)
Reproducibility:            always
Severity:                   major
Priority:                   normal
Status:                     new
Zaptel Version:             1.4.9.2 
SVN Branch (only for SVN checkouts, not tarball releases): N/A 
SVN Revision (number only!):  
Disclaimer on File?:        N/A 
Request Review:              
====================================================================== 
Date Submitted:             04-10-2008 05:44 CDT
Last Modified:              04-11-2008 12:29 CDT
====================================================================== 
Summary:                    [patch] bad master selection (2 different ways)
Description: 
Current zaptel master handling work in the following way:
The first SPAN become a master
If a new registered !!! SPAN is preferred , it is the new master
If a span is unregistered - the new master is the first available span
If master SPAN is in alarm state - the new master is the first non in
alarm state SPAN

So
1. ztdummy once loaded will be the master forever.
2. even not-running SPAN can be a master
3. master implementation is ugly...

====================================================================== 

---------------------------------------------------------------------- 
 tzafrir - 04-11-08 12:29  
---------------------------------------------------------------------- 
First-off, thanks for your contribution. The selection of the master is an
area that needs improvements in Zaptel.

Does the ztdummy patch means that ztdummy would do nothing if there's any
other span?


Also, from a very short reading of your patch: what is "pref_master" (vs.
the existing "prefmaster")? 

Issue History 
Date Modified   Username       Field                    Change               
====================================================================== 
04-11-08 12:29  tzafrir        Note Added: 0085361                          
======================================================================