Ori.livneh has submitted this change and it was merged.

Change subject: beta: fix ansi escapes for wmf-beta-autoupdater
......................................................................


beta: fix ansi escapes for wmf-beta-autoupdater

* the ANSI escape to reset color was wrong. Fixed it up.
* changed colors scheme entirely
* created entry for level CRITICAL, although it is unused.

Change-Id: I8423bd8e9c072fb8b55622be20b34686e6ca0651
---
M modules/beta/templates/wmf-beta-autoupdate.py.erb
1 file changed, 11 insertions(+), 8 deletions(-)

Approvals:
  Ori.livneh: Looks good to me, approved
  jenkins-bot: Verified



diff --git a/modules/beta/templates/wmf-beta-autoupdate.py.erb 
b/modules/beta/templates/wmf-beta-autoupdate.py.erb
index a7fe331..1cbd6b4 100755
--- a/modules/beta/templates/wmf-beta-autoupdate.py.erb
+++ b/modules/beta/templates/wmf-beta-autoupdate.py.erb
@@ -35,18 +35,21 @@
     options = parse_args()
     logging.basicConfig(level=options.log_level)
     # Color codes http://www.tldp.org/HOWTO/Bash-Prompt-HOWTO/x329.html
-    logging.addLevelName(
-        logging.DEBUG, "\033[0;37m%s\033[1;m" %
+    logging.addLevelName(  # cyan
+        logging.DEBUG, "\033[36m%s\033[0m" %
         logging.getLevelName(logging.DEBUG))
-    logging.addLevelName(
-        logging.INFO, "\033[1;33m%s\033[1;m" %
+    logging.addLevelName(  # green
+        logging.INFO, "\033[32m%s\033[0m" %
         logging.getLevelName(logging.INFO))
-    logging.addLevelName(
-        logging.WARNING, "\033[1;31m%s\033[1;m" %
+    logging.addLevelName(  # yellow
+        logging.WARNING, "\033[33m%s\033[0m" %
         logging.getLevelName(logging.WARNING))
-    logging.addLevelName(
-        logging.ERROR, "\033[1;41m%s\033[1;m" %
+    logging.addLevelName(  # red
+        logging.ERROR, "\033[31m%s\033[0m" %
         logging.getLevelName(logging.ERROR))
+    logging.addLevelName(  # red background
+        logging.CRITICAL, "\033[41m%s\033[0m" %
+        logging.getLevelName(logging.CRITICAL))
 
     logger = logging.getLogger('main')
 

-- 
To view, visit https://gerrit.wikimedia.org/r/152063
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings

Gerrit-MessageType: merged
Gerrit-Change-Id: I8423bd8e9c072fb8b55622be20b34686e6ca0651
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Hashar <has...@free.fr>
Gerrit-Reviewer: BryanDavis <bda...@wikimedia.org>
Gerrit-Reviewer: Hashar <has...@free.fr>
Gerrit-Reviewer: Ori.livneh <o...@wikimedia.org>
Gerrit-Reviewer: jenkins-bot <>

_______________________________________________
MediaWiki-commits mailing list
MediaWiki-commits@lists.wikimedia.org
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits

Reply via email to