From 8ac77c4a1c9cf62fe7d9e4c4a7dd50fbf1500a0b Mon Sep 17 00:00:00 2001 From: Patrick Donnelly Date: Mon, 6 Jun 2022 14:12:09 -0400 Subject: [PATCH] qa: correct line processing logic Most runs don't use --no-nested-subset and for those that used --subset, the if conditions would correctly pickup "seed" (when it mattered). However, when --subset was not specified in the original run, the "seed" was not correctly picked up. Therefore, the result of inserting the "if no_nested_subset is None:" before the "elif seed is None:" caused it to never read the seed for most folks teuthology runs. Signed-off-by: Patrick Donnelly --- teuthology/report.py | 4 +--- 1 file changed, 1 insertion(+), 3 deletions(-) diff --git a/teuthology/report.py b/teuthology/report.py index b75d0ad76..d7375f361 100644 --- a/teuthology/report.py +++ b/teuthology/report.py @@ -413,10 +413,8 @@ class ResultsReporter(object): subset = self._parse_log_line(line, 'subset:') if no_nested_subset is None: no_nested_subset = self._parse_log_line(line, 'no_nested_subset:') - elif seed is None: + if seed is None: seed = self._parse_log_line(line, 'seed:') - else: - break if subset is not None: subset = tuple(int(i) for i in subset.split('/')) if no_nested_subset is not None: -- 2.47.3