sr-dev September 2017

sr-dev@lists.kamailio.org

26 participants
271 discussions

[kamailio/kamailio] kamailio inserts buggy sip header (#1257)

by defeden

when kamailio try to dispatch the requests to Asterisk, it inserts a sip header without newline, so the sip header will be concatenated with the next header herein an example: sip:edi-2001@217.108.243.20:12983;transport=TCP;obContact: <sip:edi-2001@10.42.0.11:5060;transport=UDP> -- 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/1257

6 years, 7 months

[kamailio/kamailio] pkg/kamailio/obs: Created links for dists: (#1244)

by sergey-safarov

- Fedora 25, 26 - RHEL 6, 7 - OpenSUSE Leap, Tumbleweed #### Pre-Submission Checklist - [ ] Commit message has the format required by CONTRIBUTING guide - [x] Commits are split per component (core, individual modules, libs, utils, ...) - [x] Each component has a single commit (if not, squash them into one commit) - [ x No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### Type Of Change - [ ] Small bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality) #### Checklist: - [ ] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description Purpose of PR - trac packaging in one SPEC file for all RPM based dists. You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1244 -- Commit Summary -- * pkg/kamailio/obs: Created links for dists: -- File Changes -- A pkg/kamailio/fedora/25 (1) A pkg/kamailio/fedora/26 (1) M pkg/kamailio/obs/kamailio.spec (9) A pkg/kamailio/opensuse/1315 (1) A pkg/kamailio/opensuse/1330 (1) A pkg/kamailio/rhel/6 (1) A pkg/kamailio/rhel/7 (1) -- Patch Links -- https://github.com/kamailio/kamailio/pull/1244.patch https://github.com/kamailio/kamailio/pull/1244.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/1244

6 years, 7 months

[kamailio/kamailio] pkg/kamailio/alpine: Set of updates alpine packaging and docker (#1259)

by sergey-safarov

#### Pre-Submission Checklist - [x] Commit message has the format required by CONTRIBUTING guide - [ ] Commits are split per component (core, individual modules, libs, utils, ...) - [ ] Each component has a single commit (if not, squash them into one commit) - [ ] No commits to README files for modules (changes must be done to docbook files in `doc/` subfolder, the README file is autogenerated) #### Type Of Change - [x] Small bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds new functionality) - [ ] Breaking change (fix or feature that would change existing functionality) #### Checklist: - [ ] PR should be backported to stable branches - [x] Tested changes locally - [ ] Related to issue #XXXX (replace XXXX with an open issue number) #### Description Changes is documented in in commit messages You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1259 -- Commit Summary -- * pkg/kamailio/alpine: Removed custom kamailio.cfg, now used default * pkg/kamailio/alpine_docker: Allow build docker image from forked repos * pkg/kamailio/alpine_docker: Fixed missed symbolic links -- File Changes -- M pkg/kamailio/alpine/APKBUILD (13) D pkg/kamailio/alpine/kamailio.cfg (106) M pkg/kamailio/alpine_docker/build.sh (10) -- Patch Links -- https://github.com/kamailio/kamailio/pull/1259.patch https://github.com/kamailio/kamailio/pull/1259.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/1259

6 years, 7 months

[kamailio/kamailio] memory configuration using environment vars (#1252)

by sergey-safarov

### Description Docker container at runtime is configured using environment vars. Kamailio now not supports configure memory using environment vars and need use workarroud like [entrypoint.sh](https://github.com/kamailio/kamailio/blob/master/pkg/kamaili…. I want remove this workarround but need to add ability to configure kamailio memory using vars `SHM_MEMORY` and `PKG_MEMORY` when kamailio started without `-m` and `-M` keys -- 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/1252

6 years, 7 months

git:master:c5ffc265: Merge pull request #1258 from sergey-safarov/alpine

by GitHub

Module: kamailio Branch: master Commit: c5ffc265cf83c7e329ee8d21c3bafba78b670c58 URL: https://github.com/kamailio/kamailio/commit/c5ffc265cf83c7e329ee8d21c3bafba… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: GitHub <noreply(a)github.com> Date: 2017-09-30T14:23:21+02:00 Merge pull request #1258 from sergey-safarov/alpine pkg/kamailio/alpine_docker: Docker args now passed to kamailio #1252 --- Modified: pkg/kamailio/alpine/kamailio.cfg Modified: pkg/kamailio/alpine_docker/Dockerfile Modified: pkg/kamailio/alpine_docker/README.md Modified: pkg/kamailio/alpine_docker/build.sh Removed: pkg/kamailio/alpine_docker/entrypoint.sh --- Diff: https://github.com/kamailio/kamailio/commit/c5ffc265cf83c7e329ee8d21c3bafba… Patch: https://github.com/kamailio/kamailio/commit/c5ffc265cf83c7e329ee8d21c3bafba… --- diff --git a/pkg/kamailio/alpine/kamailio.cfg b/pkg/kamailio/alpine/kamailio.cfg index a8ca9990e9..6f91c23492 100644 --- a/pkg/kamailio/alpine/kamailio.cfg +++ b/pkg/kamailio/alpine/kamailio.cfg @@ -28,7 +28,6 @@ loadmodule "rr.so" loadmodule "maxfwd.so" loadmodule "usrloc.so" loadmodule "registrar.so" -loadmodule "mi_fifo.so" loadmodule "pv.so" loadmodule "xlog.so" @@ -38,7 +37,6 @@ loadmodule "xlog.so" modparam ( "usrloc", "db_mode", 0 ) modparam ( "rr", "enable_full_lr", 1 ) -modparam ( "mi_fifo", "fifo_name", "/var/run/kamailio/kamailio_fifo") #-------------------------------------------------------- diff --git a/pkg/kamailio/alpine_docker/Dockerfile b/pkg/kamailio/alpine_docker/Dockerfile index 7b55b404ff..5933ce2993 100644 --- a/pkg/kamailio/alpine_docker/Dockerfile +++ b/pkg/kamailio/alpine_docker/Dockerfile @@ -1,6 +1,5 @@ FROM scratch ADD kamailio_img.tar.gz / -COPY entrypoint.sh / -ENTRYPOINT ["/entrypoint.sh"] +ENTRYPOINT ["kamailio", "-DD", "-E"] diff --git a/pkg/kamailio/alpine_docker/README.md b/pkg/kamailio/alpine_docker/README.md index c5c7e074e0..ffc1a67702 100644 --- a/pkg/kamailio/alpine_docker/README.md +++ b/pkg/kamailio/alpine_docker/README.md @@ -6,19 +6,13 @@ Size of container decreased to 50MB (23MB compressed) Significantly increased security - removed all libs except libc, busybox, tcpdump, dumpcap, kamailio and dependent libs. Docker container is created useing Alpine linux packaging -Used environment variables --------------------------- - -1. ```SHM_MEMORY``` - amount of shared memory to allocate for the running Kamailio server (in Mb), default value 64Mb; -2. ```PKG_MEMORY``` - amount of per-process (package) memory to allocate for Kamailio (in Mb), default value 8Mb - Usage container --------------- ```sh docker run --net=host --name kamailio \ -v /etc/kamailio/:/etc/kamailio \ - kamailio/kamailio + kamailio/kamailio -m 64 -M 8 ``` systemd unit file diff --git a/pkg/kamailio/alpine_docker/build.sh b/pkg/kamailio/alpine_docker/build.sh index 3d9c52a2c0..510325b199 100755 --- a/pkg/kamailio/alpine_docker/build.sh +++ b/pkg/kamailio/alpine_docker/build.sh @@ -119,7 +119,6 @@ make_image_tar() { cd $BUILD_ROOT tar xzf $TMP_TAR /bin/busybox --install -s bin - sed -i -e '/mi_fifo/d' etc/kamailio/kamailio.cfg tar czf /usr/src/kamailio/pkg/kamailio/alpine_docker/$IMG_TAR * } diff --git a/pkg/kamailio/alpine_docker/entrypoint.sh b/pkg/kamailio/alpine_docker/entrypoint.sh deleted file mode 100755 index 3720d06984..0000000000 --- a/pkg/kamailio/alpine_docker/entrypoint.sh +++ /dev/null @@ -1,14 +0,0 @@ -#!/bin/sh -# -# Created by Sergey Safarov <s.safarov(a)gmail.com> -# - -SHM_MEMORY=${SHM_MEMORY:-64} -PKG_MEMORY=${PKG_MEMORY:-8} -trap 'kill -SIGTERM "$pid"' SIGTERM - -/usr/sbin/kamailio -DD -E -m $SHM_MEMORY -M $PKG_MEMORY & -pid="$!" - -wait $pid -exit 0

6 years, 7 months

git:master:d98e7570: pkg/kamailio/alpine_docker: Docker args now passed to kamailio #1252

by Sergey Safarov

Module: kamailio Branch: master Commit: d98e7570c458fb4992c6e49395b099a09bb5e133 URL: https://github.com/kamailio/kamailio/commit/d98e7570c458fb4992c6e49395b099a… Author: Sergey Safarov <s.safarov(a)gmail.com> Committer: Sergey Safarov <s.safarov(a)gmail.com> Date: 2017-09-30T05:40:17-04:00 pkg/kamailio/alpine_docker: Docker args now passed to kamailio #1252 --- Modified: pkg/kamailio/alpine_docker/Dockerfile Modified: pkg/kamailio/alpine_docker/README.md Removed: pkg/kamailio/alpine_docker/entrypoint.sh --- Diff: https://github.com/kamailio/kamailio/commit/d98e7570c458fb4992c6e49395b099a… Patch: https://github.com/kamailio/kamailio/commit/d98e7570c458fb4992c6e49395b099a… --- diff --git a/pkg/kamailio/alpine_docker/Dockerfile b/pkg/kamailio/alpine_docker/Dockerfile index 7b55b404ff..5933ce2993 100644 --- a/pkg/kamailio/alpine_docker/Dockerfile +++ b/pkg/kamailio/alpine_docker/Dockerfile @@ -1,6 +1,5 @@ FROM scratch ADD kamailio_img.tar.gz / -COPY entrypoint.sh / -ENTRYPOINT ["/entrypoint.sh"] +ENTRYPOINT ["kamailio", "-DD", "-E"] diff --git a/pkg/kamailio/alpine_docker/README.md b/pkg/kamailio/alpine_docker/README.md index c5c7e074e0..ffc1a67702 100644 --- a/pkg/kamailio/alpine_docker/README.md +++ b/pkg/kamailio/alpine_docker/README.md @@ -6,19 +6,13 @@ Size of container decreased to 50MB (23MB compressed) Significantly increased security - removed all libs except libc, busybox, tcpdump, dumpcap, kamailio and dependent libs. Docker container is created useing Alpine linux packaging -Used environment variables --------------------------- - -1. ```SHM_MEMORY``` - amount of shared memory to allocate for the running Kamailio server (in Mb), default value 64Mb; -2. ```PKG_MEMORY``` - amount of per-process (package) memory to allocate for Kamailio (in Mb), default value 8Mb - Usage container --------------- ```sh docker run --net=host --name kamailio \ -v /etc/kamailio/:/etc/kamailio \ - kamailio/kamailio + kamailio/kamailio -m 64 -M 8 ``` systemd unit file diff --git a/pkg/kamailio/alpine_docker/entrypoint.sh b/pkg/kamailio/alpine_docker/entrypoint.sh deleted file mode 100755 index 3720d06984..0000000000 --- a/pkg/kamailio/alpine_docker/entrypoint.sh +++ /dev/null @@ -1,14 +0,0 @@ -#!/bin/sh -# -# Created by Sergey Safarov <s.safarov(a)gmail.com> -# - -SHM_MEMORY=${SHM_MEMORY:-64} -PKG_MEMORY=${PKG_MEMORY:-8} -trap 'kill -SIGTERM "$pid"' SIGTERM - -/usr/sbin/kamailio -DD -E -m $SHM_MEMORY -M $PKG_MEMORY & -pid="$!" - -wait $pid -exit 0

6 years, 7 months

git:master:6e809f61: pkg/kamailio/alpine removed mi_fifo from config file.

by Sergey Safarov

Module: kamailio Branch: master Commit: 6e809f6126f9da45c1c12488b68f342a14260086 URL: https://github.com/kamailio/kamailio/commit/6e809f6126f9da45c1c12488b68f342… Author: Sergey Safarov <s.safarov(a)gmail.com> Committer: Sergey Safarov <s.safarov(a)gmail.com> Date: 2017-09-30T05:19:55-04:00 pkg/kamailio/alpine removed mi_fifo from config file. pkg/kamailio/alpine_docker: Removed workaround for non existent module mi_fifo --- Modified: pkg/kamailio/alpine/kamailio.cfg Modified: pkg/kamailio/alpine_docker/build.sh --- Diff: https://github.com/kamailio/kamailio/commit/6e809f6126f9da45c1c12488b68f342… Patch: https://github.com/kamailio/kamailio/commit/6e809f6126f9da45c1c12488b68f342… --- diff --git a/pkg/kamailio/alpine/kamailio.cfg b/pkg/kamailio/alpine/kamailio.cfg index a8ca9990e9..6f91c23492 100644 --- a/pkg/kamailio/alpine/kamailio.cfg +++ b/pkg/kamailio/alpine/kamailio.cfg @@ -28,7 +28,6 @@ loadmodule "rr.so" loadmodule "maxfwd.so" loadmodule "usrloc.so" loadmodule "registrar.so" -loadmodule "mi_fifo.so" loadmodule "pv.so" loadmodule "xlog.so" @@ -38,7 +37,6 @@ loadmodule "xlog.so" modparam ( "usrloc", "db_mode", 0 ) modparam ( "rr", "enable_full_lr", 1 ) -modparam ( "mi_fifo", "fifo_name", "/var/run/kamailio/kamailio_fifo") #-------------------------------------------------------- diff --git a/pkg/kamailio/alpine_docker/build.sh b/pkg/kamailio/alpine_docker/build.sh index 3d9c52a2c0..510325b199 100755 --- a/pkg/kamailio/alpine_docker/build.sh +++ b/pkg/kamailio/alpine_docker/build.sh @@ -119,7 +119,6 @@ make_image_tar() { cd $BUILD_ROOT tar xzf $TMP_TAR /bin/busybox --install -s bin - sed -i -e '/mi_fifo/d' etc/kamailio/kamailio.cfg tar czf /usr/src/kamailio/pkg/kamailio/alpine_docker/$IMG_TAR * }

6 years, 7 months

[kamailio/kamailio] pkg/kamailio/alpine_docker: Docker args now passed to kamailio #1252 (#1258)

by sergey-safarov

You can view, comment on, or merge this pull request online at: https://github.com/kamailio/kamailio/pull/1258 -- Commit Summary -- * pkg/kamailio/alpine removed mi_fifo from config file. * pkg/kamailio/alpine_docker: Docker args now passed to kamailio #1252 -- File Changes -- M pkg/kamailio/alpine/kamailio.cfg (2) M pkg/kamailio/alpine_docker/Dockerfile (2) M pkg/kamailio/alpine_docker/README.md (8) M pkg/kamailio/alpine_docker/build.sh (1) D pkg/kamailio/alpine_docker/entrypoint.sh (14) -- Patch Links -- https://github.com/kamailio/kamailio/pull/1258.patch https://github.com/kamailio/kamailio/pull/1258.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/1258

6 years, 7 months

git:master:563004ca: auth_radius: added section ids in docs

by Daniel-Constantin Mierla

Module: kamailio Branch: master Commit: 563004ca54cb31d19e3bd9e30633c9f860070883 URL: https://github.com/kamailio/kamailio/commit/563004ca54cb31d19e3bd9e30633c9f… Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Committer: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: 2017-09-30T09:23:34+02:00 auth_radius: added section ids in docs --- Modified: src/modules/auth_radius/doc/auth_radius_admin.xml --- Diff: https://github.com/kamailio/kamailio/commit/563004ca54cb31d19e3bd9e30633c9f… Patch: https://github.com/kamailio/kamailio/commit/563004ca54cb31d19e3bd9e30633c9f…

6 years, 7 months

[kamailio/kamailio] Problem with process limit of count wtimer secondary process (#1237)

by Julien Chavanton

### Description Kamailio won't start after pulling from master Commit triggering the problem ``` git show ec15b23f25ba2502a661157ae69b4dc1db923f66 Author: Daniel-Constantin Mierla <miconda(a)gmail.com> Date: Fri Sep 1 12:00:06 2017 +0200 core: main - don't count wtimer secondary process separately - it is registered via sr_wtimer_init() - fixes the extra empty entry in process table reported by Juha Heinanen ``` #### Reproduction Always with specific configuration (I can diagnose further is needed) #### Log Messages When trying to start Kamailio ``` 0(26583) CRITICAL: <core> [core/pt.c:285]: fork_process(): Process limit of 13 exceeded. Will simulate fork fail. 0(26583) CRITICAL: <core> [main.c:1720]: main_loop(): cannot fork tcp main process: Invalid argument ``` ### Possible Solutions Not sure is the config I am using is not supported or if Kamailio code should be fixed. rolling back the commit is fixing the issue `git revert ec15b23f25ba2502a661157ae69b4dc1db923f66` -- 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/1237

6 years, 7 months

← Newer
1
...
5
6
7
8
9
10
11
...
28
Older →

Jump to page:

2024

2023

2022

2021

2020

2019

2018

2017

2016

2015

2014

2013

2012

2011

2010

2009

2008

sr-dev September 2017