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>
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>
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
RDO provides a sane copy of this file and it's causing issues on RHEL platforms.
Change-Id: Ia75850fd53829593d556e1e3fb1650f8ad919415
Signed-off-by: Lance Albertson <lance@osuosl.org>
fog-openstack-1.x already appends "auth/tokens" so we no longer need to
do that.
In addition, re-add openstack_endpoint_type which is now fixed now that this [1]
has been released upstream.
[1] https://github.com/fog/fog-openstack/pull/494
Change-Id: I06ee3681ff8bc90f15b2b0ecbdcaebe475709b31
Signed-off-by: Lance Albertson <lance@osuosl.org>
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>
This patch removes the openstack_user resource with :grant_domain
action. A user is always created within a specific domain; such a
membership cannot be tacked on later. This resource gave the users the
role intended for their project for the domain (i.e., for the Default
domain instead of for the service project).
We add the domain_name attribute that creates the users in the
desired domain. Note that this change needs a sufficiently recent
openstackclient cookbook -- otherwise the domain_name attribute is
ignored (which does not matter as long as the users are to be
created in the Default domain).
Change-Id: Ie793b3d2e0419da90888edbf33ff28d819892dcf
This patch removes the role_name when using openstack_user's :create
action (it gets ignored by the target method).
Note that the spec test would still pass if only the line in
identity_registration.rb (but not the test) were changed, because the
code that actually does grant the role to the resource is executed right
after user creation and before any tests check the resource for the
existence of the role_name attribute. In other words: if the argument
were required in a call but only supplied in another call, the spec
tests would not catch it. Something to watch out for.
Change-Id: I45754d269598b821f0b86b6257556ebfe74dcd6b
- 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
When running with Ceph as Gnocchi storage backend, the upgrade may fail
in certain situations, so we need to be able to insert some option into
the call.
Change-Id: I52be4a184be2b0cca2289acce86d8fdf3b807651
- Remove hardcoded old package version, always upgrade to latest
version instead
- Install python-gnocchiclient needed by ceilometer in order to access
the gnocchi backend
Change-Id: I6f6af90d3107df86bd0abbaacfca0836f7771cfe
- Now use cookbook-openstackclient to create endpoints role service and
user
- added domain creation and access granting
- added values to work with identity_v3
- rewrote specs to work again
- updated readme
Change-Id: I6c91c874013190522dd7f1fd6060dc3796dc80fd
Depends-On: I0f8955f05de9b33711c54b9a198f45018cceb8e1
Depends-On: I2d404a424bd79a6e9b282304e21591fa33a48981
Depends-On: If7b4d6e563081a0be9957353d73ef61a9688df56
Depends-On: I84f850f32f25a318c3ed3c7337a0dfa6f641a5fe
* 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
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