openstack/puppet-keystone
Code Issues Proposed changes
367 Commits 6 Branches 45 Tags
f95297ee0e
Commit Graph

15 Commits

Author SHA1 Message Date
Xingchao Yu
49b248344f Update allowed_hosts conditional statement 
In the origin keystone::db::mysql, if the value of $allowed_hosts
contains or equals to $host, then puppet will complain duplicate
declaration error. This patch is aim to update the allowed_hosts
conditonal statement in keystone::db::mysql.

There are two cases to pass $allowed_hosts to $real_allowed_hosts:

   - If $allowed_hosts is array,then remove $host from $allowed_hosts;
   - elsif $allowed_hosts is string and not equivalent to $host;

At last, if $real_allowed_hosts is not undef, then run
keystone::db::mysql::host_access

Fix bug 1206444

Change-Id: I8701aea9344a9151ce3d7ac8fa5792895a5aac6c
 2013-08-01 14:48:31 +08:00
danehans
21d91fde33 Remove Unnecessary mysql::server Dependency 
Previously, the db::mysql class required mysql::server.  This is
unnecessary since the mysql::db define requires the mysql::config
class for db creattion.  Additionally, this prevents users from
using a different class such as galera::server to manage
the database.

Change-Id: Ifb0fa261f9f5921cf9fe5b309decceab608d4726
 2013-07-18 22:38:28 +00:00
Mathieu Gagné
ce84d712de Various Puppet lint fixes 
* Fixes following warnings:
    * indentation of => is not properly aligned
    * string containing only a variable
* Fixes following errors:
    * tab character found
    * two-space soft tabs not used
* Remove quotes around class in include/require statements

Change-Id: I7e17d9153d413792e32f9d7c430dfbd37852ba51
 2013-07-01 17:07:54 -04:00
Dan Bode
49796d3e10 remove trailing whitespace 
Change-Id: I1982e0deac14a829daacbc64a1d89a2277e18a93
 2013-05-08 14:13:55 -07:00
Joe Topjian
1090ee2ea7 Removed quotes in allowed_hosts as it was causing a failure when allowed_hosts is an array of hosts 2013-03-24 21:43:29 +00:00
Vladimir Kuklin
adab1bd0b1 Ensure that mysql users are created before keystone-manage db_sync. 
Otherwise keystone-manage might not have rights to populate keystone DDL.
 2012-11-12 19:32:31 +04:00
Dan Bode
3e3201c15e Updates modules documentation. 2012-10-31 12:34:22 -07:00
Dan Bode
11b39e2bab minor style refactor 
move reps to top of manifest
 2012-10-23 15:32:40 -07:00
Dan Bode
5c86b6417c unset password defaults. 
setting passwords by default in the keystone
manifest is a potential security risk.

This commit unsets them to force users to set their own.
 2012-10-23 15:32:40 -07:00
Joe Topjian
83406ecb9c Setup better db reps 
Ensure that mysql is installed before we configure
keystone to use it.
 2012-10-23 15:32:39 -07:00
John Chilton
96bd60ed06 Allow override of mysql charset. 2012-05-16 09:31:37 -05:00
John Chilton
990522b8d8 Depend on mysql::config instead of mysql::server in order to allow mysql to fully configured before creating database. 2012-05-16 09:01:38 -05:00
François Charlier
64e77ac38f Update to use keystone::db::mysql::host_access 2012-04-24 12:55:18 +02:00
François Charlier
18c611bd76 Allow the database to be on another host 
Without any keystone package
Handling allowed_hosts
 2012-04-24 12:47:50 +02:00
Dan Bode
8d21eb9121 Move mysql to db::mysql 
Since keystone can use different databases, it makes
sense to move the mysql classes to db::mysql. This way
there is a clear standard for where other DB implementations
should be located.

This commit moves keystone::mysql to keystone::db::mysql
and keystone::mysql::access to keystone::db::mysql::host_access.
 2012-04-23 21:59:58 -07:00