New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pkg/kamailio: Fix of ticket 1085 #1086
Conversation
This commit message is not describing properly what it adds or fixes. You can add a reference to a bug report, but the commit must have proper description text of what it does. The commit messages are used to generate the changelog files for release. You have to update the commit message in order to get the pull request merged. |
a8c35c3
to
c691ceb
Compare
@miconda PR updated |
@linuxmaniac - are you ok with the changes proposed for deb packaging regarding the systemd? |
See my comment at #1085 (comment) |
What happens if you have a non-working config and you have |
@linuxmaniac yes, unit will be restarted infinity, Rate of restarts may limiter using "RestartSec" keyword |
@linuxmaniac we can also mark exit code for broken config as SuccessExitStatus. Than unit will not restart in case of broken config |
@linuxmaniac changes is not required.
|
Any resolution on this? Is it breaking things and should be discarded or not? |
@linuxmaniac - any spare time soon to review and decide what to do with this PR? |
I still don't see why we need this. What is the case of "non null exit"? |
When kamailio cannot allocate memory (no free memeory on host), then we got non null exit code. |
I am going to close this. Being interpretable, one can also do local chances of the systemd confs. @linuxmaniac being the official maintainer of Kamailio debs inside Debian process has the final decision on maintaining those specs. @sergey-safarov you can make a patch only for rpm specs and can be considered as a dedicated pull request. |
@linuxmaniac |
No description provided.