[Asterisk-code-review] channel: Use LOG WARNING instead of debug in CHECK BLOCKING. (asterisk[13])

[Corey Farrell]

Corey Farrell has posted comments on this change. ( https://gerrit.asterisk.org/8075 )

Change subject: channel: Use LOG_WARNING instead of debug in CHECK_BLOCKING.
......................................................................


Patch Set 2:

> Have you done any investigation into the case that occurred under
 > the testsuite execution? We should ensure we understand fix those
 > cases before this is merged in, or else we will get issue reports
 > when people upgrade if it occurs under normal cases.

Good point (and no I have not).  I'm going to abandon this for now. 
It's obviously not causing major issues and I don't think it'll have simple fixes (or even be simple to triage).  One of the failures is in autoservice_run which is run in a thread started by ast_autoservice_start (the starter thread).  Bridging and ARI are also problems.

Maybe I'll dig deeper at some point in the future but it's unlikely to happen before the release of Asterisk 16.


-- 
To view, visit https://gerrit.asterisk.org/8075
To unsubscribe, visit https://gerrit.asterisk.org/settings

Gerrit-Project: asterisk
Gerrit-Branch: 13
Gerrit-MessageType: comment
Gerrit-Change-Id: Ia6cacc9c862f2653d1b650511725e53e48e8e653
Gerrit-Change-Number: 8075
Gerrit-PatchSet: 2
Gerrit-Owner: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: Corey Farrell <git at cfware.com>
Gerrit-Reviewer: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Jenkins2
Gerrit-Reviewer: Joshua Colp <jcolp at digium.com>
Gerrit-Comment-Date: Wed, 31 Jan 2018 16:53:52 +0000
Gerrit-HasComments: No
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20180131/d6bdc990/attachment.html>