Summary:	ASTERISK-15616: [patch] Reload command does not update the SLA configuration properly
Reporter:	Marcelo (mbonin)	Labels:
Date Opened:	2010-02-12 14:08:00.000-0600	Date Closed:	2010-06-23 12:28:35
Priority:	Major	Regression?	No
Status:	Closed/Complete	Components:	Core/Configuration
Versions:		Frequency of Occurrence
Related Issues:
Environment:		Attachments:	( 0) sla_reload.patch
Description:	When you configure the SLA feature at fist time and perform a reload in the console, doing #sla show trunks or #sla show stations, it does not reflect the changes, and the feature does not work. But if you have some previous SLA configuration and modify it, when reload the configuration the console shows both configurations, the old and the new, and the new configuration does not work.
Comments:	By: Marcelo (mbonin) 2010-06-18 10:50:22 Any news about this issue?? By: Paul Belanger (pabelanger) 2010-06-18 11:57:11 Per the Asterisk maintenance timeline page at http://www.asterisk.org/asterisk-versions maintenance (bug) support for the 1.6.0 and 1.6.1 branches has ended. For continued maintenance support please move to the 1.6.2 branch. More information on this change can be found in the release announcement: http://www.asterisk.org/node/49924 By: Marcelo (mbonin) 2010-06-18 12:08:46 I understand, but I submited the bug on 02/12/2010 and this version was moved to mantainance on 03/31/2010, so, I opened the issue before the release 1.6.0.x gone to mantainance. How should I proceed now? By: Paul Belanger (pabelanger) 2010-06-18 12:19:41 Retest with 1.6.2, if the problem still exists we will fix it.  You would then be responsible for backporting the fix. By: Terry Wilson (twilson) 2010-06-21 16:20:11 I can verify this is still an issue in trunk. Patch coming for trunk which applies cleanly to 1.6.2 (with fuzz) By: Terry Wilson (twilson) 2010-06-21 16:21:10 mbonin: could you test the attached patch and verify that it solves the issue for you? By: Digium Subversion (svnbot) 2010-06-23 12:21:40 Repository: asterisk Revision: 272109 U   trunk/apps/app_meetme.c ------------------------------------------------------------------------ r272109 | twilson | 2010-06-23 12:21:39 -0500 (Wed, 23 Jun 2010) | 12 lines Make sure reload updates SLA config Even if there are no stations or trunks defined, we need to start the sla thread to make sure we get the reload event. Also, when doing a reload we need to remove the existing trunks and stations or they end up hanging around. (closes issue ASTERISK-15616) Reported by: mbonin Patches:      sla_reload.patch uploaded by twilson (license 396) Tested by: twilson ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=272109 By: Digium Subversion (svnbot) 2010-06-23 12:28:34 Repository: asterisk Revision: 272124 _U  branches/1.6.2/ U   branches/1.6.2/apps/app_meetme.c ------------------------------------------------------------------------ r272124 | twilson | 2010-06-23 12:28:34 -0500 (Wed, 23 Jun 2010) | 19 lines Merged revisions 272109 via svnmerge from https://origsvn.digium.com/svn/asterisk/trunk ........  r272109 | twilson | 2010-06-23 12:21:40 -0500 (Wed, 23 Jun 2010) | 12 lines    Make sure reload updates SLA config    Even if there are no stations or trunks defined, we need to start the sla  thread to make sure we get the reload event. Also, when doing a reload we need  to remove the existing trunks and stations or they end up hanging around.    (closes issue ASTERISK-15616)  Reported by: mbonin  Patches:        sla_reload.patch uploaded by twilson (license 396)  Tested by: twilson ........ ------------------------------------------------------------------------ http://svn.digium.com/view/asterisk?view=rev&revision=272124