Skip to content

Commit cead054

Browse files
author
Thomas Preud'homme
committed
Fix use of undefined session variable
Commit eb0f1b5 introduced uses of variable session in lnt/server/reporting/summaryreport.py but one of them is in function _build_data_table() which does not have a session parameter. This commit adds the parameter and update call sites accordingly, fixing one of Flake8's F821 warning (undefined name). Reviewed By: tnfchris Differential Revision: https://reviews.llvm.org/D94837
1 parent 180e859 commit cead054

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

lnt/server/reporting/summaryreport.py

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -176,7 +176,7 @@ def should_be_in_report(machine):
176176
# <machine id>)
177177

178178
self.data_table = {}
179-
self._build_data_table()
179+
self._build_data_table(session)
180180

181181
# Compute indexed data table by applying the indexing functions.
182182
self._build_indexed_data_table()
@@ -187,7 +187,7 @@ def should_be_in_report(machine):
187187
# Build final organized data tables.
188188
self._build_final_data_tables()
189189

190-
def _build_data_table(self):
190+
def _build_data_table(self, session):
191191
def get_nts_datapoints_for_sample(ts, sample):
192192
# Get the basic sample info.
193193
run_id = sample[0]

0 commit comments

Comments
 (0)