Commit Graph

36 Commits

Author SHA1 Message Date
Clark Boylan ae52920b72 Use opendev git url
We want to get the source from the source.

Change-Id: I2cff44ca067ef0b9c342d9d60161b762cd10e57f
2019-04-22 08:21:47 -07:00
OpenDev Sysadmins 387d4102a1 OpenDev Migration Patch
This commit was bulk generated and pushed by the OpenDev sysadmins
as a part of the Git hosting and code review systems migration
detailed in these mailing list posts:

http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003603.html
http://lists.openstack.org/pipermail/openstack-discuss/2019-April/004920.html

Attempts have been made to correct repository namespaces and
hostnames based on simple pattern matching, but it's possible some
were updated incorrectly or missed entirely. Please reach out to us
via the contact information listed at https://opendev.org/ with any
questions you may have.
2019-04-19 19:26:09 +00:00
Ian Wienand 1ab3c2ad81 Replace openstack.org git:// URLs with https://
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: Iae9cb7f3fdd8fb5c0bcb1112aa4eca9f3430193f
2019-03-24 20:35:45 +00:00
Zuul d3528346d4 Merge "Don't create reviewday subdirectory" 2018-08-01 23:17:15 +00:00
Zuul fe55026ce8 Merge "Add beaker tests" 2018-08-01 23:16:08 +00:00
Zuul f639f9fe4a Merge "Fix exec idempotency" 2018-08-01 23:15:51 +00:00
Colleen Murphy 790bf6a783 Don't create reviewday subdirectory
Perhaps because of the new resource ordering algorithm, on puppet 4 the
/var/lib/reviewday/reviewday file resource blocks the
/var/lib/reviewday/reviewday vcsrepo resource and results in failures
like:

 Error: /Stage[main]/Main/Reviewday::Site[reviewday]/Vcsrepo[/var/lib/reviewday/reviewday]/ensure: change from absent to latest failed: Could not create repository (non-repository at path)

On both puppet 3 and puppet 4 the vcsrepo resource will create the
directory on its own, and it can also manage the directory's owner and
group itself, so there is no need for it to be managed independently.

Change-Id: Ic64d39a9f257d623ca13e9d38708eaa1e32e8c18
2018-07-19 20:12:25 +02:00
Colleen Murphy 3a042122ec Add beaker tests
Add basic tests to ensure the manifests can be applied cleanly and the
site comes up.

Change-Id: Id477b53849f6efd9715f89ca54eaa899fe08c90f
2018-07-12 11:34:11 +02:00
Colleen Murphy dd41a34c67 Update Gemfile for Zuulv3
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: Ib85eddeda99d5fc2422a8b7b994cb79e835d1f4a
2018-07-12 09:57:45 +02:00
Colleen Murphy 3d9839bfa4 Fix exec idempotency
The install-reviewday-dependencies exec doesn't need to be executed on
every puppet run, only when the source repo is updated, so add a
refreshonly => true to ensure it only reacts to changes in the source
repository.

Change-Id: Id3a747e49416398e3fdde9aeb843370890dbe572
2018-07-11 22:58:07 +02:00
Colleen Murphy cb806fc80b Depend on helper gem for spec_helper_acceptance
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: I11122f08a677a581941b86b6368600e7c09fa4e2
2017-08-18 10:41:44 +02:00
Colleen Murphy 1f329da154 Fix beaker on xenial
Add a xenial nodeset and update the spec helper to install puppet 3 from
the Ubuntu repos instead of from puppetlabs.

Change-Id: Ia336c416340d5034e3760e4d7eeb5b5be3763385
2017-06-24 16:31:49 +02:00
Spencer Krum 8bd8585606 Use new infra_spec_helper for gem dependencies
Change-Id: I1197843a3a081c3650260ecdd814e9b7a18a8af8
2016-06-21 18:40:03 -07:00
Andreas Jaeger a0dd0a2468 Add /usr/local/bin to path for cron job
gerrit-dash-creator is used by reviewday and installed in
/usr/local/bin. Set the path up correctly.

