Commit Graph

46 Commits

Author SHA1 Message Date
Monty Taylor a06b1ae633 Allow removing openstackwatch
We don't use it anymore and removing it would be nice.

Change-Id: If4781b2d57445150ab6fdd80222112ccba654f20
2018-02-02 11:06:15 +00:00
Paul Belanger dbfd559052 Revert "Remove python-six on centos"
It seems after 7.4 point release, python-six is used my other packages beside cloud-init (specifically libstoragemgmt). For now, revert so we can get puppet working again. Then start discussions on creating DIB images for control plane servers.

This reverts commit 5c956c5995.

Change-Id: Iff7723b24eaa9aed207efd330fc0ae756020a072
2018-01-08 16:34:00 +00:00
Monty Taylor 5c956c5995
Remove python-six on centos
This results in the removal of cloud-init, which is not used by anything
in Infra land. Should this cause further issues, this module should
be reworked to install jeepyb into a virtualenv instead.

Change-Id: I24b4ce816e442a273d244b6bcb66a29c198d7c86
2017-09-20 14:41:14 -05:00
Monty Taylor 387701d1dd
Don't remove PyYAML - because cloud-init
We can just not install it.

Change-Id: Ib9b4b842d7426c5f95b2347fbabb5e876c2808b9
2017-09-19 13:11:07 -05:00
Monty Taylor 604504fd98
Stop installing python depends from distro
Mixing distro and non-distro packages tends to equal unhappy.

Change-Id: I4fa88bdb10e8cb76694f4d04d3239b38f445868a
2017-09-19 12:09:40 -05:00
Colleen Murphy c40bb69d82 Properly escape % in cron command
crontab syntax requires % to be escaped, but puppet complains about '\%'
in strings, so we need to double escape it in puppet. This will result
in a single-escaped % in the crontab.

Change-Id: I2fcb3fca5b2d09e8b0a15036c8d4cbad1a83ba3a
2017-06-14 00:29:15 +02:00
Colleen Murphy 2a0cb0897f Fix cron command string
The % does not need to be escaped and trying to causes the syntax check
to fail.

Change-Id: I7bf80cbd68f96fac20c2bd81dea6a5692fd03cf4
2017-06-11 22:23:49 +02:00
Yolanda Robla 0f6076c432 Require packages before installing jeepyb
Currently the jeepyb install has a race. install_jeepyb is
called without requiring the needed packages. This causes
that puppet needs to be run at least twice to converge.

Adding the proper require on place to fix that problem,
also switching to virtual resources and require, to avoid
cross duplicates with other projects such as puppet-openstackci

Change-Id: I34864ed7f0ba991d48de80fe6d64eebcd3df300a
2016-08-12 17:47:05 +02:00
James E. Blair 6b1051fc6c Use libxslt1-dev rather than libxslt-dev
This is the actual package name.  Using the unversioned package
will cause puppet to attempt to install it each time.

Change-Id: I03850efb0f2b0cf3e00dbf906af97ca51b0840d5
2016-07-08 08:20:49 -07:00
Philip Marc Schwartz 9d391f24b4 Update pip provider to use openstack_pip provider
Change-Id: I0493df84bef696f31af075af5ceb7c47a09c57fa
Signed-off-by: Philip Marc Schwartz <philip@progmad.com>
2016-06-16 10:19:38 -04:00
Yolanda Robla 830295783d Fix package name on RedHat
The package installation was still incorrect for Redhat case

Change-Id: I2c21058857c4b471fe0b1d0bccb3cb19b3fb6e69
2016-06-09 15:27:21 +02:00
Yolanda Robla de2a976f98 Add missing packages to jeepyb install
The pip install currently failed because of missing
libffi-dev and libssl-dev/openssl-devel packages

Change-Id: I91b4851b58a41764289273d5b6b548e6dbcac3a4
2016-06-08 09:42:05 +02:00
Paul Belanger 0d881c04eb Revert "Add missing packages to jeepyb install"
This is currently failing on our git servers, which our centos 7.

This reverts commit eb53f6e4ea.

