The script will exit non-zero if any of a set of tests fail, but it
isn't necessarily obvious from the output what was a warning and what
caused the failure. Let's make this obvious in the output. In all cases
where we set a future failure, explicitly tell the caller via stdout.
The official importer instance (the only known instance) had been at
100% phasing of main for months, but accidentally reverted to the old
default of 1% as the configuration was lost when it was converted to a
proper service.
Let's move the default to match the production instance to reduce
the likelyhood of mistakes.
Log when emails are sent, not when they're not sent
mailer.notify() queues up package results to send email summary
notifications after some interval has passed. It returns True when that
email actually is sent, False if no email needed sent, and throws
exceptions on actual email service faults.
This warning is thus incorrect; the false return doesn't indicate a
failure, just that it wasn't time to send an email. And in fact it's
probably not interesting to log anyway. Instead, log when emails *are*
sent.
systemd: Drop explicit setting of StandardOutput to journal
With StandardOutput left undefined, the system's setting (from
/etc/systemd/system.conf) is used instead, which defaults to
journal already. There is no particular reason to override the
system settings.
