sr-dev March 2014

sr-dev@lists.kamailio.org

29 participants
298 discussions

[tracker] Comment added: Kamailio doesn't to handle any SIP message at all after several dialogs... (CPU high usage)
by sip-router 04 Mar '14

04 Mar '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#397 - Kamailio doesn't to handle any SIP message at all after several dialogs... (CPU high usage) User who did this - Jack Wang (GoGoJack) ---------- OMG...I DO NOW understand why my config.mak may have some problems - For taking a customized settings, the right way should be done such like "make cfg ARCH=arm" before (and the -DUSE_PTHREAD_MUTEX was set right there), but I modify it after "make cfg" which may cause abnormal processing afterwards. Oh My poor concept about the Makefile system :P ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=397#comment1333 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

1 0

[tracker] Comment added: Kamailio doesn't to handle any SIP message at all after several dialogs... (CPU high usage)
by sip-router 04 Mar '14

04 Mar '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#397 - Kamailio doesn't to handle any SIP message at all after several dialogs... (CPU high usage) User who did this - Jack Wang (GoGoJack) ---------- Do you mean take the -DUSE_PTHREAD_MUTEX option instead? I'll compiled with it and check the result. Thanks. ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=397#comment1332 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

1 0

[tracker] Task closed: Improvement for db_cluster
by sip-router 03 Mar '14

03 Mar '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task is now closed: FS#405 - Improvement for db_cluster User who did this - Daniel-Constantin Mierla (miconda) Reason for closing: Implemented More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=405 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

1 0

[tracker] Comment added: Improvement for db_cluster
by sip-router 03 Mar '14

03 Mar '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#405 - Improvement for db_cluster User who did this - Daniel-Constantin Mierla (miconda) ---------- Pushed to master branch. ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=405#comment1331 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

1 0

git:master: db_cluster: proper title for example in docs
by Daniel-Constantin Mierla 03 Mar '14

03 Mar '14

