Skip to content

Commit

Permalink
XDPBenchMeasurement: rename loop variable
Browse files Browse the repository at this point in the history
The 'result' name conflicts with the 'result' from the main for loop,
this overrides the value of the 'result' name which is used a few lines
lower to calculate measurement_success.

Signed-off-by: Ondrej Lichtner <[email protected]>
  • Loading branch information
olichtne committed Nov 26, 2024
1 parent aadca4c commit 5870448
Showing 1 changed file with 2 additions and 2 deletions.
4 changes: 2 additions & 2 deletions lnst/RecipeCommon/Perf/Measurements/XDPBenchMeasurement.py
Original file line number Diff line number Diff line change
Expand Up @@ -218,8 +218,8 @@ def report_results(cls, recipe: BaseRecipe, results: list[AggregatedXDPBenchMeas

recipe_result = ResultType.PASS
metrics = {"Generator": generator, "Receiver": receiver}
for name, result in metrics.items():
if cls._invalid_flow_duration(result):
for name, metric_result in metrics.items():
if cls._invalid_flow_duration(metric_result):
recipe_result = ResultType.FAIL
desc.append("{} has invalid duration!".format(name))

Expand Down

0 comments on commit 5870448

Please sign in to comment.