Project

General

Profile

Feature #9470

Option to send Puppet error emails for all hosts to a user

Added by Andy Taylor almost 6 years ago. Updated over 2 years ago.

Status:
Closed
Priority:
Normal
Assignee:
Category:
E-Mail
Target version:
Difficulty:
Triaged:
Bugzilla link:

Description

The new email features in 1.7.x are a welcome addition, but they cause a fairly serious issue for me - I no longer receive all the Puppet errror reports. The new approach means that only owners of VMs receive the error emails, which in my environment is developers. I don't want them to receive the notifications, but I do want our Operations team to receive them.

Could an option be added under 'My account > Mail Preferences' to receive all Puppet error reports? Or alternatively have another option in the 'Puppet error state' dropdown so the list of options would look like this:

No emails
Subscribe to my hosts
Subscribe to all hosts

Thanks,

Andy


Related issues

Related to Foreman - Tracker #7585: Mail notifications trackerNew

Related to Foreman - Bug #15247: toplevel constant PuppetError referenced by MailNotification::PuppetErrorClosed2016-05-31

Associated revisions

Revision 55e7865d (diff)
Added by Amir Fefer almost 5 years ago

Fixes #9470 - Adds an option to send puppet error emails for all host to a user

Adds an option to the puppet error state notification - Subscribe to all hosts, when error occurs from any authorize host, the user will get an email

Revision f329b160 (diff)
Added by Dominic Cleal almost 5 years ago

refs #9470 - set :limit on column, fix MailNotification initializer

MailNotification.new failed unless it had a Hash argument, hence the
factory had been changed from default behaviour. It also then ignored
all attributes except a given few listed in initialize_with.

History

#1 Updated by Dominic Cleal almost 6 years ago

#2 Updated by Amir Fefer about 5 years ago

  • Assignee set to Amir Fefer

#3 Updated by The Foreman Bot about 5 years ago

  • Status changed from New to Ready For Testing
  • Pull request https://github.com/theforeman/foreman/pull/2923 added

#4 Updated by Amir Fefer almost 5 years ago

  • Status changed from Ready For Testing to Closed
  • % Done changed from 0 to 100

#5 Updated by The Foreman Bot almost 5 years ago

  • Pull request https://github.com/theforeman/foreman/pull/3306 added

#6 Updated by Dominic Cleal almost 5 years ago

  • Legacy Backlogs Release (now unused) set to 136

#7 Updated by Dominic Cleal over 4 years ago

  • Related to Bug #15247: toplevel constant PuppetError referenced by MailNotification::PuppetError added

Also available in: Atom PDF