sr-dev April 2017

sr-dev@lists.kamailio.org

31 participants
426 discussions

by Daniel-Constantin Mierla

Hello, I am considering to release Kamailio v5.0.1 sometime next week, likely on Wednesday, April 5, 2017. Should anyone be aware of issues not listed yet on bug tracker, report them there as soon as possible to try to fix. Soon after, we should release a new version from branch 4.4 and the last one from branch 4.3. Cheers, Daniel -- Daniel-Constantin Mierla www.twitter.com/miconda -- www.linkedin.com/in/miconda Kamailio Advanced Training - May 22-24 (USA) - www.asipto.com Kamailio World Conference - May 8-10, 2017 - www.kamailioworld.com

7 years, 7 months

git:5.0:c5d5f520: pkg/kamailio/deb: version set to 5.0.1

by Victor Seva

Module: kamailio Branch: 5.0 Commit: c5d5f5203dd66dd6e83644973f128ca2c1551c32 URL: https://github.com/kamailio/kamailio/commit/c5d5f5203dd66dd6e83644973f128ca… Author: Victor Seva <linuxmaniac(a)torreviejawireless.org> Committer: Victor Seva <linuxmaniac(a)torreviejawireless.org> Date: 2017-04-05T10:59:43+02:00 pkg/kamailio/deb: version set to 5.0.1 --- Modified: pkg/kamailio/deb/debian/changelog Modified: pkg/kamailio/deb/jessie/changelog Modified: pkg/kamailio/deb/precise/changelog Modified: pkg/kamailio/deb/sid/changelog Modified: pkg/kamailio/deb/stretch/changelog Modified: pkg/kamailio/deb/trusty/changelog Modified: pkg/kamailio/deb/wheezy/changelog Modified: pkg/kamailio/deb/xenial/changelog --- Diff: https://github.com/kamailio/kamailio/commit/c5d5f5203dd66dd6e83644973f128ca… Patch: https://github.com/kamailio/kamailio/commit/c5d5f5203dd66dd6e83644973f128ca… --- diff --git a/pkg/kamailio/deb/debian/changelog b/pkg/kamailio/deb/debian/changelog index 11e8940..673f70e 100644 --- a/pkg/kamailio/deb/debian/changelog +++ b/pkg/kamailio/deb/debian/changelog @@ -1,3 +1,9 @@ +kamailio (5.0.1) unstable; urgency=medium + + * version set to 5.0.1 + + -- Victor Seva <vseva(a)debian.org> Wed, 05 Apr 2017 10:59:02 +0200 + kamailio (5.0.0) unstable; urgency=medium * version set to 5.0.0 diff --git a/pkg/kamailio/deb/jessie/changelog b/pkg/kamailio/deb/jessie/changelog index 11e8940..673f70e 100644 --- a/pkg/kamailio/deb/jessie/changelog +++ b/pkg/kamailio/deb/jessie/changelog @@ -1,3 +1,9 @@ +kamailio (5.0.1) unstable; urgency=medium + + * version set to 5.0.1 + + -- Victor Seva <vseva(a)debian.org> Wed, 05 Apr 2017 10:59:02 +0200 + kamailio (5.0.0) unstable; urgency=medium * version set to 5.0.0 diff --git a/pkg/kamailio/deb/precise/changelog b/pkg/kamailio/deb/precise/changelog index 11e8940..673f70e 100644 --- a/pkg/kamailio/deb/precise/changelog +++ b/pkg/kamailio/deb/precise/changelog @@ -1,3 +1,9 @@ +kamailio (5.0.1) unstable; urgency=medium + + * version set to 5.0.1 + + -- Victor Seva <vseva(a)debian.org> Wed, 05 Apr 2017 10:59:02 +0200 + kamailio (5.0.0) unstable; urgency=medium * version set to 5.0.0 diff --git a/pkg/kamailio/deb/sid/changelog b/pkg/kamailio/deb/sid/changelog index 11e8940..673f70e 100644 --- a/pkg/kamailio/deb/sid/changelog +++ b/pkg/kamailio/deb/sid/changelog @@ -1,3 +1,9 @@ +kamailio (5.0.1) unstable; urgency=medium + + * version set to 5.0.1 + + -- Victor Seva <vseva(a)debian.org> Wed, 05 Apr 2017 10:59:02 +0200 + kamailio (5.0.0) unstable; urgency=medium * version set to 5.0.0 diff --git a/pkg/kamailio/deb/stretch/changelog b/pkg/kamailio/deb/stretch/changelog index 11e8940..673f70e 100644 --- a/pkg/kamailio/deb/stretch/changelog +++ b/pkg/kamailio/deb/stretch/changelog @@ -1,3 +1,9 @@ +kamailio (5.0.1) unstable; urgency=medium + + * version set to 5.0.1 + + -- Victor Seva <vseva(a)debian.org> Wed, 05 Apr 2017 10:59:02 +0200 + kamailio (5.0.0) unstable; urgency=medium * version set to 5.0.0 diff --git a/pkg/kamailio/deb/trusty/changelog b/pkg/kamailio/deb/trusty/changelog index 11e8940..673f70e 100644 --- a/pkg/kamailio/deb/trusty/changelog +++ b/pkg/kamailio/deb/trusty/changelog @@ -1,3 +1,9 @@ +kamailio (5.0.1) unstable; urgency=medium + + * version set to 5.0.1 + + -- Victor Seva <vseva(a)debian.org> Wed, 05 Apr 2017 10:59:02 +0200 + kamailio (5.0.0) unstable; urgency=medium * version set to 5.0.0 diff --git a/pkg/kamailio/deb/wheezy/changelog b/pkg/kamailio/deb/wheezy/changelog index 11e8940..673f70e 100644 --- a/pkg/kamailio/deb/wheezy/changelog +++ b/pkg/kamailio/deb/wheezy/changelog @@ -1,3 +1,9 @@ +kamailio (5.0.1) unstable; urgency=medium + + * version set to 5.0.1 + + -- Victor Seva <vseva(a)debian.org> Wed, 05 Apr 2017 10:59:02 +0200 + kamailio (5.0.0) unstable; urgency=medium * version set to 5.0.0 diff --git a/pkg/kamailio/deb/xenial/changelog b/pkg/kamailio/deb/xenial/changelog index 11e8940..673f70e 100644 --- a/pkg/kamailio/deb/xenial/changelog +++ b/pkg/kamailio/deb/xenial/changelog @@ -1,3 +1,9 @@ +kamailio (5.0.1) unstable; urgency=medium + + * version set to 5.0.1 + + -- Victor Seva <vseva(a)debian.org> Wed, 05 Apr 2017 10:59:02 +0200 + kamailio (5.0.0) unstable; urgency=medium * version set to 5.0.0

