Update ChefSpec due to changes made in apache2 cookbook.
Depends-On: https://review.opendev.org/756168
Change-Id: Ia19ea6bebf787f1d8b1542bb6c895a294a5e72bc
Signed-off-by: Lance Albertson <lance@osuosl.org>
- Cookstyle fixes
- Refactor Berksfile to use groups so we can exclude integration testing
cookbooks
- Update documentation
- Cleanup line wraps
- Enable sensitive resources for the template[/etc/aodh/aodh.conf],
template[/etc/gnocchi/gnocchi.conf] and
template[/etc/ceilometer/ceilometer.conf] to resources improve
security.
- Update delivery configuration to exclude integration cookbooks
- Fix ChefSpec output.
- Add missing ChefSpec tests
- Switch package installations to send packages as arrays instead of individual
package resources. This generally speeds up chef runs.
- Cleanup array syntax using %w() instead of []
Depends-On: https://review.opendev.org/701027
Depends-On: https://review.opendev.org/706101
Depends-On: https://review.opendev.org/706151
Change-Id: I73e3c3dce64798a4d6ba5c94cc45dfabc4d2d0ff
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
This adds management of /etc/ceilometer/polling.yaml which defines which metrics
to poll and how often. This pulls in the default configuration which the same
for both RHEL and Ubuntu and adds two new attributes for configuring the file.
In addition, restart various services related to the configuration file.
Change-Id: I69ed0a8933a971d9cac20dbd4133197a7e4274af
Signed-off-by: Lance Albertson <lance@osuosl.org>
The API service for Ceilometer was deprecated in Ocata [1] and apparently we
missed that. This removes all references to the API service which is no longer
needed.
[1] https://docs.openstack.org/releasenotes/ceilometer/ocata.html#deprecation-notes
Change-Id: I7a888a4aaadaa8213d6763854b39d78910fad0db
Signed-off-by: Lance Albertson <lance@osuosl.org>
This adds a template for managing the pipeline.yaml file which allows one to
configure the pipeline publishers such as gnocchi or prometheus. The template
file was taken from the latest Rocky tarball for ceilometer.
Change-Id: I938219bfdcfa69a910cae4b72a40c19d5124d99e
This fixes two primary issues:
- Removal of using node['openstack']['api']['auth']['version'] and converting to
using ::URI.decode
- Renaming endpoint attributes from telemetry-metric to telemetry_metric to
match what's used in openstack-common. Also rename all attributes to using the
underscore to minimize confusion.
Change-Id: I59c960a18f9e88eea494ec3c7bdbf189309f23e7
Signed-off-by: Lance Albertson <lance@osuosl.org>
- Switched default linter to cookstyle
- Renamed rake tasks to better conform with Chef conventions
- Normalized template banner
Change-Id: If802d82561619a0b7951422dc0e1e913f2c4c8b4
- Style and lint fixes for newer chefdk
- Removed ancient Gemfile
- Rewrote metadata.rb for readability
Change-Id: Iba9ba7110dfde7635edc6706ecbbafbdbb161002
- there was a bug when creating the
node['apache']['run_dir']}/ceilometer directory
which won't be created when the enclosing folder doesn't exist.
This is the case especially on Compute-Nodes in Multi-Node environments
Change-Id: Id7fa2f415b85c83c978cf391c7ad4afe08239250
- corrects deamon_process to daemon_process
- removes duplicate Listen directive in wsgi-template.conf.erb that gets
emitted in ports.conf that the apache2 cookbook lays down
- adds a run directory for ceilometer api socket
Change-Id: Ia369a50e42453428b0c572659b0c4078920e9890
* 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
Remove deprecated keys and use identity_uri via the new transform
helper method.
Change-Id: Ibd21f81607f5ef1b02e07e4da3e5ea306f7d6f8b
Implements: blueprint identity-uri
Now that admin_endpoint, public_endpoint, and internal_endpoint
in the common library are working, these are the changes to use
them in the openstack-telemetry recipes.
Change-Id: Ibddb87c1ae2e92c43c0b88495d363b6d8179d300
Partial-Bug: 1412919
1. If the compute driver is vmwareapi.VMwareESXDriver or
vmwareapi.VMwareVCDriver, set the hypervisor_inspector to vsphere and
add vmware section in conf file.
2. Reuse the vmware related attribute from compute cookbooks since
ceilometer need monitor the same vsphere.
3. Update the testcases
Closes-Bug: #1351619
Change-Id: Ie67e84e5695bd5f9ceaa6ed6682cfe33f7ed0b68
*_python_packages attributes are being moved to -common in order to
remove the duplication from all the cookbooks which are using them
Change-Id: I03d0acf12d4c53952498beeaefad9f3b237ccb9d
Implements: blueprint move-python-db-client-attrs-to-common
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
The policy file is just a dup of what's already in the distro
packages. Until the cookbooks/chef support a policy type resource
duping them does not make sense and causes churn to keep in sync
with base openstack code.
Change-Id: Ib09d5efdf3927a381501eebd78d5c4db5133f057
Implements: blueprint remove-policy-templates
Closes-Bug: #1324187
Add the ability to override the default listen IP/port for the api
service in the config file.
Use the ['telemetry-api-bind'] namespace to derive these values
Implements blueprint increase-ip-binding-flexibility
Change-Id: Ia51bd1b760563f06ed36ec8190f36f4e03a79695
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
Add a new attribute
default['openstack']['telemetry']['platform']['db2_python_packages']
Change-Id: I51c6974956ea5247f31f1d060cc8404610fab1d1
Implements: blueprint db2-support
The new get_secret method allows the 'secrets' databag to be renamed,
just like all other databags.
Change-Id: Ic8be5d19a112d78e22716c0cd571efc7186ac9ce
Related-Bug: #1288784
Currently, qpid's password is stored in an attribute in the common
cookbook. This change causes the password to be looked up via
get_password method, which is how we obtain the rabbitmq password.
Change-Id: I7afe9a0a069c6527ced4df7f094c31c1ca4acec4
Implements: blueprint use-data-bag-for-qpid-password
Now that almost all of the rubocop blueprints have been completed, make
one final pass through all of the cookbooks ensuring they're all in
sync with each other.
- Upgrade rubocop to 0.18.1
- Fix violations caused by 0.18.1 upgrade
- Move Excludes for non-existent folders to Includes so they
automatically cover future additions
Change-Id: I62cb43675f56469cefaa480e5043514741bedb04
Implements: blueprint lint-and-unit-testing-for-havana
This change removes the attributes we'll be moving to openstack-common
and changes reference of those attributes to reflect new (more
consistent) namespace.
Change-Id: I30e37b4ff6369c19978973a8890ef2122692db44
Implements: blueprint clean-up-attr-for-mq-and-db
- 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
The user_password, service_password and db_password functions are redundant
since they simply call "secret". Creates a get_password function thata will
accept a "type" of db, service or user.
All instances of these calls have been changed to call get_password
Change-Id: Ia4dbfe4ff794efc84c900950a666085deebaf7f3
Partial-Bug: #1195915
Moving all qpid parameters under default["openstack"]["metering"]
["mq"]["qpid"] so that other qpid options can be added under it.
Rabbit parameters will be brought under "mq" eventually. MQ is
selected through default["openstack"]["metering"]["mq"]["service_type"].
Default is "rabbitmq".
Implements: blueprint qpid-activemq-support
Change-Id: I4dd153a631063ee7c518e2d49fe01aa39a6f261d
The other openstack services cookbooks refer to the db password databag by the
service name (nova, swift, quantum) and therefore openstack-metering cookbook
should call the db password databag by service name 'ceilometer' to make it
consistent
Change-Id: I0a17e065f630f8506a5dd94b3a1ff8ed44482adc
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
Ghanshyam Mann