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 <mail@gmelikov.ru> Reviewed-by: Ryan Moeller <ryan@iXsystems.com> Reviewed-by: John Kennedy <john.kennedy@delphix.com> Signed-off-by: Brian Behlendorf <behlendorf1@llnl.gov> Closes #10323
This commit is contained in:
parent
e1fcd940e7
commit
c87f958668
|
@ -272,8 +272,8 @@ def process_results(pathname):
|
||||||
pattern_log = r'^\s*Log directory:\s*(\S*)'
|
pattern_log = r'^\s*Log directory:\s*(\S*)'
|
||||||
|
|
||||||
d = {}
|
d = {}
|
||||||
for l in f.readlines():
|
for line in f.readlines():
|
||||||
m = re.match(pattern, l)
|
m = re.match(pattern, line)
|
||||||
if m and len(m.groups()) == 4:
|
if m and len(m.groups()) == 4:
|
||||||
summary['total'] += 1
|
summary['total'] += 1
|
||||||
if m.group(4) == "PASS":
|
if m.group(4) == "PASS":
|
||||||
|
@ -281,7 +281,7 @@ def process_results(pathname):
|
||||||
d[m.group(1)] = m.group(4)
|
d[m.group(1)] = m.group(4)
|
||||||
continue
|
continue
|
||||||
|
|
||||||
m = re.match(pattern_log, l)
|
m = re.match(pattern_log, line)
|
||||||
if m:
|
if m:
|
||||||
summary['logfile'] = m.group(1)
|
summary['logfile'] = m.group(1)
|
||||||
|
|
||||||
|
|
Loading…
Reference in New Issue