[Asterisk-code-review] taskprocessor: Enable subsystems and overload by subsystem (asterisk[13])
George Joseph
asteriskteam at digium.com
Fri Feb 15 13:19:07 CST 2019
Hello Friendly Automation,
I'd like you to reexamine a change. Please visit
https://gerrit.asterisk.org/11001
to look at the new patch set (#2).
Change subject: taskprocessor: Enable subsystems and overload by subsystem
......................................................................
taskprocessor: Enable subsystems and overload by subsystem
To prevent one subsystem's taskprocessors from causing others
to stall, new capabilities have been added to taskprocessors.
* Any taskprocessor name that has a '/' will have the part
before the '/' saved as its "subsystem".
Examples:
"sorcery/acl-0000006a" and "sorcery/aor-00000019"
will be grouped to subsystem "sorcery".
"pjsip/distributor-00000025" and "pjsip/distributor-00000026"
will bn grouped to subsystem "pjsip".
Taskprocessors with no '/' have an empty subsystem.
The "core show taskprocessors" CLI command now shows the subsystem.
* When a taskprocessor enters high-water alert status and it
has a non-empty system, the subsystem name will be marked as
"overloaded".
* When a taskprocessor leaves high-water alert status and it
has a non-empty system, the subsystem name will be unmarked as
"overloaded".
* A new api ast_taskprocessor_is_subsystem_overloaded() has been
added that checks if a specific subsystem is overloaded.
* The pjsip distributor now uses
ast_taskprocessor_is_subsystem_overloaded("pjsip") to determine
if it should pause accepting new messages.
REMINDER: The taskprocessor code doesn't take any action itself based
on high-water alerts or overloading. It's up to taskprocessor users
to check and take action themselves. Currently only the pjsip
distributor does this.
Change-Id: I8c19068bb2fc26610a9f0b8624bdf577a04fcd56
---
M include/asterisk/taskprocessor.h
M main/stasis.c
M main/taskprocessor.c
M res/res_pjsip.c
M res/res_pjsip/pjsip_distributor.c
5 files changed, 99 insertions(+), 13 deletions(-)
git pull ssh://gerrit.asterisk.org:29418/asterisk refs/changes/01/11001/2
--
To view, visit https://gerrit.asterisk.org/11001
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings
Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: newpatchset
Gerrit-Change-Id: I8c19068bb2fc26610a9f0b8624bdf577a04fcd56
Gerrit-Change-Number: 11001
Gerrit-PatchSet: 2
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Friendly Automation (1000185)
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20190215/c035d130/attachment.html>
More information about the asterisk-code-review
mailing list