From 005709066bf8086ca50607e2d378ae71779dc8e2 Mon Sep 17 00:00:00 2001 From: Akash Vacher Date: Mon, 9 Jul 2018 10:07:57 -0700 Subject: [PATCH] Fixed flake8 errors for kafka/tools/assigner/actions/demote.py The flake8 errors were part of the reason why Travis build was failing for PR #91 for linkedin/kafka-tools. The other factor was tox running into an upstream python bug [1] for which PR #94 has been raised. I confirmed that after these 2 factors were eliminated, the build passes [2]. [1] https://bugs.python.org/issue10496 [2] https://github.com/akashvacher/kafka-tools/commit/2228e65f43ce86b28bae8b32e35ad703d54a18a0 --- kafka/tools/assigner/actions/demote.py | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/kafka/tools/assigner/actions/demote.py b/kafka/tools/assigner/actions/demote.py index 3edf92c..a0782a8 100644 --- a/kafka/tools/assigner/actions/demote.py +++ b/kafka/tools/assigner/actions/demote.py @@ -31,9 +31,8 @@ def __init__(self, args, cluster): def _add_args(cls, parser): parser.add_argument( '-b', '--brokers', type=int, help='List of broker ids to demote', required=True, nargs='*') - parser.add_argument('-t', '--topics', required=False, nargs='*', help=\ - 'Optional list of topics to use. If not specified, leadership of all topics lead by the brokers to ' \ - 'demote will be changed.') + parser.add_argument('-t', '--topics', required=False, nargs='*', + help='Optional list of topics to use. If not specified, leadership of all topics lead by the brokers to demote will be changed.') def process_cluster(self): brokers_to_demote = set(self.args.brokers)