Change-Id: I39c8ad1ebc5971eec7ed70bd975051296738774d
2016-04-21 18:56:56 +02:00
Andreas Jaeger b02d9846ca Create logs for update_review
The cron job looks failing, put the output to a log file.
Setup log file path and logrotate for it.

Change-Id: Ie26e5a9f07ae122842338cf833d4462a9ce7559f
2016-04-20 19:54:08 +02:00
Spencer Krum b5aa6662cb Pin google-api-client; sanitize Gemfile
Change-Id: Id47bc79f471c66254d0042921034e07213dc6513
2016-04-14 15:23:29 -07:00
Jenkins 96d01e8a31 Merge "Order of the class parameters is refactored" 2016-04-13 18:13:45 +00:00
Yolanda Robla 8eb5a620b8 Fix path for reviewday dependencies
Currently puppet is failing because it cannot find pip.
That is because the exec path is set to reviewday directory.
Add /usr/local/bin and /bin as valid directories in the path
as well.

Change-Id: I3dc47004f1ace03ffa78fabdc13158dbcc16bb28
2016-04-13 12:34:09 +02:00
rossella dee4cb5373 Run pip install to install the dependencies
Id2e8590193fbf21a52a767323a77df9be4048b5a requires
gerrit-dash-creator to be installed on the host where
reviewday runs. This patch runs pip install to install
all the dependencies.

Change-Id: I42ea25bdf69a1d75db24785db0e86396206fd48d
2016-04-05 10:50:29 +02:00
Andrey Nikitin be98d090a2 Order of the class parameters is refactored
Order and indentation 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: I005c80760893c1d0023104832af436ac3890e7b3
2016-03-22 12:44:59 +03:00
Spencer Krum 5796e7f8bd Fix target path for regular git clone during tests
Use same target directory for zuul-cloner and
the regular git command.

Change-Id: I60222d61f5a15236668449c719dca896fad4cce4
Co-Authored-By: Fabien Boucher <fabien.boucher@enovance.com>
2015-08-19 16:41:43 -07:00
Colleen Murphy e0cf7c4d0b Add Gemfile and puppet 4 checks
In anticipation of puppet 4, start trying to deal with puppet 4 things
that can be helpfully predicted by puppet lint plugins. Also fix arrow
alignment errors caught by the bundle-installed puppet-lint.

Change-Id: I060c65c6ca9292216620a851d402359c33ca9927
2015-08-13 17:56:04 -07:00
Colleen Murphy c006a4e12e Set reasonable defaults for reviewday::site
Empty string is not an appropriate default value for parameters that
are used unconditionally in the puppet manifest or template. For
example, the name of the http root directory cannot be an empty string,
so it is not sensible to use empty string as a default. This patch
attempts to set reasonable defaults for every parameter. For the key
parameters, we change the default from empty string to undef and update
the conditional logic related to those parameters.

Change-Id: I6d2e7037385a4082726e5e7db93b18a92b4a8ace
2015-08-13 17:56:00 -07:00
Colleen Murphy b445a0968c Fix template variables
Using ruby methods to access manifest variables is deprecated in puppet
3 and will be removed in puppet 4. This patch fixes the templates to
use ruby instance variables instead.