7 years, 7 months

git:master:fc732895: modules: readme files regenerated - mtree ... [skip ci]

by Kamailio Dev

Module: kamailio Branch: master Commit: fc732895c19f2e9fcb00c8b55d5eee0c4fffaf52 URL: https://github.com/kamailio/kamailio/commit/fc732895c19f2e9fcb00c8b55d5eee0… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2017-04-05T09:46:40+02:00 modules: readme files regenerated - mtree ... [skip ci] --- Modified: src/modules/mtree/README --- Diff: https://github.com/kamailio/kamailio/commit/fc732895c19f2e9fcb00c8b55d5eee0… Patch: https://github.com/kamailio/kamailio/commit/fc732895c19f2e9fcb00c8b55d5eee0… --- diff --git a/src/modules/mtree/README b/src/modules/mtree/README index 541d744..18f3386 100644 --- a/src/modules/mtree/README +++ b/src/modules/mtree/README @@ -175,14 +175,15 @@ modparam("mtree", "db_url", "dbdriver://username:password@dbhost/dbname") 3.2. db_table (string) - Name of database table where data for trees is stored. It is ignored if - a 'mtree' parameter is defined. + Name of database table where data for many trees is stored. It is + ignored if a 'mtree' parameter is defined. The SQL scripts creates a + table named 'mtrees' that can be used for this parameter. - Default value is “mtrees”. + Default value is “” (no table name). Example 1.2. Set db_table parameter ... -modparam("mtree", "db_table", "mymtrees") +modparam("mtree", "db_table", "mtrees") ... 3.3. mtree (string)

7 years, 7 months

git:5.0:deefd28f: modules: readme files regenerated - modules ... [skip ci]

by Kamailio Dev