Module: sip-router Branch: master Commit: c318fb9bd1f3674c0bf85f8b50db030039d4e5bd URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=c318fb9… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Mon Mar 3 21:16:38 2014 +0100 db_cluster: proper title for example in docs --- modules/db_cluster/README | 4 ++-- modules/db_cluster/doc/db_cluster_admin.xml | 2 +- 2 files changed, 3 insertions(+), 3 deletions(-) diff --git a/modules/db_cluster/README b/modules/db_cluster/README index 0364bf6..8750efc 100644 --- a/modules/db_cluster/README +++ b/modules/db_cluster/README @@ -38,7 +38,7 @@ Daniel-Constantin Mierla 1.2. Set cluster parameter 1.3. Set inactive_interval parameter 1.4. Set max_query_length parameter - 1.5. Set cluster parameter + 1.5. Sample of usage Chapter 1. Admin Guide @@ -188,7 +188,7 @@ modparam("db_cluster", "max_query_length", 5) to be used for parallel writing from acc and round-robin reading by sqlops. - Example 1.5. Set cluster parameter + Example 1.5. Sample of usage ... modparam("db_cluster", "connection", "c1=>mysql://kamailio:kamailiorw@localhost/kamailio1") diff --git a/modules/db_cluster/doc/db_cluster_admin.xml b/modules/db_cluster/doc/db_cluster_admin.xml index 17543fd..ba07023 100644 --- a/modules/db_cluster/doc/db_cluster_admin.xml +++ b/modules/db_cluster/doc/db_cluster_admin.xml @@ -195,7 +195,7 @@ modparam("db_cluster", "max_query_length", 5) sqlops. </para> <example> - <title>Set <varname>cluster</varname> parameter</title> + <title>Sample of usage</title> <programlisting format="linespecific"> ... modparam("db_cluster", "connection",

1 0

git:master: db_cluster: new parameter max_query_length
by Daniel-Constantin Mierla 03 Mar '14

03 Mar '14

Module: sip-router Branch: master Commit: 0411d363ab8eb9cf146852a1c6cd8cf146acfc6d URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=0411d36… Author: Morten Tryfoss <morten(a)tryfoss.no> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Mon Mar 3 19:53:42 2014 +0100 db_cluster: new parameter max_query_length - set threshold in second after which a connection is marked inactive if the query fails - allow to let connectioncs being active if the error is due to some sql mistake - default is 0 (set connection innactive immediately, same behavior as so far) --- modules/db_cluster/README | 23 ++++++++++++++++- modules/db_cluster/db_cluster_mod.c | 2 + modules/db_cluster/dbcl_api.c | 34 +++++++++++++++++++++++---- modules/db_cluster/doc/db_cluster_admin.xml | 23 ++++++++++++++++++ 4 files changed, 75 insertions(+), 7 deletions(-) diff --git a/modules/db_cluster/README b/modules/db_cluster/README index aecbcee..0364bf6 100644 --- a/modules/db_cluster/README +++ b/modules/db_cluster/README @@ -28,6 +28,7 @@ Daniel-Constantin Mierla 3.1. connection (str) 3.2. cluster (str) 3.3. inactive_interval (int) + 3.4. max_query_length (int) 4. Usage @@ -36,7 +37,8 @@ Daniel-Constantin Mierla 1.1. Set connection parameter 1.2. Set cluster parameter 1.3. Set inactive_interval parameter - 1.4. Set cluster parameter + 1.4. Set max_query_length parameter + 1.5. Set cluster parameter Chapter 1. Admin Guide @@ -53,6 +55,7 @@ Chapter 1. Admin Guide 3.1. connection (str) 3.2. cluster (str) 3.3. inactive_interval (int) + 3.4. max_query_length (int) 4. Usage @@ -90,6 +93,7 @@ Chapter 1. Admin Guide 3.1. connection (str) 3.2. cluster (str) 3.3. inactive_interval (int) + 3.4. max_query_length (int) 3.1. connection (str) @@ -148,6 +152,21 @@ modparam("db_cluster", "cluster", "cls1=>con1=9s8p;con2=9s8p") modparam("db_cluster", "inactive_interval", 180) ... +3.4. max_query_length (int) + + How long (seconds) a failed db operation needs to last before + deactivating the connection for inactive_interval seconds. This + prevents disabling of connections that reply fast with error codes, + thus being active (e.g., due to primary key insert errors). In such + cases, the database server is active. + + Default value is 0. + + Example 1.4. Set max_query_length parameter +... +modparam("db_cluster", "max_query_length", 5) +... + 4. Usage Practically, all the modules that want to use a cluster, have to set @@ -169,7 +188,7 @@ modparam("db_cluster", "inactive_interval", 180) to be used for parallel writing from acc and round-robin reading by sqlops. - Example 1.4. Set cluster parameter + Example 1.5. Set cluster parameter ... modparam("db_cluster", "connection", "c1=>mysql://kamailio:kamailiorw@localhost/kamailio1") diff --git a/modules/db_cluster/db_cluster_mod.c b/modules/db_cluster/db_cluster_mod.c index c0b78cb..1d15531 100644 --- a/modules/db_cluster/db_cluster_mod.c +++ b/modules/db_cluster/db_cluster_mod.c @@ -37,6 +37,7 @@ int dbcl_con_param(modparam_t type, void *val); int dbcl_cls_param(modparam_t type, void *val); int dbcl_inactive_interval = 300; +int dbcl_max_query_length = 0; /*! \brief * DB Cluster module interface @@ -53,6 +54,7 @@ static param_export_t params[] = { {"connection", STR_PARAM|USE_FUNC_PARAM, (void*)dbcl_con_param}, {"cluster", STR_PARAM|USE_FUNC_PARAM, (void*)dbcl_cls_param}, {"inactive_interval", INT_PARAM, &dbcl_inactive_interval}, + {"max_query_length", INT_PARAM, &dbcl_max_query_length}, {0, 0, 0} }; diff --git a/modules/db_cluster/dbcl_api.c b/modules/db_cluster/dbcl_api.c index 0269343..2629a84 100644 --- a/modules/db_cluster/dbcl_api.c +++ b/modules/db_cluster/dbcl_api.c @@ -33,10 +33,12 @@ #include "../../trim.h" #include "../../globals.h" #include "../../lib/srdb1/db.h" +#include "../../timer.h" #include "dbcl_data.h" #include "dbcl_api.h" +extern int dbcl_max_query_length; #define DBCL_READ(command) \ do {\ @@ -44,6 +46,7 @@ int i;\ int j;\ int k;\ + unsigned int sec = 0;\ db1_con_t *dbh=NULL;\ dbcl_cls_t *cls=NULL;\ cls = (dbcl_cls_t*)_h->tail;\ @@ -60,6 +63,7 @@ {\ LM_DBG("serial operation - cluster [%.*s] (%d/%d)\n",\ cls->name.len, cls->name.s, i, j);\ + sec = get_ticks();\ dbh = cls->rlist[i].clist[j]->dbh;\ ret = cls->rlist[i].clist[j]->dbf.command;\ if (ret==0) {\ @@ -69,7 +73,10 @@ LM_DBG("serial operation - failre on cluster"\ " [%.*s] (%d/%d)\n",\ cls->name.len, cls->name.s, i, j);\ - dbcl_inactive_con(cls->rlist[i].clist[j]);\ + sec = get_ticks() - sec;\ + if(sec >= dbcl_max_query_length){\ + dbcl_inactive_con(cls->rlist[i].clist[j]);\ + }\ }\ }\ }\ @@ -83,6 +90,7 @@ {\ LM_DBG("round robin operation - cluster [%.*s] (%d/%d)\n",\ cls->name.len, cls->name.s, i, j);\ + sec = get_ticks();\ dbh = cls->rlist[i].clist[j]->dbh;\ ret = cls->rlist[i].clist[j]->dbf.command;\ if (ret==0)\ @@ -94,7 +102,10 @@ LM_DBG("round robin operation - failre on cluster"\ " [%.*s] (%d/%d)\n",\ cls->name.len, cls->name.s, i, j);\ - dbcl_inactive_con(cls->rlist[i].clist[j]);\ + sec = get_ticks() - sec;\ + if(sec >= dbcl_max_query_length){\ + dbcl_inactive_con(cls->rlist[i].clist[j]);\ + }\ }\ }\ }\ @@ -118,6 +129,7 @@ int i;\ int j;\ int k;\ + unsigned int sec = 0;\ db1_con_t *dbh=NULL;\ dbcl_cls_t *cls=NULL;\ cls = (dbcl_cls_t*)_h->tail;\ @@ -136,6 +148,7 @@ {\ LM_DBG("serial operation - cluster [%.*s] (%d/%d)\n",\ cls->name.len, cls->name.s, i, j);\ + sec = get_ticks();\ dbh = cls->wlist[i].clist[j]->dbh;\ ret = cls->wlist[i].clist[j]->dbf.command;\ if (ret==0) {\ @@ -145,7 +158,10 @@ LM_DBG("serial operation - failure on cluster"\ " [%.*s] (%d/%d)\n",\ cls->name.len, cls->name.s, i, j);\ - dbcl_inactive_con(cls->wlist[i].clist[j]);\ + sec = get_ticks() - sec;\ + if(sec >= dbcl_max_query_length){\ + dbcl_inactive_con(cls->wlist[i].clist[j]);\ + }\ }\ }\ }\ @@ -159,6 +175,7 @@ {\ LM_DBG("round robin operation - cluster [%.*s] (%d/%d)\n",\ cls->name.len, cls->name.s, i, j);\ + sec = get_ticks();\ dbh = cls->wlist[i].clist[j]->dbh;\ ret = cls->wlist[i].clist[j]->dbf.command;\ if (ret==0)\ @@ -170,7 +187,10 @@ LM_DBG("round robin operation - failure on cluster"\ " [%.*s] (%d/%d)\n",\ cls->name.len, cls->name.s, i, j);\ - dbcl_inactive_con(cls->wlist[i].clist[j]);\ + sec = get_ticks() - sec;\ + if(sec >= dbcl_max_query_length){\ + dbcl_inactive_con(cls->wlist[i].clist[j]);\ + }\ }\ }\ }\ @@ -183,6 +203,7 @@ {\ LM_DBG("parallel operation - cluster [%.*s] (%d/%d)\n",\ cls->name.len, cls->name.s, i, j);\ + sec = get_ticks();\ dbh = cls->wlist[i].clist[j]->dbh;\ rc = cls->wlist[i].clist[j]->dbf.command;\ if(rc==0) {\ @@ -192,7 +213,10 @@ LM_DBG("parallel operation - failure on cluster"\ " [%.*s] (%d/%d)\n",\ cls->name.len, cls->name.s, i, j);\ - dbcl_inactive_con(cls->wlist[i].clist[j]);\ + sec = get_ticks() - sec;\ + if(sec >= dbcl_max_query_length){\ + dbcl_inactive_con(cls->wlist[i].clist[j]);\ + }\ }\ ret |= rc;\ }\ diff --git a/modules/db_cluster/doc/db_cluster_admin.xml b/modules/db_cluster/doc/db_cluster_admin.xml index d849f20..17543fd 100644 --- a/modules/db_cluster/doc/db_cluster_admin.xml +++ b/modules/db_cluster/doc/db_cluster_admin.xml @@ -146,6 +146,29 @@ modparam("db_cluster", "inactive_interval", 180) </programlisting> </example> </section> + <section id="db_cluster.p.max_query_length"> + <title><varname>max_query_length</varname> (int)</title> + <para> + How long (seconds) a failed db operation needs to last before + deactivating the connection for inactive_interval seconds. This + prevents disabling of connections that reply fast with error + codes, thus being active (e.g., due to primary key insert errors). + In such cases, the database server is active. + </para> + <para> + <emphasis> + Default value is 0. + </emphasis> + </para> + <example> + <title>Set <varname>max_query_length</varname> parameter</title> + <programlisting format="linespecific"> +... +modparam("db_cluster", "max_query_length", 5) +... +</programlisting> + </example> + </section> </section> <section> <title>Usage</title>

1 0

[tracker] Task closed: uac_redirect not honoring q-value for max_total = 1
by sip-router 03 Mar '14

03 Mar '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task is now closed: FS#349 - uac_redirect not honoring q-value for max_total = 1 User who did this - Daniel-Constantin Mierla (miconda) Reason for closing: Fixed More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=349 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

1 0

[tracker] Comment added: uac_redirect not honoring q-value for max_total = 1
by sip-router 03 Mar '14

03 Mar '14

THIS IS AN AUTOMATED MESSAGE, DO NOT REPLY. The following task has a new comment added: FS#349 - uac_redirect not honoring q-value for max_total = 1 User who did this - Daniel-Constantin Mierla (miconda) ---------- Finally got some time to come back to it. I pushed a patch in master for now: - http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=a9b3286… If you can test and report when all is ok, then I will backport. If any issues, then reopen this item. ---------- More information can be found at the following URL: http://sip-router.org/tracker/index.php?do=details&task_id=349#comment1330 You are receiving this message because you have requested it from the Flyspray bugtracking system. If you did not expect this message or don't want to receive mails in future, you can change your notification settings at the URL shown above.

1 0

git:master: uac_redirect: select the contacts with highest q when a limit is set
by Daniel-Constantin Mierla 03 Mar '14

03 Mar '14

Module: sip-router Branch: master Commit: a9b3286a5ace0f66852054634b46c0647c8d1271 URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=a9b3286… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Mon Mar 3 19:15:30 2014 +0100 uac_redirect: select the contacts with highest q when a limit is set - reported by Stefan Prelle, FS#349 --- modules/uac_redirect/rd_funcs.c | 12 ++++++++---- 1 files changed, 8 insertions(+), 4 deletions(-) diff --git a/modules/uac_redirect/rd_funcs.c b/modules/uac_redirect/rd_funcs.c index f85d84d..6bed77f 100644 --- a/modules/uac_redirect/rd_funcs.c +++ b/modules/uac_redirect/rd_funcs.c @@ -165,7 +165,7 @@ static int sort_contacts(hdr_field_t *chdr, contact_t **ct_array, ct_list->uri.len,ct_list->uri.s,q); /*insert the contact into the sorted array */ for(i=0;i<n;i++) { - /* keep in mind that the contact list is reversts */ + /* keep in mind that the contact list is reversed */ if (q_array[i]<=q) continue; break; @@ -285,14 +285,18 @@ static int shmcontact2dset(struct sip_msg *req, struct sip_msg *sh_rpl, goto restore; } - /* to many branches ? */ + i=0; + + /* more branches than requested in the parameter + * - add only the last ones from sorted array, + * because the order is by increasing q */ if (max!=-1 && n>max) - n = max; + i = n - max; added = 0; /* add the sortet contacts as branches in dset and log this! */ - for ( i=0 ; i<n ; i++ ) { + for ( ; i<n ; i++ ) { LM_DBG("adding contact <%.*s>\n", scontacts[i]->uri.len, scontacts[i]->uri.s); if(sruid_next(&_redirect_sruid)==0) {

1 0

git:master: kamctl: db connect - new command for db management subset
by Daniel-Constantin Mierla 03 Mar '14

03 Mar '14

Module: sip-router Branch: master Commit: e395c9bb0131d7fce80af67c54cd813532e5f2ed URL: http://git.sip-router.org/cgi-bin/gitweb.cgi/sip-router/?a=commit;h=e395c9b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Mon Mar 3 18:16:54 2014 +0100 kamctl: db connect - new command for db management subset - execute to command line client tool and stay connected using the attributes from kamctlrc - handy when kamctlrc is set, to avoid typing all the attributes when willing to connect to kamailio database --- utils/kamctl/kamctl | 7 +++++++ utils/kamctl/kamctl.mysql | 7 +++++++ 2 files changed, 14 insertions(+), 0 deletions(-) diff --git a/utils/kamctl/kamctl b/utils/kamctl/kamctl index 519da55..30dddcf 100755 --- a/utils/kamctl/kamctl +++ b/utils/kamctl/kamctl @@ -1010,6 +1010,13 @@ db_ops() { QUERY="SELECT * FROM $1 WHERE $2=$3\\G;" $DBROCMD "$QUERY" ;; + connect) + shift + if [ -z "$DBCMDCONNECT" ] ; then + merr "extension not implemented" + exit 1 + fi + $DBCMDCONNECT *) usage_db_ops exit 1 diff --git a/utils/kamctl/kamctl.mysql b/utils/kamctl/kamctl.mysql index e533e9c..1b25d2a 100644 --- a/utils/kamctl/kamctl.mysql +++ b/utils/kamctl/kamctl.mysql @@ -42,7 +42,14 @@ mysql_ro_query() { $MYSQL $2 -h $DBHOST -u $DBROUSER "-p$DBROPW" -e "$1 ;" $DBNAME } +mysql_connect() { + # if password not yet queried, query it now + prompt_pw "MySQL password for user '$DBRWUSER@$DBHOST'" + $MYSQL $2 -h $DBHOST -u $DBRWUSER "-p$DBRWPW" $DBNAME +} + DBCMD=mysql_query DBROCMD=mysql_ro_query DBRAWPARAMS="-B" +DBCMDCONNECT=mysql_connect

1 0

Jump to page:

2025

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev March 2014