[Asterisk-code-review] testsuite: avoid hang on waitfullybooted (testsuite[certified/18.9])

Friendly Automation asteriskteam at digium.com
Thu Sep 8 11:37:57 CDT 2022


Friendly Automation has submitted this change. ( https://gerrit.asterisk.org/c/testsuite/+/19133 )

Change subject: testsuite: avoid hang on waitfullybooted
......................................................................

testsuite: avoid hang on waitfullybooted

Python 3.6 compatibility change

ASTERISK-30208

Change-Id: Ieba157910e9c7422f3c54f4d31a89a4e99ec3c84
(cherry picked from commit a43658375c8d3c7994ecff39c870e44f31b6331c)
---
M lib/python/asterisk/asterisk.py
1 file changed, 17 insertions(+), 1 deletion(-)

Approvals:
  Friendly Automation: Looks good to me, approved; Approved for Submit




diff --git a/lib/python/asterisk/asterisk.py b/lib/python/asterisk/asterisk.py
index 66c1124..1c0edfd 100644
--- a/lib/python/asterisk/asterisk.py
+++ b/lib/python/asterisk/asterisk.py
@@ -23,6 +23,8 @@
 
 from .config import ConfigFile
 
+from subprocess import PIPE
+
 from twisted.internet import reactor, protocol, defer, utils, error
 from twisted.python.failure import Failure
 
@@ -963,7 +965,7 @@
             completed = subprocess.run([self.ast_binary,
                 "-C", "%s" % os.path.join(self.astetcdir, "asterisk.conf"),
                 "-rx", "%s" % cli_cmd],
-                capture_output=True, encoding='utf-8')
+                stdout=PIPE, stderr=PIPE, encoding='utf-8')
 
             if responsekey not in completed.stdout:
                 CLIRetVal = 2

-- 
To view, visit https://gerrit.asterisk.org/c/testsuite/+/19133
To unsubscribe, or for help writing mail filters, visit https://gerrit.asterisk.org/settings

Gerrit-Project: testsuite
Gerrit-Branch: certified/18.9
Gerrit-Change-Id: Ieba157910e9c7422f3c54f4d31a89a4e99ec3c84
Gerrit-Change-Number: 19133
Gerrit-PatchSet: 1
Gerrit-Owner: George Joseph <gjoseph at digium.com>
Gerrit-Reviewer: Friendly Automation
Gerrit-CC: Michael Bradeen <mbradeen at sangoma.com>
Gerrit-MessageType: merged
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.digium.com/pipermail/asterisk-code-review/attachments/20220908/cec17544/attachment.html>


More information about the asterisk-code-review mailing list