Change-Id: I739c9598021b3114e89b61edfce6e8a7a4954aac
2016-06-07 21:20:25 +00:00
Yolanda Robla eb53f6e4ea Add missing packages to jeepyb install
The pip install currently failed because of
missing libffi-dev and libssl-dev packages

Change-Id: If0964fc92656ab605fc05e7260dc8b230a071a7a
2016-06-06 16:54:10 +02:00
James E. Blair e4c352d286 Protect gcc package from duplicate declaration
Change-Id: Ida5333c8f5ef54c584ab0c48507d32a086a264ba
2016-04-13 16:11:34 -07:00
Andrey Nikitin eaac86d3f4 Order of the classes parameters is refactored
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: I6dbfc2fd3769b174e493bd0392a652eca160f2bd
2016-03-21 12:51:28 +03:00
Steve Kowalik 59bc52871b Install libxml2-dev{,el} and libxslt-dev{,el}
In preperation for moving the Zanata scripts from project-config into
jeepyb, lxml will be added to its requirement. This requires the
development libraries libxml2 and libxslt to be installed, so ensure
they are.

Change-Id: Id21eb3039f25e5c675674b5ca53179bd9d3c918f
2015-12-18 14:55:27 +11:00
Jenkins 4b3de83a24 Merge "Remove mysql::python class inclusion" 2015-11-25 00:03:14 +00:00
Paul Belanger e331b17e25 Upgrade all packages to the newest version
Change-Id: I44cd912a9b525f29bf4ec9cf85848d331ab693e4
Signed-off-by: Paul Belanger <pabelanger@redhat.com>
2015-11-13 11:02:48 -05:00
Colleen Murphy e14978a3c9 Remove mysql::python class inclusion
When the dependent patch is merged, Jeepyb will depend on PyMySql and
install it from pip as part of its requirements. Therefore it will no
longer be necessary to include the mysql::python class, which installs
the python-mysqldb package from Ubuntu packages. Removing this class
inclusion will make transitioning to an up-to-date version of the mysql
puppet module easier, as the current version moves the mysql::python
class to mysql::bindings::python.

Change-Id: I7dec1c6f588cb3e1d0eb3cb2eb6398df45ebdc22
Depends-On: I3afeaebeb8416388d5866fc3994357935079c97d
2015-10-14 14:10:21 -07:00
Colleen Murphy ce4349d156 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 lint
errors caught by the puppet-lint-absolute_classname-check gem as well
as arrow alignment errors not caught before.

Change-Id: I6ee8cc21247258d9a37ce3304c207c4b637378f7
2015-08-03 18:35:43 -07:00
Colleen Murphy 0563885683 Fix parameters for jeepyb::openstackwatch
The swift_username, swift_password, swift_auth_url, and json_url
parameters all must be specified in order to work correctly, a default
value of empty string will cause brokenness. This patch removes the
default values for those parameters so that puppet will warn the user
that these values are required.

For the auth_version parameter, a default value of 1.0 is a safe
potential guess, so we change the default value from empty string to
'1.0'.

The feed parameter is no longer used as of
fdc8fd45bb, so we remove it.

Change-Id: I0a10c0df80cb6a74e7bcc17d96e27e62373c61ff
Depends-On: Ie3c4f5184445b4653997c091279f21a3e9503b84
2015-08-03 18:35:30 -07:00
Jenkins 67c5b14de3 Merge "Add helpers to execute manage_projects" 2015-05-04 15:03:37 +00:00
Yolanda Robla e331c50cec Add helpers to execute manage_projects
Manage_projects is run in two different ways:
one in a cronjob, one due to subscription. Create
helper classes to isolate that behaviour, to move
logic from system-config.

Change-Id: I4132646446dfe575c333b981aea9a7778c13bcf0
2015-04-20 21:50:46 +01:00
K Jonathan Harker 39bb5b4d7b Parameterize git revision of jeepyb
Allow for the git revision to be set arbitrarily in addition to the source
URI.

Change-Id: I453dcb74b32939f76820736d1ccd064df8561ca2
2015-04-06 12:10:18 -07:00
Anita Kuno 88feb8d40f Add logging to manage-projects
This patch adds logging to the puppet exec for manage-projects
within gerrit.pp.

It also adds logging to jeepyb, which can also cover logging for
manage-projects, which is a script within jeepyb.

