Merge "Add ability to change the slave gerrit user"

modules/jenkins/files/gitconfig

@@ -1,3 +0,0 @@
-[user]
-	name = OpenStack Jenkins
-	email = jenkins@openstack.org

modules/jenkins/manifests/jenkinsuser.pp

@@ -3,6 +3,9 @@
 class jenkins::jenkinsuser(
   $ssh_key = '',
   $ensure = present,
+  $gitfullname = 'OpenStack Jenkins',
+  $gitemail = 'jenkins@openstack.org',
+  $gerrituser = 'jenkins',
 ) {
 
   group { 'jenkins':
@@ -40,7 +43,7 @@ class jenkins::jenkinsuser(
     owner   => 'jenkins',
     group   => 'jenkins',
     mode    => '0640',
-    source  => 'puppet:///modules/jenkins/gitconfig',
+    content => template('jenkins/gitconfig.erb'),
     require => File['/home/jenkins'],
   }
 

modules/jenkins/manifests/slave.pp

@@ -4,6 +4,9 @@ class jenkins::slave(
   $ssh_key = '',
   $user = true,
   $python3 = false,
+  $gitfullname = 'OpenStack Jenkins',
+  $gitemail = 'jenkins@openstack.org',
+  $gerrituser = 'jenkins',
 ) {
 
   include pip
@@ -13,6 +16,9 @@ class jenkins::slave(
     class { 'jenkins::jenkinsuser':
       ensure      => present,
       ssh_key     => $ssh_key,
+      gitfullname => $gitfullname,
+      gitemail    => $gitemail,
+      gerrituser  => $gerrituser,
     }
   }
 

modules/jenkins/templates/gitconfig.erb

@@ -0,0 +1,6 @@
+[user]
+	name = <%= scope.lookupvar('jenkins::jenkinsuser::gitfullname') %>
+	email = <%= scope.lookupvar('jenkins::jenkinsuser::gitemail') %>
+[gitreview]
+	rebase = false
+	username = <%= scope.lookupvar('jenkins::jenkinsuser::gerrituser') %>