From c87f9586687d53bc7c3f2e8887841267a3960269 Mon Sep 17 00:00:00 2001 From: Brian Behlendorf Date: Thu, 14 May 2020 09:41:29 -0700 Subject: [PATCH] flake8 E741 variable name warning Update the zts-report.py script to conform to the flake8 E741 rule. "Variables named I, O, and l can be very hard to read. This is because the letter I and the letter l are easily confused, and the letter O and the number 0 can be easily confused." - https://www.flake8rules.com/rules/E741.html Reviewed-by: George Melikov Reviewed-by: Ryan Moeller Reviewed-by: John Kennedy Signed-off-by: Brian Behlendorf Closes #10323 --- tests/test-runner/bin/zts-report.py | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/tests/test-runner/bin/zts-report.py b/tests/test-runner/bin/zts-report.py index d74aa9d7ae..5420ff9d29 100755 --- a/tests/test-runner/bin/zts-report.py +++ b/tests/test-runner/bin/zts-report.py @@ -272,8 +272,8 @@ def process_results(pathname): pattern_log = r'^\s*Log directory:\s*(\S*)' d = {} - for l in f.readlines(): - m = re.match(pattern, l) + for line in f.readlines(): + m = re.match(pattern, line) if m and len(m.groups()) == 4: summary['total'] += 1 if m.group(4) == "PASS": @@ -281,7 +281,7 @@ def process_results(pathname): d[m.group(1)] = m.group(4) continue - m = re.match(pattern_log, l) + m = re.match(pattern_log, line) if m: summary['logfile'] = m.group(1)