diff options
author | Alexis Lothoré <alexis.lothore@bootlin.com> | 2024-02-05 16:57:30 +0100 |
---|---|---|
committer | Richard Purdie <richard.purdie@linuxfoundation.org> | 2024-02-06 10:31:25 +0000 |
commit | 3567c21af8ed65448f9325ee3fe85b8be839e1b5 (patch) | |
tree | 00c63e8d244302b37fe5d7cecfaf4b91cc6b091e /meta/lib/oeqa | |
parent | c2ba125dc30fb1ef0bf96152863db22159f4b31c (diff) | |
download | openembedded-core-3567c21af8ed65448f9325ee3fe85b8be839e1b5.tar.gz |
patchtest-send-results: properly parse test status
patchtest-send-results currently search the word "FAIL" in the whole
testresults file to decide whether it should send a report to patch
submitter or not. This global search can lead to false positives, for
example if the commit subject contains the word "FAIL" (as observed in
[1])
Prevent those false positives by explicitely parsing the test status from
each line. Each test result line, generated by the patchtest script, is
expected to have the following format:
<STATUS>: <some info, depending on the status>
[1] https://lore.kernel.org/openembedded-core/0101018d79bfe020-06f2ce89-ea19-456b-92e7-66ee1c710fd1-000000@us-west-2.amazonses.com/
Signed-off-by: Alexis Lothoré <alexis.lothore@bootlin.com>
Signed-off-by: Richard Purdie <richard.purdie@linuxfoundation.org>
Diffstat (limited to 'meta/lib/oeqa')
0 files changed, 0 insertions, 0 deletions