Andrew Bogott has uploaded a new change for review.

  https://gerrit.wikimedia.org/r/69496


Change subject: Fix string continuation and a few pep8 issues.
......................................................................

Fix string continuation and a few pep8 issues.

I'm not sure this file even parsed before, and I don't know how
it got past jenkins.  It's presence was causing pep8 tests to
fail for any subsequent patches.

Change-Id: If0ed48a8899bc7a59b8fbf3dea42ade5daa3c5f6
---
M files/mysql/skrillex.py
1 file changed, 10 insertions(+), 8 deletions(-)


  git pull ssh://gerrit.wikimedia.org:29418/operations/puppet 
refs/changes/96/69496/1

diff --git a/files/mysql/skrillex.py b/files/mysql/skrillex.py
index 7b26a0c..cb0aab9 100755
--- a/files/mysql/skrillex.py
+++ b/files/mysql/skrillex.py
@@ -15,18 +15,20 @@
 
     parser = argparse.ArgumentParser(
         description='skrillex [options] db::group  \"some sql;\"',
-        epilog='group must be one of the following:
-        all::all : all sanitarium and all labsdb instances
-        sanitarium::all : all sanitarium instances
-        labsdb::all : all labsdb instances
-        sanitarium::SHARD : specific sanitarium instance
-        labsdb::SHARD : specific labsdb instance')
+        epilog='group must be one of the following: '
+        'all::all : all sanitarium and all labsdb instances '
+        'sanitarium::all : all sanitarium instances '
+        'labsdb::all : all labsdb instances '
+        'sanitarium::SHARD : specific sanitarium instance '
+        'labsdb::SHARD : specific labsdb instance')
     parser.add_argument('-c', '--config',
-                        help='Specify yaml config file. Default 
/etc/skrill.yaml')
+                        help='Specify yaml config file. '
+                             'Default /etc/skrill.yaml')
     parser.add_argument('-q', '--query',
                         help='Specify a MySQL query to run', required=True)
     parser.add_argument('-g', '--group',
-                        help='Specify a group of mysql instances on which to 
run a query',
+                        help='Specify a group of mysql instances '
+                             'on which to run a query',
                         required=True)
     parser.add_argument('-a', '--forall', action='store_true',
                         help='Run on all databases in the MySQL instance')

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

Gerrit-MessageType: newchange
Gerrit-Change-Id: If0ed48a8899bc7a59b8fbf3dea42ade5daa3c5f6
Gerrit-PatchSet: 1
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Andrew Bogott <abog...@wikimedia.org>

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

Reply via email to