I'm currently using version 4.4.2 and am having trouble trying to reset the statistics in the TMX module. According to its documentation, all statistics except “inuse_transactions” can be reset.
Checking the tmx_mod.c files for the statistics definition, I saw that the STAT_NO_RESET flag wasn't used for any statistic listed there (which seems odd because I think it should have been set for the inuse_transactions statistic):
`static stat_export_t mod_stats[] = {
{"UAS_transactions", STAT_IS_FUNC, (stat_var**)tmx_stats_uas_trans },
{"UAC_transactions", STAT_IS_FUNC, (stat_var**)tmx_stats_uac_trans },
{"2xx_transactions", STAT_IS_FUNC, (stat_var**)tmx_stats_trans_2xx },
{"3xx_transactions", STAT_IS_FUNC, (stat_var**)tmx_stats_trans_3xx },
{"4xx_transactions", STAT_IS_FUNC, (stat_var**)tmx_stats_trans_4xx },
{"5xx_transactions", STAT_IS_FUNC, (stat_var**)tmx_stats_trans_5xx },
{"6xx_transactions", STAT_IS_FUNC, (stat_var**)tmx_stats_trans_6xx },
{"inuse_transactions", STAT_IS_FUNC, (stat_var**)tmx_stats_trans_inuse },
{"active_transactions", STAT_IS_FUNC, (stat_var**)tmx_stats_trans_active},
{"rpl_received", STAT_IS_FUNC, (stat_var**)tmx_stats_rcv_rpls },
{"rpl_absorbed", STAT_IS_FUNC, (stat_var**)tmx_stats_abs_rpls },
{"rpl_generated", STAT_IS_FUNC, (stat_var**)tmx_stats_rld_loc_rpls},
{"rpl_relayed", STAT_IS_FUNC, (stat_var**)tmx_stats_rld_rcv_rpls},
{"rpl_sent", STAT_IS_FUNC, (stat_var**)tmx_stats_rld_tot_rpls},
{0, 0, 0}
};`
All seems ok, but when I try to reset the stats, the command simply doesn't work. The values remain unaltered. The clear command doesn't work either, it just returns the current values.
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/820
I'm currently using version 4.4.2 and I am came across an issue while using the ratelimit module with the FEEDBACK algorithm.
Checking the ratelimit.c to see how the cpu load value is calculated I saw that the module parses only the line corresponding to the overall CPU usage in the "/proc/stat" file.
First, the module calculates the amount of CPU that is idle, in percentage.
But, when the module is going to calculate the percentage of the overall CPU usage, it first divides the idle value obtained previously by the number of CPU cores, and then subtracts this value from 1.
The mistake here seems to be the division by the number of CPU cores, because the overall porcentage retrieved from the /proc/stat file is already in the range from 0-100, regardless of the number of CPU cores
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/issues/871
Hello all,
The way sipcapture now works, for requests it inserts the actual method
name into the "method" field. But for replies, it inserts the SIP Status
(i.e. 200, 407) into the "method" field.
It would be better to extract the actual method name from the CSEQ header
and insert it into the "method" field, as well as adding a "status" field
to the tables and inserting there the actual SIP Status.
I've made a patch for sipcapture.c and sipcapture.h which do just that.
I will also be submitting a PR to homer for this, as db tables need to be
modified to accept the status field.
How do i do the pull request?
I don't seem to be allowed to do commits to a branch i created locally
Regards,
Regards,
David Villasmil
email: david.villasmil.work(a)gmail.com
<https://mail.google.com/mail/?view=cm&fs=1&tf=1&to=david.villasmil.work@gma…>
phone: +34669448337
Two fixes in this PR.
The first is to perform the ntohs also on hep messaging carrying payload other than SIP.
The scond is to take in count the definition of captid chunk in HEPv3 (uint32 BE) different from the one for HEPv2 (uint16 LE).
You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/874
-- Commit Summary --
* sipcapture: perform ntohs on src and dst port when payload is not SIP
* sipcapture: fix captid extraction from hep header for v3
-- File Changes --
M modules/sipcapture/hep.c (18)
M modules/sipcapture/hep.h (6)
M modules/sipcapture/sipcapture.c (10)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/874.patchhttps://github.com/kamailio/kamailio/pull/874.diff
--
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/kamailio/kamailio/pull/874