Module: kamailio Branch: 5.0 Commit: deefd28f893bee96f91591dec9d3ec93c39cf3c1 URL: https://github.com/kamailio/kamailio/commit/deefd28f893bee96f91591dec9d3ec9… Author: Kamailio Dev <kamailio.dev(a)kamailio.org> Committer: Kamailio Dev <kamailio.dev(a)kamailio.org> Date: 2017-04-05T09:46:20+02:00 modules: readme files regenerated - modules ... [skip ci] --- Modified: src/modules/mtree/README --- Diff: https://github.com/kamailio/kamailio/commit/deefd28f893bee96f91591dec9d3ec9… Patch: https://github.com/kamailio/kamailio/commit/deefd28f893bee96f91591dec9d3ec9… --- diff --git a/src/modules/mtree/README b/src/modules/mtree/README index 541d744..18f3386 100644 --- a/src/modules/mtree/README +++ b/src/modules/mtree/README @@ -175,14 +175,15 @@ modparam("mtree", "db_url", "dbdriver://username:password@dbhost/dbname") 3.2. db_table (string) - Name of database table where data for trees is stored. It is ignored if - a 'mtree' parameter is defined. + Name of database table where data for many trees is stored. It is + ignored if a 'mtree' parameter is defined. The SQL scripts creates a + table named 'mtrees' that can be used for this parameter. - Default value is “mtrees”. + Default value is “” (no table name). Example 1.2. Set db_table parameter ... -modparam("mtree", "db_table", "mymtrees") +modparam("mtree", "db_table", "mtrees") ... 3.3. mtree (string)

7 years, 7 months

