The opendev project has been moving away from puppet and this is one of
the puppet modules that is no longer used. To simplify things for us we
are taking the extra step of retiring this repo.
Change-Id: I577721e214940a0c25c1a22bd98adc6de83c8b56
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: I809974db704f559e7de255479246ed1ab4243c34
With all our channels mode +r these days, the first few the meetbot
attempts to join are rejected while NickServ is still chewing on the
identify command. Use the supybot configuration option to delay
joining until we get identification confirmed.
Change-Id: I0631cfbba46dfe3c23ff5e455f2ac941b5655c28
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: I3d9b01fe25acb5887859fa65d8ca309ce4c965c8
Use the new $content parameter of httpd::vhost instead of $template.
This way, the template gets rendered within the scope where it assumes
most of its variables are, and doesn't need to use the scope object
which doesn't work within a defined type. This will ensure the template
keeps working on puppet 4. This also requires adjusting some of the
template parameters that had previously been correctly scoped from the
httpd::vhost defined type.
Depends-On: https://review.openstack.org/570824
Change-Id: I3a45e99e3355d1e59d57d3d989cc03d253529bf2
Add functional tests to ensure that the meetbot module can be applied
without errors, and that the service is accessible. It also validates
that the vhost template renders as expected, since this tends to be one
of the harder parts to get right in module development.
Change-Id: I812a7503d0fda4823bbc0bb7b268f8287b2def8b
In the httpd module, serveraliases defaults to undef, which evaluates to
nil in ERB templates. Ensure the template checks for nil values so that
the ServerAlias line is only included if there is a server alias.
Also correct the variable usage in the index.html.erb template.
Change-Id: Ibde097b7dd407884336154a318ec4c5661fda03c
Without this patch, with puppet 3, if the $manage_index variable is set,
it is ignored, since the conditional compares the string 'manage_index'
to the value 'true'. With puppet 4, this would cause a syntax error.
Change-Id: If9dc13515059a2c148ddde3108b256feb3bc1298
Using local ruby variables in puppet ERB templates was deprecated in
puppet 3 and doesn't work any more in puppet 4. Access these values as
ruby instance variables instead.
Change-Id: I957a5f1c679f0a2172d1a1b88040ae91e239e627
Looks like the location of plugins directory has changed in
ubuntu-xenial. Update manifiest to support both xenial and trusty.
Update service to subscribe to /opt/meetbot this way we don't need to
know which plugins directory is needed when on xenial or trusty.
Change-Id: I70a06093bbb1119aed52627f2a4bf1d07b74cd5b
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
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: I26872912f8ff42227f0d88130807f35d900d1209
Add a xenial nodeset and update the spec helper to install puppet 3 from
the Ubuntu repos instead of from puppetlabs.
Change-Id: Ic67cccd07fba170edf43f71e844703f0b388c171
Use same target directory for zuul-cloner and
the regular git command.
Change-Id: I0db9ab98cff46147ddb186a99c7129bfb47c8212
Co-Authored-By: Fabien Boucher <fabien.boucher@enovance.com>
Now that puppet-httpd has been fully migrated too, we can remove this
obsolete logic.
Change-Id: I7d796a2808891fb686a753d3148204e305b88606
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
In anticipation of puppet 4, start trying to deal with puppet 4 things
that can be helpfully predicted by puppet lint plugins. This patch also
corrects lint errors caught by the puppet-lint-empty-check gem.
The $vhost_extra default was changed from empty string to undef, which
will be evaluated as nil in the template and will result in the same
behavior (printing nothing) if left unset.
Change-Id: I31588fb2d4c1894ad9785cd001100a83f4736f2f
puppet-httpd is the openstack-infra version of puppetlabs-apache
(0.0.4) release.
This patchset will remove the puppetlabs-apache namespace from -infra
allowing for possible future patchsets to use newer puppetlabs-apache
modules.
Change-Id: I39d337bd09ede21289be67c3c2666aa543e3ec3e
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
We want to install our own index page, so make the management of
the index.html file optional, defaulting to true.
Change-Id: I09fd41a186eb5edf9368974415cf555c22bbf9d0
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: I365d37b2a23e75b190add49eab0b1745c41d6e7e
There is a tool, irclog2html, that knows how to take directories of irc
logs and turn them in to formatted html with per-line links. Since we do
that for build logs, and the logs associated with meetings, it seems
only sane that we do it for channel logs.
The tool groks mtime, so it will skip previously produced logs, and will
overwrite current streaming logs each time it's run. We'll need to run
the command one time by hand first, because the first run might take a
bit to eat through our history.
Change-Id: Ieed2c80d945ee35b29887d672cd9ebbd9ece2188
undefined method 'latest' for Puppet::Type::Vcsrepo::ProviderBzr
The module documentation seems to use "present" everywhere instead.
Change-Id: Ia6e116b65b7a8b25c40fd289dc3172f5abcf2da7
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
Add the Bugtracker plugin to the supybot used for meetbot (OpenStack).
This will restore the response of bug information to the channel when
a user references a bug in a message (e.g. 'bug XXXXXX').
Change-Id: I3ee3f1fa516bf1607ba6e79c6f911190a49b0161
We have a cgit server now, which means we should replace
all references to github with references to git.openstack.org.
Change-Id: I68ad1ce514fb4326c7d9940b5a84999af5b58562
* modules/meetbot/manifests/site.pp: Specify a nondefault vhost
template.
* modules/meetbot/templates/vhost.erb: New file, similar in content
to the default puppetlabs-apache vhost template but forcing MIME
type text/plain and a default encoding of UTF-8 for files ending in
a .log suffix.
Change-Id: Ief0253c2baac32b126472d0f97f0aa503adbbd7f
There have been general requests for more logging of more things. I
think we might have a longer discussion there. But recently there have
been specific requests about -infra. I think that since discussions
around the gate and rather wide-spread project concerns wind up
happening in infra, we should definitely start logging it, and then we
can talk about whether other team channels should be logged or not.
Made this use an array of channels rather than a string for sanity.
Change-Id: I3f3e82aa86155d1ac3010ca3c3ade006b7a26793