Delete update_users.py that no longer needed

Delete update_users.py since this script is not used any more.
Use update_gerrit_users.py instead.

Change-Id: Ie72b565df058a34e044921288013498b5be9477e
Reviewed-on: https://review.openstack.org/12487
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Tested-by: Jenkins
Fix launchapd user syncing on review-dev.

Launchpad user syncing to review-dev's gerrit instance was not working
as it was using the wrong ssh key to ssh into gerrit's command line
interface. Correct the ssh key and add proper logging to the sync script
on this host.

Change-Id: I1cbaafd736dafba6afee35ec04ed0024b0f2565a
Reviewed-on: https://review.openstack.org/12475
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Tested-by: Jenkins
Add coverage reports for heat

Change-Id: I1efc4a8ac59ba330a0db5984e380af84770422d3
Reviewed-on: https://review.openstack.org/12530
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Tested-by: Jenkins
Remove jenkins_job_builder module.

The jenkins job builder is now part of the jenkins module (as
jenkins::job_builder). Remove the now redundant and confusing
jenkins_job_builder module.

Change-Id: Ied1c1e646febd095a3377083f42a4e36f6bb634e
Reviewed-on: https://review.openstack.org/12385
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Monty Taylor <mordred@inaugust.com>
Reviewed-by: Monty Taylor <mordred@inaugust.com>
Tested-by: Jenkins
Check tox runs for sudo attempts.

If Jenkins has attempted to sudo during a tox run, fail the
test.

Also, remove "-e" from run-tox, so that pip freeze always runs.

Change-Id: I98230e4de03667f15f119fac45e1eba0ac249c1c
Reviewed-on: https://review.openstack.org/12458
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Approved: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
Set jenkins job default logrotate to 1 day.

Remove timeout and logrotate sections, except where overriding
defaults.

Change-Id: If65952da0bb59319e9c413349a95b4e7b39eece2
Reviewed-on: https://review.openstack.org/12516
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Approved: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
Pass sysadmins list into node defs.

Pass the sysadmins list into each node definition. This allows us to
retrieve the data from hiera rather than hard coding it in the puppet
manifests. Also, update test script to use bogus sysadmin data when
testing.

Change-Id: Ide3560f16bce4d66fb95cc5021fc879476e6a712
Reviewed-on: https://review.openstack.org/12512
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Monty Taylor <mordred@inaugust.com>
Reviewed-by: Monty Taylor <mordred@inaugust.com>
Tested-by: Jenkins
Gerrit Trivial Rebase Detection

Adds trivial_rebase.py for bug 881184.

* modules/openstack_project/files/gerrit/scripts/trivial_rebase.py: A
modified version of the contrib/trivial_rebase.py included in upstream
Gerrit's git tree (we'll try to get individual patches submitted
upstream soon):

    git clone https://gerrit.googlesource.com/gerrit

* modules/openstack_project/files/gerrit/patchset-created: Moved to
modules/openstack_project/templates/gerrit_patchset-created.erb and
added an entry for the trivial_rebase.py script with variables for its
command-line options sourced from gerrit.pp and review{,_dev}.pp.

* modules/openstack_project/manifests/gerrit.pp: Changed the file
definition for /home/gerrit2/review_site/hooks/patchset-created to use a
template, adding the ssh_host_key and trivial_rebase_role_id variables
it requires. Added a file entry for trivial_rebase.py as well, since
we're putting it in modules/openstack_project instead of modules/gerrit
where the existing scripts reside (we'll eventually want to move all of
modules/gerrit/files/scripts out of there at some point, but not now).

* modules/openstack_project/manifests/review_dev.pp: Override the
trivial_rebase_role_id variable to trivial-rebase@review-dev.o.o on
review-dev.

* modules/openstack_project/manifests/review.pp: Override the
trivial_rebase_role_id variable to trivial-rebase@review.o.o on review.

Change-Id: I941f15525c72b84708ae1de6832834eb53ab6863
Reviewed-on: https://review.openstack.org/12373
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
Update docs with new steps to add projects.

The process to add new projects to gerrit has changed since the docs
were originaly written and they need an update. Add steps for zuul, use
review.o.o instead of gerrit.o.o, and don't set shared=group when
running `git init` for the local git replicas.

Change-Id: Ief34dd4afce2fcc5884b1a0aca361b7105fc3790
Reviewed-on: https://review.openstack.org/12386
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Monty Taylor <mordred@inaugust.com>
Reviewed-by: Monty Taylor <mordred@inaugust.com>
Tested-by: Jenkins
Add volume-api project.

Add volume-api project. Close git pull requests and configure zuul to
build this project using netconn-api-20 as the example config.