[kamailio/kamailio] mtree reload xmlrpc error on kamailio 5.0.0 (#1057)

by David Uczen

### Description I have 2 mtrees defined that get loaded on startup. I have not specified the `db_table` value but have changed each of the values for `tname_column`, `tprefix_column`, and `tvalue_column`. `kamcmd mtree.summary` and `kamcmd mtree.list` show that the data was successfully loaded. I expected that running `kamcmd mtree.reload` would reload these mtrees from the database. Running this or sending the same command as an http xmlrpc request yields "error: 500 - Mtree Reload Failed". The same error is received if I try to reload a specific mtree. Prior to this version we were successfully reloading these mtrees with MI commands. ### Troubleshooting #### Reproduction Use the following mtree paramters with a database table that has the name, prefix, and value fields. ``` modparam("mtree", "tname_column", "name") modparam("mtree", "tprefix_column", "prefix") modparam("mtree", "tvalue_column", "value") modparam("mtree", "mtree", "name=some_mtree;dbtable=some_table;type=0;multi=0") ``` Verify the mtrees were successfully created and attempt to reload with the mtree.reload RPC command. We are not using an mtrees table and this throws an error that the table does not exist (see logs below). Try creating the mtrees table according to the script located at utils/kamctl/mysql/mtree-create.sql and insert the tname, tprefix, and tvalue values used above. The same error is thrown with a different log entry. #### Log Messages  Without mtrees table: ``` Apr 4 21:45:51 localhost /usr/local/sbin/kamailio[26416]: ERROR: db_mysql [km_dbase.c:128]: db_mysql_submit_query(): driver error on query: Table 'db_name.mtrees' doesn't exist (1146) Apr 4 21:45:51 localhost /usr/local/sbin/kamailio[26416]: ERROR: <core> [db_query.c:132]: db_do_query_internal(): error while submitting query Apr 4 21:45:51 localhost /usr/local/sbin/kamailio[26416]: ERROR: mtree [mtree_mod.c:724]: mt_load_db_trees(): Error while querying db Apr 4 21:45:51 localhost /usr/local/sbin/kamailio[26416]: ERROR: mtree [mtree_mod.c:928]: rpc_mtree_reload(): cannot re-load mtrees from database ``` With mtrees table: ``` Apr 4 21:41:47 localhost /usr/local/sbin/kamailio[26416]: ERROR: db_mysql [km_dbase.c:128]: db_mysql_submit_query(): driver error on query: Unknown column 'name' in 'field list' (1054) Apr 4 21:41:47 localhost /usr/local/sbin/kamailio[26416]: ERROR: <core> [db_query.c:132]: db_do_query_internal(): error while submitting query Apr 4 21:41:47 localhost /usr/local/sbin/kamailio[26416]: ERROR: mtree [mtree_mod.c:724]: mt_load_db_trees(): Error while querying db Apr 4 21:41:47 localhost /usr/local/sbin/kamailio[26416]: ERROR: mtree [mtree_mod.c:928]: rpc_mtree_reload(): cannot re-load mtrees from database ``` ### Additional Information According to the documentation found here: http://kamailio.org/docs/modules/5.0.x/modules/mtree.html the db_table param should be ignored if a mtree paramter is defined so I would not expect to need the mtrees table. The data is successfully loaded at startup so I believe the issue is just with the reload command. Our kamailio config and database schema has not changed and we were able to reload mtrees successfully with MI commands in previous releases. * **Kamailio Version** - output of `kamailio -v` ``` version: kamailio 5.0.0 (x86_64/linux) flags: STATS: Off, USE_TCP, USE_TLS, USE_SCTP, TLS_HOOKS, DISABLE_NAGLE, USE_MCAST, DNS_IP_HACK, SHM_MEM, SHM_MMAP, PKG_MALLOC, Q_MALLOC, F_MALLOC, TLSF_MALLOC, DBG_SR_MEMORY, USE_FUTEX, FAST_LOCK-ADAPTIVE_WAIT, USE_DNS_CACHE, USE_DNS_FAILOVER, USE_NAPTR, USE_DST_BLACKLIST, HAVE_RESOLV_RES ADAPTIVE_WAIT_LOOPS=1024, MAX_RECV_BUFFER_SIZE 262144, MAX_LISTEN 16, MAX_URI_SIZE 1024, BUF_SIZE 65535, DEFAULT PKG_SIZE 8MB poll method support: poll, epoll_lt, epoll_et, sigio_rt, select. id: unknown compiled on 14:35:44 Apr 4 2017 with gcc 4.8.5 ``` * **Operating System**: ``` Linux proxy.vagrant.vm 3.10.0-514.6.2.el7.x86_64 #1 SMP Thu Feb 23 03:04:39 UTC 2017 x86_64 x86_64 x86_64 GNU/Linux CentOS Linux release 7.3.1611 (Core) ``` -- 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/1057

7 years, 7 months

git:5.0:2df615b6: mtree: clarifications about db table parameter in docs

by Daniel-Constantin Mierla

Module: kamailio Branch: 5.0 Commit: 2df615b6891d822d9e892513b13ff48a3be22faa URL: https://github.com/kamailio/kamailio/commit/2df615b6891d822d9e892513b13ff48… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2017-04-05T09:39:03+02:00 mtree: clarifications about db table parameter in docs (cherry picked from commit 1c1e51388aad6c5ba064c283d5e124367b30ad7e) --- Modified: src/modules/mtree/doc/mtree_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/2df615b6891d822d9e892513b13ff48… Patch: https://github.com/kamailio/kamailio/commit/2df615b6891d822d9e892513b13ff48… --- diff --git a/src/modules/mtree/doc/mtree_admin.xml b/src/modules/mtree/doc/mtree_admin.xml index ec6b2d1..0162e32 100644 --- a/src/modules/mtree/doc/mtree_admin.xml +++ b/src/modules/mtree/doc/mtree_admin.xml @@ -86,19 +86,20 @@ modparam("mtree", "db_url", "&exampledb;") <section> <title><varname>db_table</varname> (string)</title> <para> - Name of database table where data for trees is stored. It is ignored if a - 'mtree' parameter is defined. + Name of database table where data for many trees is stored. It is ignored + if a 'mtree' parameter is defined. The SQL scripts creates a table named + 'mtrees' that can be used for this parameter. </para> <para> <emphasis> - Default value is <quote>mtrees</quote>. + Default value is <quote></quote> (no table name). </emphasis> </para> <example> <title>Set <varname>db_table</varname> parameter</title> <programlisting format="linespecific"> ... -modparam("mtree", "db_table", "mymtrees") +modparam("mtree", "db_table", "mtrees") ... </programlisting> </example>

7 years, 7 months

git:5.0:97481887: mtree: revert a0e4a9c18e and let db_table name not set by default

by Daniel-Constantin Mierla

Module: kamailio Branch: 5.0 Commit: 9748188734df2a49cc40d96d42641c87b5962603 URL: https://github.com/kamailio/kamailio/commit/9748188734df2a49cc40d96d42641c8… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2017-04-05T09:38:54+02:00 mtree: revert a0e4a9c18e and let db_table name not set by default - it breaks some runtime rpc commands, reported by GH #1057 - reset it also if set by mistake when mtree param is used (cherry picked from commit a4cdacd0b63245c48fcc6e0a20c9b665ef877090) --- Modified: src/modules/mtree/mtree_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/9748188734df2a49cc40d96d42641c8… Patch: https://github.com/kamailio/kamailio/commit/9748188734df2a49cc40d96d42641c8… --- diff --git a/src/modules/mtree/mtree_mod.c b/src/modules/mtree/mtree_mod.c index ee03da8..259ef1d 100644 --- a/src/modules/mtree/mtree_mod.c +++ b/src/modules/mtree/mtree_mod.c @@ -74,7 +74,9 @@ CREATE TABLE mtrees ( /** parameters */ static str db_url = str_init(DEFAULT_DB_URL); -static str db_table = str_init("mtrees"); +/* default name created by sql scripts is 'mtrees' + * - don't set it here with default value, only via config param */ +static str db_table = str_init(""); static str tname_column = str_init("tname"); static str tprefix_column = str_init("tprefix"); static str tvalue_column = str_init("tvalue"); @@ -269,6 +271,9 @@ static int mod_init(void) } pt = pt->next; } + /* reset db_table value */ + db_table.s = ""; + db_table.len = 0; } else { if(db_table.len<=0) {

7 years, 7 months

git:master:1c1e5138: mtree: clarifications about db table parameter in docs

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 1c1e51388aad6c5ba064c283d5e124367b30ad7e URL: https://github.com/kamailio/kamailio/commit/1c1e51388aad6c5ba064c283d5e1243… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2017-04-05T09:36:17+02:00 mtree: clarifications about db table parameter in docs --- Modified: src/modules/mtree/doc/mtree_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/1c1e51388aad6c5ba064c283d5e1243… Patch: https://github.com/kamailio/kamailio/commit/1c1e51388aad6c5ba064c283d5e1243… --- diff --git a/src/modules/mtree/doc/mtree_admin.xml b/src/modules/mtree/doc/mtree_admin.xml index ec6b2d1..0162e32 100644 --- a/src/modules/mtree/doc/mtree_admin.xml +++ b/src/modules/mtree/doc/mtree_admin.xml @@ -86,19 +86,20 @@ modparam("mtree", "db_url", "&exampledb;") <section> <title><varname>db_table</varname> (string)</title> <para> - Name of database table where data for trees is stored. It is ignored if a - 'mtree' parameter is defined. + Name of database table where data for many trees is stored. It is ignored + if a 'mtree' parameter is defined. The SQL scripts creates a table named + 'mtrees' that can be used for this parameter. </para> <para> <emphasis> - Default value is <quote>mtrees</quote>. + Default value is <quote></quote> (no table name). </emphasis> </para> <example> <title>Set <varname>db_table</varname> parameter</title> <programlisting format="linespecific"> ... -modparam("mtree", "db_table", "mymtrees") +modparam("mtree", "db_table", "mtrees") ... </programlisting> </example>

7 years, 7 months

git:master:a4cdacd0: mtree: revert a0e4a9c18e and let db_table name not set by default

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: a4cdacd0b63245c48fcc6e0a20c9b665ef877090 URL: https://github.com/kamailio/kamailio/commit/a4cdacd0b63245c48fcc6e0a20c9b66… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2017-04-05T09:34:07+02:00 mtree: revert a0e4a9c18e and let db_table name not set by default - it breaks some runtime rpc commands, reported by GH #1057 - reset it also if set by mistake when mtree param is used --- Modified: src/modules/mtree/mtree_mod.c --- Diff: https://github.com/kamailio/kamailio/commit/a4cdacd0b63245c48fcc6e0a20c9b66… Patch: https://github.com/kamailio/kamailio/commit/a4cdacd0b63245c48fcc6e0a20c9b66… --- diff --git a/src/modules/mtree/mtree_mod.c b/src/modules/mtree/mtree_mod.c index ee03da8..259ef1d 100644 --- a/src/modules/mtree/mtree_mod.c +++ b/src/modules/mtree/mtree_mod.c @@ -74,7 +74,9 @@ CREATE TABLE mtrees ( /** parameters */ static str db_url = str_init(DEFAULT_DB_URL); -static str db_table = str_init("mtrees"); +/* default name created by sql scripts is 'mtrees' + * - don't set it here with default value, only via config param */ +static str db_table = str_init(""); static str tname_column = str_init("tname"); static str tprefix_column = str_init("tprefix"); static str tvalue_column = str_init("tvalue"); @@ -269,6 +271,9 @@ static int mod_init(void) } pt = pt->next; } + /* reset db_table value */ + db_table.s = ""; + db_table.len = 0; } else { if(db_table.len<=0) {

7 years, 7 months

git:master:338e5df6: keepalive: format to match default conding style and 80 chars lines

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 338e5df6f96ccf1f3df313593d3fc9bb228f5d85 URL: https://github.com/kamailio/kamailio/commit/338e5df6f96ccf1f3df313593d3fc9b… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2017-04-05T08:52:01+02:00 keepalive: format to match default conding style and 80 chars lines --- Modified: src/modules/keepalive/api.h Modified: src/modules/keepalive/keepalive.h Modified: src/modules/keepalive/keepalive_api.c Modified: src/modules/keepalive/keepalive_core.c Modified: src/modules/keepalive/keepalive_mod.c Modified: src/modules/keepalive/keepalive_rpc.c --- Diff: https://github.com/kamailio/kamailio/commit/338e5df6f96ccf1f3df313593d3fc9b… Patch: https://github.com/kamailio/kamailio/commit/338e5df6f96ccf1f3df313593d3fc9b…

7 years, 7 months

← Newer
1
...
34
35
36
37
38
39
40
...
43
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev April 2017