Change-Id: I7f9890379b3b79b947ddbf3e8c3fb129a0f04bb9
2015-08-13 17:55:52 -07:00
Spencer Krum fd5702d378 Boilerplate beaker-rspec files
Change-Id: I061de37e033c5eb4dd3a4e86a55aba09f3c731f3
2015-07-28 02:05:44 -07:00
Jeremy Stanley 367240c5c2 Replace ci.o.o links with docs.o.o/infra
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: Ie3cfbcd08863fbe1a51549885290ffaddf228e6a
2015-05-14 21:38:18 +00:00
Ramy Asselin ede467374b Rename openstackci to openstackinfra
Change-Id: If6ebb81e27af25841121071a510298eb316e2b40
2015-04-20 13:50:17 -07:00
Ramy Asselin e0ce6ad459 Add puppet module files for puppet-reviewday
Change-Id: I168abe9e1e0e274250ebcccdb144759c8a0058d4
2015-01-29 14:01:20 -08:00
Elizabeth Krumbach 899250f862 Force reviewday ssh to gerrit to use ipv4
Running into issues running reviewday over ipv6 on static
server, updated ssh config forcing it to use only ipv4 for
ssh for now.

Change-Id: I79e0b9271ff64e09bd41b1f1fbaf18eb3f90a1c4
Reviewed-on: https://review.openstack.org/29029
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
2013-05-14 02:02:12 +00:00
Jeremy Stanley 54e3aaf95f Decrease reviewday update frequency.
* modules/reviewday/manifests/site.pp: The frequency of the update
reviewday cron job is now every half hour instead of every quarter
hour. Additionally, a lock is added to prevent overlapping runs in
case the script begins to take longer. This is needed since recent
Launchpad API changes have caused it to perform considerably slower
than before.

Change-Id: I9fcdbb350a61ab29200068406ccb8b8eaa5e4244
Reviewed-on: https://review.openstack.org/28331
Reviewed-by: Dan Prince <dprince@redhat.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
2013-05-10 16:22:28 +00:00
Elizabeth Krumbach cafa6ead92 Add gerrit url to reviewday .ssh/known_hosts file
Change-Id: I01a753ecdbd0fcb22cd26ac173f1a37eff35def1
Reviewed-on: https://review.openstack.org/25567
Reviewed-by: Dan Prince <dprince@redhat.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Reviewed-by: Khai Do <zaro0508@gmail.com>
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
2013-03-28 16:50:51 +00:00
Elizabeth Krumbach 6562b81129 Keep reviewday repo up to date on filesystem
Changes to vcsrepo stanza in reviewday puppet module to keep the
reviewday checkout up to date with changes to master.

Change-Id: I0cbe2c06622545948c9b645c45eac19887b4c440
Reviewed-on: https://review.openstack.org/25555
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
2013-03-27 20:14:56 +00:00
Elizabeth Krumbach b054f34b27 Fix permissions on reviewday directories
Reviewday home and git repository directories were owned by root,
fix those permissions and make http directory executable.

Change-Id: I3ae856a3c1ffa13d9f8a7217284d89d8035f2613
Reviewed-on: https://review.openstack.org/25424
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Approved: James E. Blair <corvus@inaugust.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
2013-03-27 16:47:46 +00:00
James E. Blair 1ed42f36b1 Fix reviewday errors.
Change-Id: I702bdd6845df2cd6e9b8c464b95cca50fb783875
Reviewed-on: https://review.openstack.org/25418
Reviewed-by: Elizabeth Krumbach <lyz@princessleia.com>
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
2013-03-26 17:47:44 +00:00
Elizabeth Krumbach 21dca835a2 Refactor reviewday into class and definition
The reviewday puppet files have been split between static class and a
definition that pulls several variables for openstack-specific use.

Change-Id: Ibeb450900a75269c4c8a5aaaa51fb105aff7af15
Reviewed-on: https://review.openstack.org/24915
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Approved: James E. Blair <corvus@inaugust.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
2013-03-26 17:34:08 +00:00
Elizabeth Krumbach 398173dd29 Add puppet files for reviewday
Initial add of puppet files so reviewday can be deployed on the
static webserver.

Fixes: bug #1082785
Change-Id: Ie5516e82bfc9dfea95b53285c46aa881d5c05f32
Reviewed-on: https://review.openstack.org/21158
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
2013-03-19 19:43:15 +00:00