opendev/system-config
Code Issues Proposed changes

Fix Gerrit MySQL DB.

Browse Source 
I hope. The grant parameter appears to want an array so give it one.
Also enforce order by requiring the mysql server and account security
settings in the DB resource.

Change-Id: I2c99c25cb09cb5b68240a5fbd146f47ba8aee410
Reviewed-on: https://review.openstack.org/14320
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
This commit is contained in:
Clark Boylan 2012-10-10 18:46:19 -07:00 committed by Jenkins
parent 3a6437a21b
commit 16ba76b01d
1 changed files with 5 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

6
modules/gerrit/manifests/init.pp
View File

@ -266,8 +266,12 @@ class gerrit($vhost_name=$fqdn,
user => 'gerrit2', user => 'gerrit2',
password => $mysql_password, password => $mysql_password,
host => 'localhost', host => 'localhost',
grant => 'all', grant => ['all'],
charset => 'latin1', charset => 'latin1',
require => [
Class['mysql::server'],
Class['mysql::server::account_security'],
],
} }
# Set up apache. # Set up apache.