| Message ID | 20240216210028.16028-1-simone.p.weiss@posteo.com |
|---|---|
| State | Accepted, archived |
| Commit | 661c0a8fd8fe7bff61cea82778c25cf24d791267 |
| Headers | show |
| Series | patchtest: add stronger indication for failed tests | expand |
Please ignore this, v2 is already sent... On Fri, 2024-02-16 at 21:00 +0000, simone.p.weiss@posteo.com wrote: > From: Simone Weiß <simone.p.weiss@posteo.com> > > Do not only log that there has been an issue but add WARNING before for > local > runs. Hopefully this helps to avoid that people reading the log to > quickly miss > issues. > > Fixes [YOCTO #15389] > > Signed-off-by: Simone Weiß <simone.p.weiss@posteo.com> > > sfdf > > Signed-off-by: Simone Weiß <simone.p.weiss@posteo.com> > --- > scripts/patchtest | 4 ++-- > 1 file changed, 2 insertions(+), 2 deletions(-) > > diff --git a/scripts/patchtest b/scripts/patchtest > index d0dc6e8fac..3163420220 100755 > --- a/scripts/patchtest > +++ b/scripts/patchtest > @@ -164,9 +164,9 @@ def run(patch, logfile=None): > if premerge_result == 2 and postmerge_result == 2: > logger.error('patchtest: No test cases found - did you specify > the correct suite directory?') > if premerge_result == 1 or postmerge_result == 1: > - logger.error('patchtest: At least one patchtest caused a > failure or an error - please check') > + logger.error('WARNING: patchtest: At least one patchtest caused > a failure or an error - please check') > else: > - logger.error('patchtest: All patchtests passed') > + logger.error('OK: patchtest: All patchtests passed') > print('------------------------------------------------------------ > ----------\n') > return premerge_result or postmerge_result >
diff --git a/scripts/patchtest b/scripts/patchtest index d0dc6e8fac..3163420220 100755 --- a/scripts/patchtest +++ b/scripts/patchtest @@ -164,9 +164,9 @@ def run(patch, logfile=None): if premerge_result == 2 and postmerge_result == 2: logger.error('patchtest: No test cases found - did you specify the correct suite directory?') if premerge_result == 1 or postmerge_result == 1: - logger.error('patchtest: At least one patchtest caused a failure or an error - please check') + logger.error('WARNING: patchtest: At least one patchtest caused a failure or an error - please check') else: - logger.error('patchtest: All patchtests passed') + logger.error('OK: patchtest: All patchtests passed') print('----------------------------------------------------------------------\n') return premerge_result or postmerge_result