I'm talking about the exception that is thrown from our mailers, for example:

app/mailers/host_mailer.rb

In the first line of #summary you can see this line:

raise ::Foreman::Exception.new(N_("Must specify a user with email enabled")) unless (user=User.find(options[:user])) && user.mail_enabled?

It's an exception, so it will be risen as a warning (will be shown in production).
As far as I looked in the code of the rake task mechanism, there is no filtering of mailers based on #mail_enabled field. The only filtering is based on frequency, so if the frequency was once set, the code shown above would run, and if the #mail_enabled property is unset - it will raise an exception.

I can confirm, i see in my production logs:

Failed to send email notification puppet_error_state: ERF42-5670 [Foreman::Exception]: unable to find recipients

since this is repeating per client with errors, it might be occurring in the logs too often, but I'm not sure if we should remove it?

Ohad Levy wrote:

I can confirm, i see in my production logs:

Failed to send email notification puppet_error_state: ERF42-5670 [Foreman::Exception]: unable to find recipients

Oh, and this is a slightly different condition (it's an alert, not a summary), but #9873 will fix that in the way it refactors alerting to groups.