mbox series

[0/2] patchtest-send-results: prevent false positives due to commit subject

Message ID 20240205155730.115895-1-alexis.lothore@bootlin.com
Headers show
Series patchtest-send-results: prevent false positives due to commit subject | expand

Message

Alexis Lothoré Feb. 5, 2024, 3:57 p.m. UTC
Hello,
this small series aims to prevent patchtest-send-results from sending
reports to submitters in a specific case: if the commit message
contains the word "FAIL", it will (wrongly) send a report to submitter
despite all tests passing / being skipped.

The first patch is a trivial cleanup while the second patch brings in the
real fix.

Alexis Lothoré (2):
  patchtest-send-results: remove unused variable
  patchtest-send-results: properly parse test status

 scripts/patchtest-send-results | 6 ++++--
 1 file changed, 4 insertions(+), 2 deletions(-)

Comments

Trevor Gamblin Feb. 6, 2024, 2:18 p.m. UTC | #1
On 2024-02-05 10:57, Alexis Lothoré via lists.openembedded.org wrote:
> Hello,
> this small series aims to prevent patchtest-send-results from sending
> reports to submitters in a specific case: if the commit message
> contains the word "FAIL", it will (wrongly) send a report to submitter
> despite all tests passing / being skipped.
>
> The first patch is a trivial cleanup while the second patch brings in the
> real fix.
Sorry for the late reply. Thanks for sending these out, nice catch!
>
> Alexis Lothoré (2):
>    patchtest-send-results: remove unused variable
>    patchtest-send-results: properly parse test status
>
>   scripts/patchtest-send-results | 6 ++++--
>   1 file changed, 4 insertions(+), 2 deletions(-)
>
>
> -=-=-=-=-=-=-=-=-=-=-=-
> Links: You receive all messages sent to this group.
> View/Reply Online (#194952): https://lists.openembedded.org/g/openembedded-core/message/194952
> Mute This Topic: https://lists.openembedded.org/mt/104178720/7611679
> Group Owner: openembedded-core+owner@lists.openembedded.org
> Unsubscribe: https://lists.openembedded.org/g/openembedded-core/unsub [tgamblin@baylibre.com]
> -=-=-=-=-=-=-=-=-=-=-=-
>