Change-Id: I4cd1137e4bf62e0d19a8a337c98b3a33df38a0c0
Fixes-bug: 1245905
2014-02-09 15:26:07 -05:00
Monty Taylor 1fdf58b0ab Remove explicit depends on transitive pip deps
These things were listed before we had jeepyb as its own things
(gerritlib in gerrit) and before jeepyb has pbr/requirements.txt
as it does now. With the move to pip install -U . in /opt/jeepyb,
there is no need to also ask puppet to manage these.

Change-Id: I7b521d03b3df8c0bde37586748769f160e615d31
2013-11-08 11:01:53 +13:00
Monty Taylor 6fcd2deb22 Use pip install instead of setup.py install
The pip vs. easy_install interactions continually get the systems
into a weird state. pip install $path_to_repo, on the other hand,
works like a charm.

Left out two graphite installs because they are doing path manipulation
and I want to do further testing before touching that.

Change-Id: I373b29aca03f1ebd20e533ebaaf5de9ef2db017e
2013-10-27 18:38:38 -04:00
Monty Taylor e64e8193e3 Use cgit server instead of github for everything
We have a cgit server now, which means we should replace
all references to github with references to git.openstack.org.

Change-Id: I68ad1ce514fb4326c7d9940b5a84999af5b58562
2013-08-19 10:19:47 -07:00
Elizabeth Krumbach 702e797fac Add gcc package dependency to jeepyb module
The entrypoints use for jeepyb won't work without pycrypto
installed, so we want to be able to pip install pycrypto which
requires the gcc package.

Change-Id: I9ff914347743c20728b5b10028866d8cbc7f0c61
2013-07-29 13:33:04 -07:00
Jeremy Stanley fc45733756 Install correct Python YAML package on CentOS/RH.
* modules/jeepyb/manifests/init.pp: Special-case the
Python YAML bindings package name depending on facter osfamily.

Change-Id: Iaa11e601e387428040a5d992219bd67812d8bac8
Reviewed-on: https://review.openstack.org/28652
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
2013-05-09 03:55:55 +00:00
Anita Kuno 3eb31b3983 Added swift store credentials to openstackwatch.
Openstackwatch is configured to store feed data in a swift object store.
This patch adds credentials to access that swift store.

Change-Id: I61d9032150ae40fb7d207b5861cf0cf448f6feda
Reviewed-on: https://review.openstack.org/24784
Reviewed-by: Elizabeth Krumbach <lyz@princessleia.com>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Approved: Monty Taylor <mordred@inaugust.com>
Reviewed-by: Monty Taylor <mordred@inaugust.com>
Tested-by: Jenkins
2013-03-25 10:15:21 +00:00
James E. Blair c5431c970f Add pkginfo to jeepyb.
Needed by the mirror script.

Change-Id: I0a30632810307a47ca473216d584dbcb9ed78a48
Reviewed-on: https://review.openstack.org/24864
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-20 19:14:57 +00:00
Anita Kuno 9bcd47d3ee Added python-swiftclient package resource to openstackwatch.pp.
Change-Id: I7dbe8413467485b81dbcfe1e1a3952770e97bac6
Reviewed-on: https://review.openstack.org/24614
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
2013-03-17 22:50:40 +00:00
Anita Kuno 533421276a Added a python-pyrss2gen package resource to openstackwatch.pp.
Change-Id: I556af39abb0446acac19381edc855172e22bdfa8
Reviewed-on: https://review.openstack.org/24607
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
2013-03-17 20:15:38 +00:00
Anita Kuno 4d1d5dda9d Change requirement on ini file object from Group to User.
Change-Id: I12a7ca31e67e9db06df7074529d5f9012e41bc0e
Reviewed-on: https://review.openstack.org/24603
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
2013-03-17 19:16:01 +00:00
Anita Kuno 1847ec3e61 Add 'managehome => true' to create home directory for openstackwatch user.
Change-Id: Ie1a0ae161858dc42c96cf2444fb2c3c97faafe85
Reviewed-on: https://review.openstack.org/24602
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
2013-03-17 18:19:26 +00:00
Anita Kuno e1d29a4f28 Add group 'openstackwatch' for the openstackwatch user.
Change-Id: Ie5452b76ba0bb404899a75f666df5ad064b26dbb
Reviewed-on: https://review.openstack.org/24598
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
2013-03-17 17:53:07 +00:00
Jeremy Stanley 77fa891783 Correct openstackwatch.ini template path.
* modules/jeepyb/manifests/openstackwatch.pp: Path to template must
prepend the module namespace.