Change-Id: I9d8e48120e25d6a28c51c0fcfba1c422560cff00
Reviewed-on: https://review.openstack.org/12387
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Monty Taylor <mordred@inaugust.com>
Reviewed-by: Monty Taylor <mordred@inaugust.com>
Tested-by: Jenkins
Make jenkins_ssh_key visible to static.pp.

openstack_project/static.pp needs to include openstack_project in order
to use openstack_project::jenkins_ssh_key. Make it so.

Change-Id: I4abe98492e32307aa1ba18c1efa485e8b5181b2e
Reviewed-on: https://review.openstack.org/12722
Approved: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Tested-by: Jenkins
Pass hiera sysadmin data to proper class.

The jenkins node includes the jenkins class and zuul class. Hiera
sysadmin data should be passed to the jenkins class and not the zuul
class. Move the argument to the jenkins class.

Change-Id: I0510446991749345d461c59083055a7750aede79
Reviewed-on: https://review.openstack.org/12723
Approved: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Tested-by: Jenkins
New job template for translation management.

Add a new job template for translation management. Add two jobs for
translation management. The first will will update transifex with the
new .pot and .po files after every merge to a project. The second will
commit the new .pot and .po files and submit for review once a day.

Add puppet manifest for the slave that these jobs will be running on.

Apply the template to nova.

Change-Id: I5242f81bd6ff13d1ed8610ef9e3b39fd3dbd7301
Reviewed-on: https://review.openstack.org/11369
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Monty Taylor <mordred@inaugust.com>
Reviewed-by: Monty Taylor <mordred@inaugust.com>
Tested-by: Jenkins
Fix PEP8 issues.

Fix some pep8 issues in doc/conf.py make the code looks pretty.

Change-Id: I137e42aed193eecccec7e4f4b295871c896640ba
Reviewed-on: https://review.openstack.org/12118
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Approved: Monty Taylor <mordred@inaugust.com>
Reviewed-by: Monty Taylor <mordred@inaugust.com>
Tested-by: Jenkins
Indentation fix

Fix several formatting issues following the guideline of PEP8:
1. List the import statements in alphabetical order.
2. Fix a lot of indentation issues.
3. Adjust the length of several lines.
4. Other PEP8 issues fixed.

Change-Id: I8fefa4db882d4fb72146052852a27b6994fc5d5b
Reviewed-on: https://review.openstack.org/12250
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Approved: Monty Taylor <mordred@inaugust.com>
Reviewed-by: Monty Taylor <mordred@inaugust.com>
Tested-by: Jenkins
Use defaults with tx init to run noninteractively.

tx init run without options expects one interactive value to be set.
Specify this host value with a command line option so that tx init can
run non interactively.

Change-Id: Ifc90fdefc0089924ee769cc1dc5db16f9e6a8804
Reviewed-on: https://review.openstack.org/12803
Approved: Clark Boylan <clark.boylan@gmail.com>
Reviewed-by: Clark Boylan <clark.boylan@gmail.com>
Tested-by: Jenkins
Use console-log scp macro to copy build logs.

Previoulsy we had spelled out that the build logs should be copied when
there were other files to copy in order to avoid multiple SCP
connections. Now, in order to copy the entire console log, we need
multiple SCP connections anyways so go back to using the console-log
macros for simplicity and consistency.

Change-Id: I6e88d08f7eec945b907353ed90ac0d482b7a9c9a
Reviewed-on: https://review.openstack.org/12532
Reviewed-by: Monty Taylor <mordred@inaugust.com>
Approved: James E. Blair <corvus@inaugust.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
Have install_modules install dependencies.

Have install_modules install module dependencies and handle module
upgrades. To make module upgrades sane, you must now specify the desired
version of a module.

Change-Id: I355308311c8555f1146999561de670294466b424
Reviewed-on: https://review.openstack.org/12732
Reviewed-by: Monty Taylor <mordred@inaugust.com>
Approved: James E. Blair <corvus@inaugust.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
Set tx to output full debugging info.

Use the --debug and --traceback flags when running `tx push` in order to
have better indications of why errors occur.

Change-Id: Ic951dc12b3efe8f8264521ed0a360e42f96cbee0
Reviewed-on: https://review.openstack.org/12817
Reviewed-by: Monty Taylor <mordred@inaugust.com>
Approved: James E. Blair <corvus@inaugust.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins
Need to git review -s before pushing to gerrit.

There is a bug in git-review where it doesn't properly configure itself
if it needs to before sending a patchset to gerrit. To get around this
explicitly configure git review.

Change-Id: I89cce741fd0ceb1e4fc4078bf2a53757cefec424
Reviewed-on: https://review.openstack.org/12880
Approved: James E. Blair <corvus@inaugust.com>
Reviewed-by: James E. Blair <corvus@inaugust.com>
Tested-by: Jenkins