Under some circumstances it can be useful to not include the default
"localuser" Exim router. Make it optional via a
default_localuser_router class parameter which defaults to true,
preserving backward compatibility.
Change-Id: Iae38041c95bc0bbcd1dfd3d0c5c626b63acd7922
In an effort to support customizing E-mail aliases per server, allow
a hash of aliases to be passed into the /etc/aliases template. While
we're here, remove the custom gerrit2 and jenkins aliases since
those can now be set individually on servers where they actually
matter rather than unnecessarily setting them everywhere.
Change-Id: I2911f157812c127a514196ae58b7609378d7d4e4
Allow callers to optionally set smtp_accept_max and
smtp_accept_max_per_host options for tuning systems to handle higher
connection volumes.
Also fix the config template's conditional for queue_smtp_domains
since it defaults to undef but was previously checked against ""
instead.
Change-Id: I5519f6c28c81a6f40c78551ac318b91cacfc04fb
Order and intendation of those parameters are changed
to follow Puppet Style Guide recommendation [0].
Moreover, it will allow to an user to find much faster
a variable in a list of variables.
[0]. https://docs.puppetlabs.com/guides/style_guide.html
Change-Id: I5df9f22616afcc9b460334d0a74a7e54973297e9
Allow complete definitions of routers and transports to be provided
to the exim class which will then be added to the exim4.conf file.
Change-Id: Id4f69580eb126d058ee699f4e7bfdb01cfc7409c
In anticipation of puppet 4, start trying to deal with puppet 4 things
that can be helpfully predicted by puppet lint plugins.
Change-Id: Id272e95c5f93e07b5ab7791badc635eb066c6763
Co-Authored-By: Colleen Murphy <colleen@gazlene.net>
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
The rest of the tree, literally everwhere, calls this value
sysadmins - but the exim module calls it sysadmin. This doesn't
really matter of course, but I noticed it while looking in to a
review on another patch and couldn't not fix it. Also, there are
two comments that are clearly remnants from having used exim.pp
as a starting point.
Change-Id: I184701c8c4d4b1e9d93d746b2ad0c5fc53443866
This time, make the default value false instead of empty string.
This reverts commit 99d3283dc246da4b4d2d26ecfb193b308881f05d
Change-Id: I88108ff75f1c2bd3aa78856c186312340258ec3c
Up to 50 simultaneous queue runners
Run queues every minute
Also adds queue_smtp_domains as a parameter (unused currently).
Change-Id: I7d93131278e5d8677445e3230f68b2a81313246a
Clean up facter osfamily matches to just use Debian, not Ubuntu.
This is manually tested and confirmed to at least be the case on
Oneiric, Precise and Quantal.
Change-Id: I27b184ac419910f9c3271c3b4e57886333282a5f
Reviewed-on: https://review.openstack.org/27399
Reviewed-by: Spencer Krum <krum.spencer@gmail.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
The install scripts now look for CentOS in release files. Also some
instances of facter's operatingsystem are switched to osfamily and
capitalization of RedHat is normalized to match what facter uses.
Change-Id: I3bbca5481d0d5e6de9e62bfd6e2b0a85264ed6ed
Reviewed-on: https://review.openstack.org/27398
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
Add some requires to the module so that we ensure the
exim package is installed before configuring the etc config
file and starting the service.
Change-Id: I5c464a9b8b6ecbdea7af3a4b9bdf344f3a40c0db
Reviewed-on: https://review.openstack.org/23552
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
Parameterizes the exim module so that it can support multiple
distributions.
As part of this patch I also removed the explicit install
of exim4-base and exim4-config on Ubuntu. Just installing
exim4-daemon-light should be fine.
Change-Id: Ie09c5fa01509ebfadc92b8540f04181a2b13b667
Reviewed-on: https://review.openstack.org/23289
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Monty Taylor <mordred@inaugust.com>
Approved: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
More fixes in preparation for our gate-ci-puppet-lint job.
Change-Id: I1cd3499e6cd958fbbacd01f3daef581a7b4d3217
Signed-off-by: Paul Belanger <paul.belanger@polybeacon.com>
Reviewed-on: https://review.openstack.org/13764
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Approved: James E. Blair <corvus@inaugust.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
When we put these all in by default, then people testing things or people
who aren't even us get the mail on their systems configured to email us.
Change-Id: Ib9d4160e0c81151bf040422c89621bc0fcefcd66
Handles "Could not perform immediate configuration on 'exim4-daemon-light'".
Change-Id: I49049071af2f5f323ef90485252d825351367522
Reviewed-on: https://review.openstack.org/353
Reviewed-by: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
Add a basic exim config to all OpenStack servers, and send
mail to Jim and Monty.
Change-Id: I8011c839daa24f1ac169782f2de5b14c11b34eee
Reviewed-on: https://review.openstack.org/265
Reviewed-by: Monty Taylor <mordred@inaugust.com>
Tested-by: Jenkins
Monty Taylor