This is a mechanically generated change to replace openstack.org
git:// URLs with https:// equivalents.
This is in aid of a planned future move of the git hosting
infrastructure to a self-hosted instance of gitea (https://gitea.io),
which does not support the git wire protocol at this stage.
This update should result in no functional change.
For more information see the thread at
http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003825.html
Change-Id: I6e3a2df384bd3b0a0106a0971cd1a74c396b8dff
The logic in the Gemfile was relying on Zuulv2 variables to find out
whether the spec helper gem was already available on disk, and since
Zuulv3 has changed things it was failing to find it and downloading the
master version instead. This patch ensures the Gemfile looks for the gem
in the right place when running in CI.
Change-Id: I673fc19ab3d254c3a58b7a250d0877aac4838f71
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
We want Mailman routers to be last, and they are, but I'd like to
replace the built-in ones in this module with user-supplied routers
instead, so move the user-supplied routers to the end.
Change-Id: I090070308785d4b55bac12806a05e2896306c0a3
Story: 2001382
Task: 6091
In an effort to thwart recent spates of spam to -owner addresses of
mailing lists, place the mailman_router last so that system_aliases
get expanded before it. This allows for tricks like aliasing a
particular -owner address to :blackhole: temporarily as a
workaround.
Change-Id: Ibea05369bb5e42bf637f84251a387ffd81577373
Instead of keeping a local copy of spec_helper_acceptance.rb and
requiring updates to all modules for any change, we can move it into the
common helper gem and require it from there. This will make it easier to
create and review changes that affect all puppet modules. Also change
the Gemfile to look for the gem in the local workspace if running in a
zuul environment.
Change-Id: Ic60a4664b47f1f3d37e3dc8831ef620aa128db0a
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
Bindep is a tool for checking the presence of binary packages needed
to use an application / library. It started life as a way to make it
easier to set up a development environment for OpenStack projects.
Change-Id: I88b77c8c34163bc11dbcb823dc1b57b0340f3193
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
The newline on the local_domains list was being trimmed causing it
to be combined with the next line.
Change-Id: I1a4798b7ce8ecdd1a45945b97455dc4ef0c01702
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
In general the options for routers and transports are not sensitive
to order, except that private options for drivers must succeed the
driver option. Therefore, it is customary to put the driver option
first.
To ensure this, we special case the driver option in the template
expansion.
Also, for the remaining options, put them in sorted order so that
the config files do not needlessly change on puppet runs.
Change-Id: Id5c21aeaede4b5d780403b00eeac4f2947d36ffe
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
Use same target directory for zuul-cloner and
the regular git command.
Change-Id: I1a3179befebcc28f95a86b7c70745e5a7e9f12cd
Co-Authored-By: Fabien Boucher <fabien.boucher@enovance.com>
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 http://ci.openstack.org/ documentation site has been deprecated,
replaced by redirects to corresponding paths within
http://docs.openstack.org/infra/ where other Project Infrastructure
documentation already resides.
Change-Id: Ie6f1879f5a48db22b55fcbead03f7ceeb4372e59
The content of this project is Apache 2 licensed, but we should
include a standard LICENSE file just to be clear about that.
Change-Id: Iee6320b9d7e35fbe8d3b0a9794f3e485c18ef2c8
Non instance variable representation is deprecated
so needs to be changed. This change changes varibles
to their instance variable representation.
See more details see:
http://docs.puppetlabs.com/guides/templating.html
Change-Id: Ib77827e01011ef6c0380c9ec7a9d147eafd8ce2f
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
If someone copies the production Zuul configuration, they may end
up sending emails as jenkins@o.o. Configure our hosts to only
accept email with an envelope sender or From header of jenkins@.o.o
if the connecting host is zuul.openstack.org (or localhost, to handle
the case of exim running on zuul.o.o itself).
Change-Id: Ic92bb39659689347fea1744ee8c1b47ef21858b0
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
Change-Id: I7b5998f50d22e103981b04706060f5127a256123
Reviewed-on: https://review.openstack.org/14506
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Approved: Clark Boylan <clark.boylan@gmail.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
The local hostname is included in local_domains by default,
but in case we're hosting mailman domains other than the local
hostname, include them.
Change-Id: I733b8fe8cacc2cfd2d7a0d33236ed9e2273c15da