From 062003ac00918c4f0114e504d9e91cb35b2b78eb Mon Sep 17 00:00:00 2001
From: Clark Boylan <clark.boylan@gmail.com>
Date: Thu, 13 Sep 2012 14:14:38 -0700
Subject: [PATCH] Use git review version 1.17 on jenkins slaves.

Downgrade git review to version 1.17 on jenkins slaves. Do this to avoid
bugs in 1.18.

Change-Id: I3102be5cabbc44a7e88206a6d2506bc7315c88f7
Reviewed-on: https://review.openstack.org/12986
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
---
 modules/jenkins/manifests/slave.pp | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/modules/jenkins/manifests/slave.pp b/modules/jenkins/manifests/slave.pp
index a01733023a..5f34bb3d85 100644
--- a/modules/jenkins/manifests/slave.pp
+++ b/modules/jenkins/manifests/slave.pp
@@ -56,7 +56,6 @@ class jenkins::slave($ssh_key, $sudo = false, $bare = false, $user = true) {
 
     # Packages that need to be installed from pip
     $pip_packages = [
-                 "git-review",
                  "setuptools-git",
                  "tox"]
 
@@ -66,6 +65,12 @@ class jenkins::slave($ssh_key, $sudo = false, $bare = false, $user = true) {
       require => Class[pip]
     }
 
+    package { 'git-review':
+      ensure => '1.17',
+      provider => pip,
+      require => Class[pip]
+    }
+
     file { 'profilerubygems':
       name => '/etc/profile.d/rubygems.sh',
       owner => 'root',