Fail puppet-lint on warnings

Puppet-lint will now fail if there is any warnings.

Disable 'class_inherits_from_params_class' check.

Change-Id: Ia1d7fa5109178f040b00abcdb3f7d84c4bdf58b4
(cherry picked from commit 48ac3fb090451b9c446a438ba5fbdc1999cb167a)
Add syslog support to the keystone module

Change-Id: I64a1959b88f1a5d7a8ce96bd1a0dbab2e9723a00
(cherry picked from commit 64474d170d5f7b38d4a221d1ade6009f44147c79)
pin to posgresql 2.5 module

b/c later than that breaks backwards
compatibility.

Change-Id: I7f8c5ad479f49c266911b2fdbc8da9eba5ddb65f
Make tenant name lookup for keystone user lazy

Previously, when you managed a keystone user, Puppet
would start by loading all information available
about each user. Most of this information was loaded
with a single call to user-list, but the tenant name
required an additional two commands (one to get the
tenantId, and an additional one to get the tenant name)

This patch makes the tenant lookup lazy, meaning that
those additional calls will only be performed when
the tenant attribute is actually used. This will
cause a significant performance increase for the
case where only a large set of users exists, and
only a small fraction of them need to be managed.

Partial-Bug: 1224179
Change-Id: I7141d8e922005ecae33c2c596c9806a03fede53f
(cherry picked from commit c194f374e2c54d03f34f28f002e478b8556c94c3)
Do not set tenant description in cache

Prevously, after the full list of tenants
was retrieved to build the cache, an
extra call was made to tenant-get per tenant to
retrieve the tenant's description.

This extra call changed the cache creation
operation from O(1) to O(N).

This commit improves the query complexity to
0(1) by moving the description lookup from
part of self.instances to the description
getter method (essentailly making it lazy).

This will work different depending on if you
are interacting with the resource in query
vs. manage mode.

In query mode, self.instances will build
out the cache withouth the description initially.
Since description is a properly, it will invoke
the getter method.

In manage mode, the description getter will only be invoked
for cases where the description is managed.

Partial-Bug: 1224179
Change-Id: Id9268678e122ba3aa3e5b0bcfd77466071a4192d
(cherry picked from commit b48032a42fd2019d0d11e1dafa22fd8431b0c1c9)
Merge "Do not set tenant description in cache" into stable/grizzly
Ensure that mysql users are created before keystone-manage db_sync.

Otherwise keystone-manage might not have rights to populate keystone DDL.
merged keystone/openldap support (from x-ldap)
Fixes puppet re-runs on controller node. Tenant transition is not supported by puppet and will 'fail', workarounded to 'warn' and thus continues. Reason: With a keystone backend to ldap, a user's tenant id is not reported by keystone user-get (and it is a hack in sql, see extra column json blob in user SQL table); thus, transitions would happen all the time (even if they are in that tenant already).
minor fixes
Merge branch 'grizzly' of https://github.com/gwdg/puppetlabs-keystone into grizzly