[kamailio/kamailio] drouting: use keepalive to monitor GW/destinations (sort_order 0 only) (#1066)
If keepalive is enabled and sort_order is 0, unavailable destinations are excluded from do_routing() resultset. You can view, comment on, or merge this pull request online at:
https://github.com/kamailio/kamailio/pull/1066
-- Commit Summary --
* drouting: use keepalive to monitor GW/destinations (sort_order 0 only)
-- File Changes --
M src/modules/drouting/README (38) M src/modules/drouting/doc/drouting_admin.xml (24) M src/modules/drouting/dr_load.c (3) M src/modules/drouting/drouting.c (61) M src/modules/drouting/prefix_tree.h (4) M src/modules/drouting/routing.c (2)
-- Patch Links --
https://github.com/kamailio/kamailio/pull/1066.patch https://github.com/kamailio/kamailio/pull/1066.diff
As a remark for the future: do not generate the README file and push it to GIT repository for exiting modules. The README is generated on server to get a coherent formatting. You can do it locally to check that changes to the docbook xml files are ok, then revert README before making the commit/pull request. You can use:
``` # generate readme locally make modules-readme modules=src/modules/mymodules # see the diffs of the readme git diff src/modules/mymodule/README # if all ok, revert it - system remove and git check out rm src/modules/mymodule/README git checkout src/modules/mymodule/README # commit the changes and push git commit src/modules/mymodule/ git push origin master:master ```
For this PR I can merge manually, but for the future it is easier to have the PR without the patch to README>
Merged manually.
Closed #1066.
thanks @miconda, I didn't know README file was autogenerated. I'll do better next PR :)
-
Daniel-Constantin Mierla -
Guillaume Bour