Remove glance_notifications from notification_topic

The default notification_topic in Glance has been changed from
'glance_notifications' to 'notifications' in grizzly-2.

Ceilometer was configured to also listen on the
'glance_notifications' notification_topic in addition to
'notifications' to patch this bug. It shouldn't anymore.

Change-Id: I3d797c7e849319a1e20ef3ecbf158e8b8579d450
Closes-bug: #1273876

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

Merge "Fix package ceilometer-alarm type error on Debian"

Merge "Only add ceilometer user to nova group if using compute agent"

Fixed mysql dependency to match other modules

Fixed the mysql dependency to match the other puppet-openstack modules.

Change-Id: I8ad1011f892e2cadfbfa3fa4ca932fae9ded5cf0

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

Fix package ceilometer-alarm type error on Debian

This commit:
  - Fix type error about ceilometer-alarm package name
    - See bug #1263946
  - Fix indentation on Debian params.pp
  - Fix spec warnings (partition_rpc_topic,record_history)

Closes-bug: #1263946

Change-Id: I1ad19af429f2787d58837e51da95e981bdd66b40

Add API binding options

Ceilometer is able to bind API server to both an IP address and a port.
This patch aims to add support for both configuration flags.

Change-Id: I83164a925cf115d091cb04d14922ee7bbdd0ac63
Signed-off-by: Emilien Macchi <emilien.macchi@enovance.com>

Merge "Add Gemfile.lock to gitignore"

Add Gemfile.lock to gitignore

Change-Id: I397c8e72ae656c03f472137041e8ab950f529dfc

Remove use of deprecated include_class matcher

include_class is deprecated since rspec-puppet 1.0.0.
contain_class should be used instead.

Closes-bug: #1263617
Change-Id: If2e5a2c467a5d1c4eb130dc6b2643d05c748abe0

TTL mechanism support with expirer service

Aims to support a TTL mechanism for data entering Ceilometer
collector database.
TTL is very useful to configure when getting a lot of meters.

implement blueprint ttl-support
Change-Id: I9d98e8b833ad94fd79722dee7226e7f4f03aaa2e
Signed-off-by: Emilien Macchi <emilien.macchi@enovance.com>

Updates to params.pp for mongo and postgres packages

*) Removed the hardcoded postgres package from db.pp
*) Changed the mongo package to python-pymongo for RHEL

Change-Id: Ib975f9ce03ee8ffa423e75b62b595bcdefdd1eed
Signed-off-by: Francesco Vollero <fvollero@redhat.com>

Merge "changed rabbit_virtualhost(wrong) to rabbit_virtual_host"

Merge "Alarm: record history support"

fix postgresql connection string

Change-Id: I55fab1b9256df92cd521141a9d04a2aa2feb08c9

changed rabbit_virtualhost(wrong) to rabbit_virtual_host

the puppet module configures a rabbit_virtualhost, there is a missing underscore (rabbit_virtual_host). changed this for manifest and tests.

Change-Id: I02b34eb6531cfa0071a7e52f5a97f0e952f8ea85

Alarm: record history support

Ceilometer Alarm is able to record alarm change events.
This patch add this support in manifest.

Change-Id: Id8ece03dfd3491cd2886a345771df9622d702190
Signed-off-by: Emilien Macchi <emilien.macchi@enovance.com>

Updated site.pp and added alarm manifests

Change-Id: Ic9e8f977984bda02f60028b2ae18605df52a9205

Added alarm-notifier and alarm-evaluator services

This patch add the two services to the current puppet module to
be in line with the havana release.

Change-Id: Idfd422078d73aaee6da28cef35d825dd24e8afb8

Allow to set full url for endpoints

This changeset allows to set full url for endpoints. The default is
still to use the {public,admin,internal}_{protocol,host} and port
variables. The {public,admin,internal}_url parameters override the
_protocol, _host & port parameters.

There were previously one parameter for each {public,admin,internal}
endpoint url component (protocol, host), but not port.
Adding more parameters to allow different port would add too much
parameters and still missing the path part.

Fixes Bug#1227742

Change-Id: I834be8ab0f160b24348d218197711e8ecc392f5b

Add ceilometer::agent::auth.

Adds a new agent::auth manifest to hold common auth
parameters used by the Ceilometer compute and central agents.

This fixes a deployment issue where you would get duplicate
auth parameter errors when trying to configure the compute
and central agent on the same machine.

Fixes Bug #1220810.

Change-Id: I167c6c570bdebb4be2ef99cd0b9cf81f37856d9f

Use connection to replace deprecated database_connection

This patch is aim to use new param connection in [database]
section to replace the deprecated option database_connection
in [DEFAULT].

Fixes bug #1222744

Change-Id: I82082b791a4c16eb50092b3a96fb8ae439af4e5b

Merge "Inline a custom file_line provider to fix agent."

Merge "Add default auth_uri setting for auth token."

Inline a custom file_line provider to fix agent.

Fixes an issue in the ceilometer::agent::compute manifest
where nova.conf config values for the notification_driver
would get added to the wrong section (always to the end of
the file).

As part of the fix a custom file_line 'after' provider which
supports a new after option has been added.
This allows us to have some control over which section *new*
file lines go into. If there are any pre-existing matching
lines in the file the assumption is that they are already in
the correct section and can be edited in place.

NOTE: I've submitted a pull request to the upstream stdlib repo here
to add the new 'after' option:

  https://github.com/puppetlabs/puppetlabs-stdlib/pull/174

Once this (or something better) lands in stdlib we can update
puppet-ceilometer to use it.

Fixes LP Bug #1217867

Change-Id: Ic09f5232b322cde687d663d1ef38ef0fd12f32ff

Merge "Handle Centos/RHEL pymongo packages"

Add default auth_uri setting for auth token.

The latest keystoneclient code now gives a Warning if auth_uri
isn't explicitly specified.

Change-Id: I0feb52f7a9140ea750254f80b6a3e6855b25f1d3

Handle Centos/RHEL pymongo packages

RHEL/Centos has a different name for pymongo packages.  Fedora doesn't
provide python-sqlite2 so we undef that.

Fixes bug 1194141

Change-Id: If03f420c41e518227a1422882842661dfc8edd7e

Fail puppet-lint on warnings

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

Change-Id: I49e181bdfd0b27a61bf7242accd352a3625eaf93

Merge "Add support for CA authentication in Keystone"

Merge "Clean all warnings for puppet-lint gate"

Merge "Use Puppet boolean for better consistency"

Clean all warnings for puppet-lint gate

Change-Id: I292adf43bd434ca38d71baae6144e6ac4fea94a9

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

Add support for CA authentication in Keystone

* Add os_cacert flag support in manifests
* Add all required tests for Agent Central

Patch in Ceilometer: https://review.openstack.org/#/c/38448/

Change-Id: I6689143daddfac71ca5dcc2c6fd55cd14a4455e3

Use Puppet boolean for better consistency

Change-Id: I894af9499f1ea2fc255aa62fa6522c04a32d8a7c

Update .fixtures.yml to use puppetlabs/inifile

Change-Id: Ifb6b85dd403b79ea5a9a4f7f2fcc66e68a96c1f5

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