Re: [PATCH] http admin commands

yes, I see your concern :)

But I think that if we want reload it should work for all bearerbox  
parts and not only smsc, something like config diff and then reload
changed parts...

Alex, don't get me wrong, I'm not against your patch, I'm just don't  
like partial support :)
Please provide userguide patch and it could be commited...

Thanks,
Alex

Am 07.07.2009 um 15:36 schrieb Alejandro Guerrieri:

> Example #1: You have a huge queue (hundreds of thousands) of  
> outgoing messages going out. Restarting the whole service can take  
> tens of minutes, specially when you're repeatedly tweaking those  
> services to try to send as fast as possible. This happened to me  
> before: the carrier was dropping the link when we were sending too  
> many messages at once, so we had to trial and error with max-pending- 
> submits and throughput, and each restart took about 15 minutes, so  
> you get the idea...
>
> Example #2: You have 20+ smsc binds. Turning all those down and up  
> again repeatedly can annoy a carrier or two, and also hurt your  
> service level agreements if any.
>
> I can foresee another scenarios where this would be valuable.
>
> Regards,
> --
> Alejandro Guerrieri
> [email protected]
>
>
>
> On 07/07/2009, at 15:13, Alexander Malysh wrote:
>
>> Hi Alex,
>>
>> patch looks ok but I don't really see why you should be unable to  
>> restart the whole bearerbox...
>> and I think reload should work for all bearerbox parts...
>>
>> therefore I'm +0 for this patch.
>>
>> Thanks,
>> Alex
>>
>> Am 07.07.2009 um 08:14 schrieb Alejandro Guerrieri:
>>
>>> This patch improves the http-based admin commands and fixes some  
>>> quirks:
>>>
>>> 1. It adds 2 new commands: add-smsc and remove-smsc that allows  
>>> for dynamic adding/removing of smsc's without restarting kannel  
>>> nor any other smsc bind.
>>>
>>> 2. start-smsc now reloads the smsc list from the config file, so  
>>> you can stop-smsc and then start-smsc to reload the configuration.  
>>> The only caveat is: do not change the id or admin-id (in that case  
>>> you'll have to remove-smsc with the old id/admin-id and add-smsc  
>>> with the new one instead).
>>>
>>> 3. Fixes a bug on stop-smsc and start-smsc. Before, you could  
>>> start/stop non-existant smsc's, and despite the request would be  
>>> ignored, the interface would say that the smsc was stopped/ 
>>> started. Now it will return an error and log the reason.
>>>
>>> 4. Fixes a bug on the id/admin-id functionality when stopping/ 
>>> starting a group with 2 or more entries with the same id and  
>>> different admin-id's. Before, the list would get corrupted when  
>>> trying to start again, now it works as expected.
>>>
>>> Userguide patch coming soon :)
>>>
>>> Please review in the meantime.
>>>
>>> Regards,
>>> --
>>> Alejandro Guerrieri
>>> [email protected]
>>>
>>>
>>> <kannel-http-admin.patch.zip>
>>
>