Remove log_dir from params and make logs configurable in init

log_dir does not seem to be used in other classes. In the use case where
it can be necessary to specify the location of ceilometer logs, this
patch makes it possible to do so while maintaining the default for
backwards compatibility. This patch also adds the possibility of
sending logs to syslog/rsyslog in the same way as implemented in
puppet-nova.

Change-Id: I1f970d5e3cc947308221fb2b2cdfb34f89c64314
Partial-Bug: #1269482
(cherry picked from commit 3685997f0ed4a88d0ccacd0415b210a948c66910)

Only add ceilometer user to nova group if using compute agent

When using ceilometer outside of the compute context (ex: with the
swift middleware), puppet-ceilometer attempts to add the user to
the nova group which does not (and probably should not) exist on
the swift nodes.

Change-Id: I6cc68152325a967c024ce611f92a90ce0afba8cc
(cherry picked from commit d064997cd4c50d9b5365fe2ffca60e14a86b08e1)

Fail puppet-lint on warnings

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

Change-Id: I49e181bdfd0b27a61bf7242accd352a3625eaf93
(cherry picked from commit ffde8588fd1360335fbfd376e00f8d53b0ef05e2)

Disable 80chars/class_parameter_defaults checks

* Disable warnings for "line has more than 80 characters"
* Disable warnings for "parameterised class parameter without a default value"

We do not wish to track warnings for such violations.

Change-Id: I4a00f219682a83f051c92b5e885e778b9422fdbf
(cherry picked from commit 9b901a5db9015129f98f189a4ddf5620f777ec7f)

Update fixtures.yml for grizzly

This commit updates the .fixtures.yml to ensure this
module can be used to install stable/grizzly from source
and updates inifile to use puppetlabs/inifile.

Change-Id: I3c9efc1924372a9005d7493716c903964600749f

Merge "Adding support for qpid in ceilometer, rspec for qpid part in ceilometer"

Update mysql allowed_hosts conditional statement

In the origin ceilometer::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 ceilometer::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
ceilometer::db::mysql::host_access

Fix bug 1206444

Change-Id: Ib6c33e6a27113d01326bee32c105fea4003059ce

Adding support for qpid in ceilometer, rspec for qpid part in ceilometer

Change-Id: I45396d3175b7984d47dbdb626c1729e0b2fdc105

Remove duplicated fixtures

Sort fixtures for better maintainability.

Change-Id: I17d4229584e2df2da2666e00f17bfcacdb8cdd03

Replace deprecated :rubygems source

Fix deprecation warnings thrown by bundler:

  The source :rubygems is deprecated because HTTP requests are insecure.
  Please change your source to 'https://rubygems.org' if possible,
  or 'http://rubygems.org' if not.

Change-Id: Ifaba2b94d7b47cd32b9bb6eec04cf7f4b89d0411

Merge "Add auth_admin_prefix support for ceilometer::api"

Switch deprecated rabbit_notifier to rpc_notifier

The 'nova.openstack.common.notifier.rabbit_notifier' is deprecated since
the G release of Openstack, it has to be replaced with
'nova.openstack.common.notifier.rpc_notifier'.

Change-Id: I2174bc1b6504bacfeb427b2f5b8f8dc19c167dcb

Add auth_admin_prefix support for ceilometer::api

To support keystone urls like http://host:80/keystone/admin

Implements blueprint serve-keystone-from-wsgi

Change-Id: I68c17a5b76ed61b18337ee580fba63200d874792

Merge "Capitalize boolean values in config files"

Merge "Add .gitreview"

Add puppet-lint to Gemfile

Change-Id: I34b20c29c27877c982c659253004df0a69cd0d27

Capitalize boolean values in config files

This patch allows the usage of Puppet boolean values and still
configure values with OpenStack boolean style: True/False.

Change-Id: I413060bc2ec0a8d795a240cc144e503efae41e49

Add .gitreview

Change-Id: Ia6bf51eb6acb37b12294f19151f63af419722f6b

Update README

Merge pull request #5 from fcharlier/lint

Merge back mgagne's 'grizzly' branch, fix some bugs & lint errors, add some tests. Fix #2 & #4

Update existing puppetdoc & add missing puppetdoc

Merge remote-tracking branch 'mgagne/grizzly' into lint

Conflicts:
manifests/db/mysql.pp
manifests/db/mysql/host_access.pp
manifests/keystone/auth.pp

Update some docs in the process

Update tests to add DEFAULT/ to nova_config

Fix notification relationship

Relationship was referring to Service[ceilometer] which does not exist.

Update puppetdoc for ceilometer::keystone::auth

Fix various puppet-lint errors and warnings

Check that rabbit_hosts overrides rabbit_host|port

Fix spec tests style

Fix the ceilometer::client class

the client python library is python-ceilometerclient
• fix package name on Debian platforms
• fix Package resource name

Remove the dbsync file, it's not used anymore

Ensure passwords/secrets are specified by user

Fix lint errors, add some documentation

Merge remote-tracking branch 'mgagne/grizzly'

Exclude 'master' branch from travis CI

'master' branch does not have the required files to make the build pass.

Fix ceilometer client package name for Debian

The package name is python-ceilometerclient, not python-ceilometer.
python-ceilometer is a package providing python libraries required
by ceilometer to work.

Add DEFAULT section to nova_config

nova_config now requires a section, add DEFAULT section.

Merge pull request #3 from aimonb/inifile-scope-fix

inifile module now requires a section

inifile module now requires a section
  * Added DEFAULT where missing is compute class

Update fixtures location to use stackforge.

Remove rabbit_host/rabbit_port config when rabbit_hosts is provided

Add rabbit_hosts parameter

Preserve rabbit_host parameter for backward compatibility

Fix notification relationship with Service[nova-compute]

Was getting an error when applying ceilometer::agent::compute:

  Could not find resource 'File_line[nova-notification-driver-common]
  File_line[nova-notification-driver-ceilometer]' for relationship
  on 'Service[nova-compute]'

Fix Ubuntu support for ceilometer::agent::compute

The libvirt group is libvirtd on Ubuntu and libvirt on Debian.

Fix service dependencies and add inheritance from ceilometer

Inheritance is required to modify User[ceilometer]

Remove useless params

Use implicit dependencies

Puppet has implicit dependencies:
- Users and Groups have an implicit dependency
- Implicitly orders users and groups
- Implicitly recognizes file hierarchy
- Implicitly creates relationship between file ownership and user resources

Add rspec tests for ceilometer::keystone::auth

Add rspec tests for ceilometer::db::mysql

Fix dependencies and various typos

Add rspec tests for ceilometer::collector

Add rspec tests for ceilometer::client

Remove inheritance from ceilometer

Partially add support for RedHat plateform

Add keystone_tenant parameter for api

Add rspec tests for ceilometer and ceilometer::api

Remove ceilometer-dbsync override

Fix dependency syntax in Modulefile

Remove version from endpoint.

The client has to specify it.

Fix adding notification_driver parameters to the nova.conf

Add required parameters to nova.conf

Fix the auth_protocol keystone_authtoken parameter name

Revert "Fix nova notifications"

This reverts commit 6c8b6630b0f671a055070c658e0e0d51350f8353.
This has been fixed in the 2012.2.2 nova packages

Fix order dependencies

Fix nova notifications

Apply nova commit 670b3882 to get notifications on the right
control_exchange

Fix glance exchange/topic to read notifications

One is a bad default value in ceilometer, waiting for the fix to be
merged.
The other is a bad value in Glance, fixed in Grizzly (maybe in 2012.2.3).

Fix groups memberships for the ceilometer user

The ceilometer  user must be  member of  libvirt group when  the compute
agent is installed

Agents, Api, Db & Collector classes inherit from ceilometer

Add “log_dir” parameter

The ceilometer user must be in the libvirt group

Ensure services are started

… or not, depending on the parameter

Update ceilometer user requirements

Restart services when config file changes

Rename default tenant to 'services' instead of 'service'

Ceilometer_config requires package ceilometer-common

Add user to localhost

Fix $auth_user var name

Merge pull request #1 from ekarlso/master

Add authentication options

Forgot to pass the password also

Only bother setting tenant_id if it's passed

Fix duplicate opt

Add authentication config opts

Fix ownership/permissions on /etc/ceilometer/*

Fix style

Ensure the ceilometer user has ResellerAdmin role

Required to poll swift usage

Add an example

Update the dbsync_command

Add the dbsync file, it's not yet in the packages

Add the metering_secret parameter

Default keystone_password to false to ensure error if not set

Add admin user/tenant/password to authtoken config

Remove wrong requirement for ceilometer-dbsync

Fix incomplete subscribe declaration

Fix user (group → gid)

Fix the name of the ceilometer config provider

Create ceilometer user/group

Add ceilometer-dbsync and fix some db-related deps

Refactoring/Style updates

Move database settings to a different class

Add dependencies into the Modulefile

Finish moving Keystone Auth from init.pp to api.pp