Re: [sr-dev] [kamailio/kamailio] Extend keepalive module documentation (#1082)

Hi @agranig, @kamailio-sync @ First, the module is still young, so the documentation may be incomplete. I will improve it asap Second, my primary idea when started this module was to implement the same "destinations probing" mechanism found in dispatcher module, both in drouting & carrierroute

Re: [sr-dev] git:5.0:4ad64c0f: db_text: add db_delim parameter

Hello, I did the revert so the debian nightly builds from branch 5.0 don't mix with the backport. Cheers, Daniel On 24.04.17 16:56, Daniel-Constantin Mierla wrote: > Hello, > > nothing to be really sorry about, we collaborate here and it is the role > of github to make it easy to revert. > > I

[sr-dev] git:5.0:a3b376a1: modules: readme files regenerated - modules ... [skip ci]

Module: kamailio Branch: 5.0 Commit: a3b376a178532508e595c7bb1ed9278351f9c0c4 URL: https://github.com/kamailio/kamailio/commit/a3b376a178532508e595c7bb1ed9278351f9c0c4 Author: Kamailio Dev Committer: Kamailio Dev Date:

[sr-dev] git:5.0:1dad5027: Revert "db_text: add db_delim parameter"

Module: kamailio Branch: 5.0 Commit: 1dad50274d4f22a50d90894b1b6dfa64baa7c638 URL: https://github.com/kamailio/kamailio/commit/1dad50274d4f22a50d90894b1b6dfa64baa7c638 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

[sr-dev] git:master:24f507f4: app_sqlang: use only function name in the exports table

Module: kamailio Branch: master Commit: 24f507f4f0643cd6157b6ac1339be77afe0bdd64 URL: https://github.com/kamailio/kamailio/commit/24f507f4f0643cd6157b6ac1339be77afe0bdd64 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

Re: [sr-dev] [kamailio/kamailio] pkg/kamailio: Fix of ticket 1085 (#1086)

@linuxmaniac we can also mark exit code for broken config as SuccessExitStatus. Than unit will not restart in case of broken config -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] ndb_redis: add pipeline suppport for REDIS Module (#1079)

Hello, there was a problem because _redisc_srv_list type was modified. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] ndb_redis: add pipeline suppport for REDIS Module (#1079)

@claudiupb pushed 1 commit. 3f696ae fix compilation error -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/kamailio/kamailio/pull/1079/files/83c68c5ededc0a5f06bb8d2fa3d778e7a6a17675..3f696aea111823bb0641ad9f255a6fb5bb15b0d3

Re: [sr-dev] [kamailio/kamailio] ndb_redis: add pipeline suppport for REDIS Module (#1079)

@claudiupb - the travis-ci is reporting compilation errors with the patches, see the details the post above. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] git:5.0:4ad64c0f: db_text: add db_delim parameter

Hello, nothing to be really sorry about, we collaborate here and it is the role of github to make it easy to revert. I think it is good to revert to keep it clean, as we build packages over the night and some people are using them. If this is something that solve some problems for you, then you

Re: [sr-dev] git:5.0:4ad64c0f: db_text: add db_delim parameter

Hello Daniel, I am sorry for this situation; do I revert the commit? Thank you, Lucian On 24.04.2017 17:41, Daniel-Constantin Mierla wrote: Hello, the rule is not to backport new features to stable versions, because we will end up in a mixture of untested code where stability cannot be

[sr-dev] git:master:8be11111: app_sqlang: link against libstdc++

Module: kamailio Branch: master Commit: 8be15e2748cbde4ce841ae72b49badb7e4d5 URL: https://github.com/kamailio/kamailio/commit/8be15e2748cbde4ce841ae72b49badb7e4d5 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

Re: [sr-dev] git:5.0:4ad64c0f: db_text: add db_delim parameter

Hello, the rule is not to backport new features to stable versions, because we will end up in a mixture of untested code where stability cannot be ensured. I don't feel that this commit is fixing an existing issues, rather making it more aesthetic in some cases. If I got it wrong, please

[sr-dev] git:master:50718957: misc/examples: more complex example of using sqlang kemi interpreter

Module: kamailio Branch: master Commit: 5071895772c42699bad6cfc982c89e3faccad29f URL: https://github.com/kamailio/kamailio/commit/5071895772c42699bad6cfc982c89e3faccad29f Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

Re: [sr-dev] [kamailio/kamailio] ndb_redis: add pipeline suppport for REDIS Module (#1079)

I resolved the conflicts in the PR. -- 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/1079#issuecomment-296679014___ Kamailio (SER) -

Re: [sr-dev] [kamailio/kamailio] ndb_redis: add pipeline suppport for REDIS Module (#1079)

@claudiupb pushed 1 commit. 83c68c5 fix indentation -- You are receiving this because you are subscribed to this thread. View it on GitHub: https://github.com/kamailio/kamailio/pull/1079/files/f3a00aa8f78a441cc427f43ba65ae8ef8434a2be..83c68c5ededc0a5f06bb8d2fa3d778e7a6a17675

Re: [sr-dev] [kamailio/kamailio] ndb_redis: add pipeline suppport for REDIS Module (#1079)

@claudiupb pushed 1 commit. f3a00aa Merge branch 'master' into redis_pipeline -- You are receiving this because you are subscribed to this thread. View it on GitHub:

Re: [sr-dev] [kamailio/kamailio] ndb_redis: add pipeline suppport for REDIS Module (#1079)

@claudiupb - can you resolve the conflicts so this PR can be merged? The module was in a broken state at the time you did the PR, which needed to be fixed, sorry for the extra work. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on

[sr-dev] git:master:3efdeefd: acc: exported functions to kemi framework

Module: kamailio Branch: master Commit: 3efdeefdf80c3bc544b19f260981ea5d98d71ff7 URL: https://github.com/kamailio/kamailio/commit/3efdeefdf80c3bc544b19f260981ea5d98d71ff7 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

[sr-dev] git:master:ec8f7722: app_sqlang: fixed compile warnings about unused code

Module: kamailio Branch: master Commit: ec8f7722b8e9380f1357796da33da735cfb72913 URL: https://github.com/kamailio/kamailio/commit/ec8f7722b8e9380f1357796da33da735cfb72913 Author: Daniel-Constantin Mierla Committer: Daniel-Constantin Mierla Date:

Re: [sr-dev] [kamailio/kamailio] dialog: execute pre script callback only when event route is set (7004969)

forget it, I was mistaken. sorry for the noise -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

[sr-dev] git:master:73ae92bc: modules: readme files regenerated - ndb_redis ... [skip ci]

Module: kamailio Branch: master Commit: 73ae92bc900472eb6afee12da6265798a59a8cef URL: https://github.com/kamailio/kamailio/commit/73ae92bc900472eb6afee12da6265798a59a8cef Author: Kamailio Dev Committer: Kamailio Dev Date:

[sr-dev] git:master:3c1f6a51: Fix ndb_redis Cluster implementation

Module: kamailio Branch: master Commit: 3c1f6a51e830f6b3c3d98b32178c3cf6af16f62d URL: https://github.com/kamailio/kamailio/commit/3c1f6a51e830f6b3c3d98b32178c3cf6af16f62d Author: Carsten Bock Committer: Carsten Bock Date: 2017-04-24T14:23:47+02:00

Re: [sr-dev] [kamailio/kamailio] pkg/kamailio: Fix of ticket 1085 (#1086)

@linuxmaniac yes, unit will be restarted infinity, Rate of restarts may limiter using "RestartSec" keyword -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] Daemon not restated if process exited with not null code (#1085)

Hello @linuxmaniac Yes, I want say "non-zero exit code" or "Unclean exit code" Clean exit code is "0" or codes marked as "SuccessExitStatus" Please see same link > SuccessExitStatus=1 2 8 SIGKILL > ensures that exit codes 1, 2, 8 and the termination signal SIGKILL are > considered clean

[sr-dev] git:master:e94e9d8b: Merge pull request #1078 from lbalaceanu/replace_db_text_delim

Module: kamailio Branch: master Commit: e94e9d8bdcdb3d340b14868f9d7e52ef0ab71894 URL: https://github.com/kamailio/kamailio/commit/e94e9d8bdcdb3d340b14868f9d7e52ef0ab71894 Author: Lucian Balaceanu Committer: GitHub Date: 2017-04-24T14:07:59+03:00

Re: [sr-dev] [kamailio/kamailio] pkg/kamailio: Fix of ticket 1085 (#1086)

What happens if you have a non-working config and you have `on-failure`? I would assume the daemon will be restarted forever -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] Daemon not restated if process exited with not null code (#1085)

What do you mean by 'not null code'? >From https://www.freedesktop.org/software/systemd/man/systemd.service.html ``` Restart: [...] If set to on-failure, the service will be restarted when the process exits with a non-zero exit code, is terminated by a signal (including on core dump, but

Re: [sr-dev] [kamailio/kamailio] pkg/kamailio: Fix of ticket 1085 (#1086)

See my comment at https://github.com/kamailio/kamailio/issues/1085#issuecomment-296592791 -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] pkg/kamailio: Fix of ticket 1085 (#1086)

@linuxmaniac - are you ok with the changes proposed for deb packaging regarding the systemd? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] ndb_redis: add pipeline suppport for REDIS Module (#1079)

Pinging Carsten (@ngvoice) -- is it needed to wait for fixing the changes done by using shm memory or this PR can be merged? -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub:

Re: [sr-dev] [kamailio/kamailio] db_text: change field separator from ":" to "|" (#1078)

Thanks! It is ok to merge. -- 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/1078#issuecomment-296590125___ Kamailio (SER) - Development

Re: [sr-dev] [kamailio/kamailio] db_text: change field separator from ":" to "|" (#1078)

Updated code in the merge branch. If this also looks good to you I will close the merge. -- You are receiving this because you are subscribed to this thread. Reply to this email directly or view it on GitHub: