[SR-Users] tsilo logging

[Alex Balashov]

If you’re asking if ts_append() has a documented return value, it does not.

A function to count transactions stored for a given <domain, RURI> seems like a logical need. Then, one could do something like:

   if(ts_trans_count(“location”, “$tu”) > 0) 
      ts_append(“location”, “$tu”);

— Alex

> On Oct 13, 2022, at 7:56 AM, Olle E. Johansson <oej at edvina.net> wrote:
> 
> 
> 
>> On 13 Oct 2022, at 13:50, Alex Balashov <abalashov at evaristesys.com> wrote:
>> 
>> Hi,
>> 
>> When there are no stored transactions for an AOR, it logs this error:
>> 
>> Oct 13 07:48:18 gw1 /usr/sbin/kamailio[14758]: ERROR: tsilo [ts_append.c:64]: ts_append(): failed to retrieve record for sip:abalashov at sip.evaristesys.com
>> 
>> But not having transactions ts_store()’d is a rather normal occurrence, right? It is the expected case the vast majority of the time that someone registers. 
>> 
>> Does that mean one is simply destined to have this scary-looking error, to the syslog ERROR facility, for a normal runtime condition? Is there not some principle against using attention-grabbing error messages this way? Would a patch to fix be welcome?
> 
> I think that’s a bad sysadm experience… Could be a WARNING or a DEBUG. 
> 
> Do you get an error code indicating that your silo is empty?
> 
> /O
> __________________________________________________________
> Kamailio - Users Mailing List - Non Commercial Discussions
> sr-users at lists.kamailio.org
> Important: keep the mailing list in the recipients, do not reply only to the sender!
> Edit mailing list options or unsubscribe:
> https://lists.kamailio.org/cgi-bin/mailman/listinfo/sr-users

-- 
Alex Balashov | Principal | Evariste Systems LLC

Tel: +1-706-510-6800 / +1-800-250-5920 (toll-free)
Web: http://www.evaristesys.com/, http://www.csrpswitch.com/