From: Patrick Donnelly Date: Mon, 6 Jun 2022 18:12:09 +0000 (-0400) Subject: qa: correct line processing logic X-Git-Tag: 1.2.0~165^2 X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F1762%2Fhead;p=teuthology.git 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 --- 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: