@bogdan-iancu, here it is, i'm back with proposed commits and changes:
1. get rid of load_balance() in favor of lb_*() family of functions to make the
work-flow more clear and defined:
lb_start(grp, rl, flgs) - strictly used to start LB session. if sessions
already started, old session will be lost and re-started again.
lb_next() - strictly used to to continue LB session previously started
by lb_start().
lb_is_started() - used to check whenever LB session is already started
or not.
lb_reset() - used to clean up LB session and everything LB internally
does for this dialog.
ln_disable_dst() - mark destination currently chosen by LB in current
active LB session as disabled.
lb_start_and_next(grp, rl, flgs) - is simply a shorthand for
'lb_is_started()?lb_next():lb_start()' to simplify scripting.
this function logically almost the same as old load_balance()
function,
so we could leave old name for compatibility reasons.
lb_count_call(......) - manually register call as a load for particular
resource(es).
2. lb_start(..., alg) parameter changed to 'flags' parameter with possible
values:
'r' - use relative versus absolute estimation
'n' - do not skip negative loads
3. new module parameter 'probing_verbose' to log INFO message whenever we
enable/disable destinations by probing/MI.
Could you please review it and give a feedback.
---
Reply to this email directly or view it on GitHub:
https://github.com/OpenSIPS/opensips/pull/345#issuecomment-64216763
_______________________________________________
Devel mailing list
Devel@lists.opensips.org
http://lists.opensips.org/cgi-bin/mailman/listinfo/devel