Alexandros Kosiaris has submitted this change and it was merged. ( https://gerrit.wikimedia.org/r/402101 )
Change subject: ircecho: Force unbuffered stdin/stdout/stderr ...................................................................... ircecho: Force unbuffered stdin/stdout/stderr stdin/stdout/stderr are effectively used by ircecho for debugging/logging messages. However these are not thread safe and ircecho uses threads. The result is usually lost logs that await in the buffer to be flushed and make debugging difficult. Pass -u to python and change from using the env variable since /usr/bin/env can't really handle arguments While the better solution would be to rewrite parts of the software to use the logging module, this is deemed enough for now Bug: T184103 Change-Id: I9a7b596725bb25be63ea3b6326ab014354ff6575 --- M modules/ircecho/files/ircecho.py 1 file changed, 1 insertion(+), 1 deletion(-) Approvals: Alexandros Kosiaris: Looks good to me, approved jenkins-bot: Verified Volans: Looks good to me, but someone else must approve diff --git a/modules/ircecho/files/ircecho.py b/modules/ircecho/files/ircecho.py index bb06ae6..c7c94ea 100755 --- a/modules/ircecho/files/ircecho.py +++ b/modules/ircecho/files/ircecho.py @@ -1,4 +1,4 @@ -#!/usr/bin/env python +#!/usr/bin/python -u # vim: set tabstop=4 shiftwidth=4 softtabstop=4 expandtab textwidth=80 smarttab # # stdin -> IRC echo bot, with optional file input support. -- To view, visit https://gerrit.wikimedia.org/r/402101 To unsubscribe, visit https://gerrit.wikimedia.org/r/settings Gerrit-MessageType: merged Gerrit-Change-Id: I9a7b596725bb25be63ea3b6326ab014354ff6575 Gerrit-PatchSet: 4 Gerrit-Project: operations/puppet Gerrit-Branch: production Gerrit-Owner: Alexandros Kosiaris <[email protected]> Gerrit-Reviewer: Alexandros Kosiaris <[email protected]> Gerrit-Reviewer: Volans <[email protected]> Gerrit-Reviewer: jenkins-bot <> _______________________________________________ MediaWiki-commits mailing list [email protected] https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
