This brings us up to date with the latest apache2 cookbook which
included a major refactor in 6.0.0 removing all of the definitions and
recipe with proper resources. Instead of using the apache2_default_site
resource, directly use a template and then enable the config file using
the apache2_site resource. This gives us the most flexibility.
Additional fixes:
- Install mod_wsgi as a package on RHEL since there is no built-in
resource for it.
- Remove hack for restarting apache.
- Convert web_app to template and subscribe to restarting apache.
- Remove resources to restore SELinux contexts since this taken care of
by Chef now automatically.
- Don't set SELinux to permissive on RHEL (I tested this works properly
with it set to enforcing).
- Remove collector recipe as this service has been deprecated upstream.
- Add openstack-ceilometer-notification package for RHEL in
agent_notification_packages as it was missing.
- Fix gnocchi-api_wsgi_and aodh-api_wsgi_file file on RHEL
- Remove openstack-aodh package in aodh_packages for RHEL as it does not
exist
- Clean up arrays in attributes using %w(foo) instead of ['foo']
- Set group for upgrade scripts so it can read files properly on RHEL
- Add missing ChefSpec tests for aodh
- Include additional cookbooks in Berksfile required for CI
Depends-On: https://review.opendev.org/702772
Depends-On: https://review.opendev.org/701824
Change-Id: Ie7e6d792a764314775c40d4489cfba05d5c483ce
I ran into a problem similar to this one [1] where testing gnocchi
commands did not work properly. The key problem is the fact that
`--skip-gnocchi-resource-types` was included in the ceilometer-upgrade
command. To allow an easy work around, I figure we can just create an
attribute which can add any option to ceilometer-upgrade so you can set
it to '' if you don't want it.
It might be debatable on whether to just default to run it with out
ceilometer-upgrade instead.
Some additional fixes include:
- Move the ceilometer-upgrade command to a new setup recipe where it
makes more sense since it needs to be run after Ceilometer and Gnocchi
are setup.
- Fix name of gnocchi-metricd_service for RHEL
- Add same apache restart fixes that we implemented in other cookbooks.
This is needed so that you can properly run ceilometer-upgrade with
gnocchi enabled.
[1] https://bugs.launchpad.net/openstack-ansible/+bug/1737096
Change-Id: I619ef044b8cb254b23e0c7bc674c46d5dd7e0076
Signed-off-by: Lance Albertson <lance@osuosl.org>
* applied new template logic and refactored all recipes accordingly
* removed vmware and suse support and specs
* removed alarm ceilometer-alarm notifier and evaluator (needs to be rewritten
for aodh)
* added recipe for telemetry-metric/gnocchi deployment
* set gnocchi as default dispatcher for ceilometer
* added initial specs for gnocchi recipes
Depends-On: I22114b6d2d46cce561a2f292b92ff5f531cff533
Change-Id: If513cb2715d8266845bd6541d12005edef70f11c
Cleaned up all the minor rubocop issues, the ones left relate to
complex logic and what I think is a bug in rubocop for nested
vs compact modules/class definitions.
Change-Id: Ib326cac0e09df628189e290807513bad59df0b76
The default dbsync command timeout value 3600 seconds may be too small
when there are large data in ceilometer database.
Make timeout to be optional to fix this problem.
Change-Id: Id4e703fab260f8e010ff089e4134e7aab8e21fd2
Closes-bug: #1389111
* Remove very old and now incorrect check for running
ceilometer-dbsync with nosql. This is fine to do now, and is
needed when nosql is used for DB2 with ceilometer.
Change-Id: Id31d7dffb3e01513bb0470a7a6f5f07118dc28ef
Closes-Bug: #1373600
Since no version is specified for any package and usually you do not
want to stick with one unspecific version you get during your first run
the action should be :upgrade.
Change-Id: Idc662737f1b03049de5555c6e1e54fc77532972f
Implements: blueprint allow-package-updates
Some things were a little off wrt. the way we usually do stuff. This
change fixes the following:
- package resources take the override settings from node attributes
- all services subscribe to their config file
- the service definitions include restart and status actions, and
enable services
Change-Id: I3d4a2f9247dda34a35897fdfe08d9e8cf27556b8
Closes-Bug: #1303777
When NoSQL is used, ceilometer db sync is not required.
Change-Id: I568baf73d95311d895cbd59119688495a0b879dc
Implements: blueprint metering-nosql-support
- Add rubocop.yml in recipes/
- Add recipes rubocop test to Strainerfile
- Update recipes to comply with rubocop
Change-Id: I7129c11c769688812e412a49cf37d4935d984985
Addresses: blueprint rubocop-for-metering
See https://bugs.launchpad.net/cloud-archive/+bug/1221945
The ubuntu cloud archive ceilometer-collector pkg has a bug in which init
script isn't installing properly gets installed as:
/etc/init/ceilometer-collector.conf.dpk-new therefore until it gets fixed this temp
solution works
Change-Id: Ie7a82f2b76e81c42c2880e3785fa9c164e7896a0
These were migrated out of `openstack-compute` and into their
own cookbook. This is an initial best-effort attempt.
Made the following changes, which differ from the recipes imported.
* Moved to openstack["metering"] attribute namespace.
* No longer performing source install. This was done for folsom.
* No longer writing out an Apache virtualhost. This was done to
protect the folsom non-authenticated API endpoint.
* Templates write out as the ceilometer user/group vs nova.
* No longer including nova-common recipe. It doesn't appear nova.conf
is necessay (although I may be wrong). The stackforge puppet manifests
do not seem to do this [1].
* Added Strainerfile for testing.
[1] https://github.com/stackforge/puppet-ceilometer).
bug 1187514
Change-Id: Ia97126e13e422037ad1f2c89debd9e4cdfa60b0a
Lance Albertson