Hello Corentin,

I'm enable to reproduce the crash manually (using send_mail directly). I guess that's a Django bug because send_mail is supposed to raise only smtplib.SMTPException (see https://docs.djangoproject.com/en/1.11/topics/email/#send-mail).
I will see if I can protect lava-logs against such issue.
In fact crashing lava-logs is always an issue.


Thanks

Le jeu. 6 déc. 2018 à 10:22, LABBE Corentin <clabbe@baylibre.com> a écrit :
On Thu, Dec 06, 2018 at 09:34:43AM +0100, Remi Duraffort wrote:
> Hello,
>
> which version of django are you using and which settings have you updated
> to fix the crash?
>
>

dpkg -l |grep django
ii  python-django-common                 1:1.10.7-2+deb9u3                        all          High-level Python web development framework (common)
ii  python3-django                       1:1.10.7-2+deb9u3                        all          High-level Python web development framework (Python 3 version)
ii  python3-django-auth-ldap             1.3.0-1~bpo9+1                           all          Django LDAP authentication backend (Python3 version)
ii  python3-django-extensions            1.7.4-1                                  all          Useful extensions for Django projects (Python 3 version)
ii  python3-django-restricted-resource   2016.8-1                                 all          Django Base model for ownership and access control (Python 3)
ii  python3-django-tables2               1.14.2-1                                 all          Table/data-grid framework for Django
ii  python3-django-testproject           0.1.3-1                                  all          Django test project support (Python 3 version)
ii  python3-django-testscenarios         0.9-1                                    all          Django unit test scenarios support (Python 3)
ii  python3-pytest-django                2.9.1-3                                  all          Django plugin for py.test.

I have simply added an email address to the admin user.

> Thanks
>
> Le mer. 5 déc. 2018 à 11:36, LABBE Corentin <clabbe@baylibre.com> a écrit :
>
> > On Wed, Dec 05, 2018 at 10:56:28AM +0100, Remi Duraffort wrote:
> > > Hello Corentin,
> > >
> > > thanks for the crash report.
> > >
> > > lava-logs is crashing but the code responsible for the crash in the
> > > notification (in lava_scheduler_app/notifications.py).
> > >
> > > Could you look at the logs and try to find a line like "[<job_id>]
> > sending
> > > email notification to <email>" in /var/log/lava-server/django.log
> > >
> > > I'm guessing that recipient.email_address is strange.
> > >
> >
> > I found the reason, admin does not have an email address.
> > But that should not crash like that.
> >
> > Regards
> >
>
>
> --
> Rémi Duraffort
> LAVA Team


--
Rémi Duraffort
LAVA Team