Change-Id: I6215736111bdd527d68003bb1d0d16271ba7920c
Reviewed-on: https://review.openstack.org/24597
Reviewed-by: Anita Kuno <akuno@lavabit.com>
Approved: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
2013-03-17 15:24:26 +00:00
Anita Kuno 8c25b734c1 Get openstackwatch working using puppet.
Created an openstackwatch puppet manifest in modules/jeepyb.
Created an openstackwatch puppet template in modules/jeepyb.
Added a puppet class in modules/openstack_project/manifests/gerrit.pp.

Change-Id: Id38cda4fdf9feee4f018bc5ccb0ebbf462311061
Reviewed-on: https://review.openstack.org/23584
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Reviewed-by: Paul Belanger <paul.belanger@polybeacon.com>
Reviewed-by: Khai Do <zaro0508@gmail.com>
Reviewed-by: Elizabeth Krumbach <lyz@princessleia.com>
Approved: Jeremy Stanley <fungi@yuggoth.org>
Tested-by: Jenkins
2013-03-17 14:31:28 +00:00
Monty Taylor d87975e9bb Rename CI to Infra in MANY MANY places.
Change-Id: I409bd50ae374e0288531f07cfeea34856c5f8067
Reviewed-on: https://review.openstack.org/17319
Approved: James E. Blair <corvus@inaugust.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Tested-by: James E. Blair <corvus@inaugust.com>
2012-12-16 17:30:47 +00:00
Clark Boylan 873257193b Fix jeepyb install require on pypi.o.o.
The jeepyb install exec requires mysql::python but does not include that
class. This results in an error on hosts that do not include
mysql::python in some other manifest eg on pypi.openstack.org.

Include mysql::python in jeepyb's init manifest so that this require
works regardless of its inclusion with other manifests.

Change-Id: Iad1b1c723280df97ee5aa32512b36cfe1dc7034e
Reviewed-on: https://review.openstack.org/17414
Reviewed-by: James E. Blair <corvus@inaugust.com>
Reviewed-by: Paul Belanger <paul.belanger@polybeacon.com>
Approved: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Tested-by: Jenkins
2012-12-04 21:57:05 +00:00
James E. Blair 034a65f15d Use the mysql class to get python-mysql.
Change-Id: I6d6addc2bc0e28b289726cddd6626669dbec1e17
Reviewed-on: https://review.openstack.org/17292
Reviewed-by: Jeremy Stanley <fungi@yuggoth.org>
Approved: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
2012-12-02 16:42:49 +00:00
James E. Blair 09ec90959f Protect pygithub with proper capitalization.
Change-Id: I8a67a9781cc638138114da2eb870bcf856a37202
Reviewed-on: https://review.openstack.org/17272
Approved: James E. Blair <corvus@inaugust.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
2012-12-01 00:58:52 +00:00
Monty Taylor 51c1135690 Protect install of every jeepyb package.
Because puppet sucks at installing packages, wrap the installation of
everything that jeepyb needs in both jeepyb and in other places with
if blocks. *facepalm*

Change-Id: I69c73dabc3ea5d68bd057141b1ad758cdd68eafc
Reviewed-on: https://review.openstack.org/17130
Reviewed-by: James E. Blair <corvus@inaugust.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Approved: Monty Taylor <mordred@inaugust.com>
Tested-by: Jenkins
2012-11-30 18:25:30 +00:00
Monty Taylor 40e454c787 Consume jeepyb.
Instead of keeping many of these files directly in the tree, use them
from the out-of-tree jeepyb project, which makes them easier to consume
for other people who are not us.

Change-Id: Id704f2e17dd80709ef63cbbf2c5475a08a835f91
Reviewed-on: https://review.openstack.org/16777
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
2012-11-30 18:02:30 +00:00