Commit Graph

471 Commits

Author SHA1 Message Date
Zuul f445dd6c14 Merge "Expose rabbit_transient_quorum_queue" 2024-03-14 18:25:37 +00:00
Takashi Kajinami a1d1578eb3 Expose rabbit_transient_quorum_queue
Depends-on: https://review.opendev.org/911021
Change-Id: I13dd3b7a88c9ea5e1d9f7543111f85b5a4e84cde
2024-03-13 16:16:33 +09:00
Takashi Kajinami 67079e54d5 cache: Support new redis options
The oslo.cache 3.7.0 release introduced some options for redis backend
and redis sentinel backend. This introduces for these parameters.

Depneds-on: https://review.opendev.org/910629
Change-Id: I6c7ca526af7ddba27660791c23d1895d668b460b
2024-03-07 22:49:25 +09:00
Zuul e3114760da Merge "cache: Support options for SASL mechanism in memcached" 2024-03-03 22:58:09 +00:00
Takashi Kajinami 66b692a0ae cache: Support options for SASL mechanism in memcached
Depends-on: https://review.opendev.org/910122
Change-Id: I5affe66258440edbd52c487374a31fca92370093
2024-03-01 10:32:11 +09:00
Takashi Kajinami b8a4f9d20e Add support for oslo.reports options
Depends-on: https://review.opendev.org/804804
Change-Id: I9bda11ff00d1043747428100f92f24d89a1df0bb
2024-01-31 11:01:59 +09:00
Takashi Kajinami e2fe2db17e Support [cache] memcache_pool_flush_on_reconnect
Depends-on: https://review.opendev.org/902861
Change-Id: I4d104dc7a3937d0a9c4ab3e02dbcd1280bcdfa09
2023-12-14 17:54:02 +09:00
Takashi Kajinami c6a7b16567 Make telemetry_secret actually required
... as is described. This option should be set to an appropriate key
value and the previous default (False) is not suitable for production
use.

Change-Id: Ic54025f2b6c1d023a8458491e3f26dda8adbceb0
2023-12-06 01:56:51 +09:00
Takashi Kajinami 235f1064dd Remove usage of deleted manifest_dir
Recent update in rspec-puppet removed some of the config interfaces for
old puppet versions[1]. This drops usage of these interfaces to resolve
the following error in unit tests.

```
An error occurred while loading ./spec/unit/provider/manila_spec.rb.
Failure/Error: c.manifest_dir = File.join(fixture_path, 'manifests')

NoMethodError:
  undefined method `manifest_dir='...
```

[1] 316d95923c

Change-Id: Ide0fcfe683cc5f41c150d3bf8486144152257c8e
2023-10-11 11:53:17 +09:00
Takashi Kajinami ccf4c50573 RabbitMQ: Add support for quorum queue options
Depends-on: https://review.opendev.org/894866
Change-Id: I007ca152da1c40564bb575b62eb8ff975970865a
2023-09-14 14:46:43 +09:00
Yadnesh Kulkarni 204a8b398d Set 'tenant_name_discovery' parameter in 'polling' section
As of now 'tenant_name_discovery'[1] is being configured in 'DEFAULT' section in ceilometer config file.

As per Ceilometer this must be configured as a polling option[2].

[1] https://github.com/openstack/ceilometer/blob/master/ceilometer/polling/manager.py#L69-L80
[2] https://github.com/openstack/ceilometer/blob/master/ceilometer/opts.py#L98

Change-Id: Ib5c273df325f5ed0dccf7f45a9c7a91aba9568c3
2023-08-25 19:14:00 +09:00
Yadnesh Kulkarni 524ae2eead Add "tenant_name_discovery" to all namespaces
Upon enabling this parameter in all namespaces, ceilometer
agents(central, compute, ipmi) will be able to identify users
and project names in the metrics generated from their nodes.

Without this change, the parameter is missing in compute and
ipmi namespace.

Change-Id: I0c047e9c8e33c05247d7af69d2a80e7cc22a0304
2023-06-13 10:55:39 +05:30
Takashi Kajinami 331b15d107 Replace legacy facts and use fact hash
... because the latest lint no longer allows usage of legacy facts and
top scope fact.

Change-Id: Iffdf359f0db141d80168e7bbaa0b10af31ae0ac6
2023-03-01 15:25:59 +09:00
Zuul 31cead15d3 Merge "Remove snmp metrics from polling.yaml" 2023-02-21 04:05:34 +00:00
Takashi Kajinami 5699fe2418 Support separate polling agents
This change introduces capability to use separate polling agents
(central, compute and ipmi) instead of the single polling service.

Change-Id: Idaa46ad71f4bedaca91b75a732649de967663659
2023-02-15 17:21:23 +09:00
Takashi Kajinami 704a6ea58c Remove snmp metrics from polling.yaml
These metrics are no longer supported by ceilometer since
GenericHardwareDeclarativePollster was removed.

Closes-Bug: #2007108
Depends-on: https://review.opendev.org/c/openstack/ceilometer/+/840808
Change-Id: I0b5d6b360e8912a78f041806323d6b5bc8d87258
2023-02-13 17:59:02 +09:00
Takashi Kajinami fa2b78f2ea notification: Support tuning batch processing
This change introduces support for the option to tune batch processing
by notification agent.

Change-Id: I9cdbbe9becfffce68966a29b9e803218637d7714
2023-01-23 23:06:47 +09:00
Takashi Kajinami 834d9a6780 Remove implementations to set up Ceilometer DB
Ceilometer no longer requires its own database since metric/event
storage was offloaded to Gnocchi/Panko. These implementations were
formally deprecated during the previous cycle[1] so can be removed.

This also updates the outdated example manifests to adapt to
the removal and the recent refactoring.

[1] 60882c29df

Change-Id: Id18b9221703eda9311b81c5a5dfdd41cc68f5424
2022-11-24 14:08:10 +09:00
Yadnesh Kulkarni b00000fb86 Add new parameter `tenant_name_discovery`
Enabling this parameter will identify user and project
names from the polled metrics [1].

These details are collected by making additional requests
to keystone service, depending upon the scale of environment,
number of projects/users and the count of metrics polled in
every iteration, enabling this could overwhelm the keystone
service.

[1] Iee5dbf09a1fd3ac571746fc66d2683eb8e6a1b27

Depends-on: https://review.opendev.org/852948
Signed-off-by: Yadnesh Kulkarni <ykulkarn@redhat.com>
Change-Id: I045c7cd7a86d0f7f97a9078cbfc74353dcff0eb8
2022-09-21 13:35:48 +09:00
Takashi Kajinami 1dcdea2e62 Forcefully purge [hardware] readonly_user_* parameters
This is follow-up of e5ce40b16d and now
enforces that the parameters are removed from ceilometer.conf because
these were already removed from ceilometer and no longer used.

Depends-on: https://review.opendev.org/840808
Change-Id: I4d9196133ead84d4b388929231ce970882fcaaa6
2022-09-08 00:05:44 +09:00
Takashi Kajinami 3cee14c87f Enable memcached and redis in acceptance tests
... because these are required as cache/coordination backend.

Change-Id: I833d06ace65f97c68c3024ef7f283c8b09181939
2022-08-31 00:51:48 +09:00
Takashi Kajinami 2f053934da Remove deprecated ceilometer::agent::polling::coordination_url
... because it was deprecated during Xena cycle[1] in favor of the new
ceilometer::coordination class.

[1] 1155b41db0

Change-Id: I0885b79bd83bcc3b41b8da0bd0fd00f29d176f13
2022-08-05 01:14:49 +09:00
Takashi Kajinami 588c9c9501 Revert "CentOS 9: Disable unit tests dependent on puppet-postgresql"
This reverts commit 23ab18bfb6.

Reason for revert:
puppet-postgresql 8.1.0 was released and now the module supports RHEL 9
(and CentOS 9 effectively).

Note:
This change adds the service_provider fact in test fact data because
it is required by puppet-postgresql.

Depends-on: https://review.opendev.org/850705
Change-Id: I4165cdb8062f22ec73fca94e184bd42960c8bead
2022-07-24 01:08:05 +09:00
Zuul 53a6f17dc7 Merge "Add acceptance tests for config management resources" 2022-07-15 19:46:09 +00:00
Takashi Kajinami 962afdd17d Fix wrong test description
The value with a white space should be accepted instead of being
rejected. This change fixes the wrong test case description and makes
sure the description explains what is actually tested.

Change-Id: I78750bdfd66785f822a37d724cd4f50eca8e0bab
2022-07-11 23:26:04 +09:00
Takashi Kajinami a48659ea0d Add acceptance tests for config management resources
Change-Id: Ibf9df166c7262d51ed937d4cf4f3fc30099fa57b
2022-07-10 12:36:25 +09:00
Takashi Kajinami a8e2dc6891 Clean up deprecated cache parameters
These parameters were deprecated during Yoga cycle[1].

[1] a7cf7060dd

Change-Id: I0358678a9298dfceb9c4ebf918ef334f0f5e496d
2022-06-19 21:53:29 +09:00
Rajesh Tailor 7d3af77f01 Fix typos in parameter descriptions and tests
Change-Id: Id17358fce48468284f162a6de35840d439871cd5
2022-06-17 19:07:22 +05:30
Zuul 210fdec48f Merge "Refactor service user/group management" 2022-06-14 02:26:50 +00:00
Takashi Kajinami c6bd935e27 Replace usage of mongodb in unit tests
... because mongodb is no longer supported[1].

[1] 2ad73a71c095bf534ed9f6a5d4356d1843a5d919

Change-Id: I1aed43e43b6c848f6c20fd569dbf4422c2125400
2022-06-08 11:05:44 +09:00
Takashi Kajinami eafc908871 Refactor service user/group management
This change refactors how the ceilometer service user and group are
managed.

- The ceilometer service user and group are created by the common
  package. While the user resource should still be declared to manage
  its group membership, we don't need the group resource.

- Introduces the configuration knob to disable user/group management.
  This would be useful in the case where all service users are
  declared externally.

Change-Id: Iaabe5b02f0ebd782debd0f3ca41e2fdafbf9c80f
2022-05-16 11:57:13 +09:00
Takashi Kajinami 2c48365980 polling: Ensure unused parameters are cleared
... instead of leaving the parameters unmanaged.

Change-Id: I0eeb25b131385ad4448fc8cec04efbdce0b7d87a
2022-05-06 11:26:26 +09:00
Takashi Kajinami 72235236a1 Remove unused "transformers" field from event pipeline configuration
Transformer was already removed from ceilometer by [1]. This field
was removed from the example configuration file when transformer was
deprecated[2].

[1] 9db5c6c9bfc66018aeb78c4a262e1bfa9b326798
[2] 1dcbd607df0696101b40f77d7721489679ebe0ba

Change-Id: I012d3f960d2a562c2735e7b5c1e408a3a1addb70
2022-03-25 08:16:38 +09:00
Takashi Kajinami 17864e2aec Remove unused "triggers" field from event pipeline configuration
The triggers field is no longer used. The field was removed from
the example file by c569fcd5ed59b00950a2ac7b9df75eb700a883e8 .

Change-Id: I03b1daffa4de60f562f747b1c88ffb444020905f
2022-03-25 08:14:54 +09:00
Takashi Kajinami 5f52d0d0e7 Globally support system scope credentials
After spending huge effort to understand the exact requirements to
enforce SRBAC, we learned it's very difficult to find the required
scope in each credential. This requires understanding implementation of
client-side as well as server-side, and requirement might be different
according to the deployment architecture or features used.

Instead of implementing support based on the actual implementation,
this introduces support for system scope credentials to all places
where keystone user credential is defined, and make all credential
configurations consistent.

Change-Id: I3a659a6b43d9c47e88334c24fb866a73a8f24a24
2022-03-04 01:16:58 +09:00
Takashi Kajinami 23ab18bfb6 CentOS 9: Disable unit tests dependent on puppet-postgresql
The puppt-postgresql module does not support CentOS 9 yet and requires
some version parameters to be run on CentOS 9. This change disables
unit tests requiring that module, until the module supports CentOS 9.

Change-Id: Ic9c6b8cd1bd6992c3ded3bbe4c07150246f86852
2022-02-16 00:17:56 +09:00
Takashi Kajinami a7cf7060dd Deprecate cache parameters in the base class
in favor of the new ceilometer::cache class. This should have been done
when we added that class.

Change-Id: Ida080fe2a71d6215a8365cfb4f9f902dfcaaa760
2022-02-04 11:30:26 +09:00
Grzegorz Grasza b13732ce19 Add HashClient retry options for the dogpile.cache backend
This patch specifies a set of options required to setup the HashClient
retry feature of dogpile.cache cache backend.

Original oslo.cache change:
https://review.opendev.org/c/openstack/oslo.cache/+/824944

Depends-On: https://review.opendev.org/826870
Change-Id: Idb2fe7e5573103d9e7ef36652fc2b4252418cdc3
2022-01-31 11:33:51 +01:00
Grzegorz Grasza f1a4758061 Add cache client retry options for the pymemcache backend
This patch specifies a set of options required to setup the retrying
wrapper feature of pymemcache (dogpile.cache) cache backend.

Original oslo.cache change:
https://review.opendev.org/c/openstack/oslo.cache/+/803747

Depends-On: https://review.opendev.org/826869
Change-Id: I6ab9dcd40678d98aceed8fe28658b984b41a63cb
2022-01-31 11:32:54 +01:00
Grzegorz Grasza 455cf1ba64 Add socket keepalive options for the pymemcache backend
This patch specifies a set of options required to setup the socket
keepalive feature of pymemcache (dogpile.cache) cache backend.

Original oslo.cache change:
https://review.opendev.org/c/openstack/oslo.cache/+/803716

Depends-On: https://review.opendev.org/807851
Change-Id: I5d7822101da301a6910f4da54569315d5c62f0b7
2022-01-27 20:08:54 +09:00
Zuul 2101af92df Merge "Allow customizing roles of the ceilometer service user" 2022-01-24 18:11:15 +00:00
Zuul 49dbd55f16 Merge "Do not test detail of keystone::auth::service_identity" 2022-01-24 17:29:20 +00:00
Takashi Kajinami 68d6a51931 Allow customizing roles of the ceilometer service user
This change introduces the capability to customize project-scoped and
system-scoped roles assigned to the ceilometer service user.

Change-Id: I6221fa85ad1fd0388c49f2ed49db1b6645dec3f5
2022-01-22 21:24:57 +09:00
Takashi Kajinami bcb4265b2b Do not test detail of keystone::auth::service_identity
... but test interfaces of that resource type. This helps us avoid
direct breakage caused by any change in puppet-keystone.

Change-Id: If1172824c6f446602632632ad749aed48fb5abfa
2022-01-22 21:24:44 +09:00
Takashi Kajinami 453cc40878 Avoid testing details of oslo::messaging(::*)
... so that any change in puppet-oslo would not directly break unit
tests.

Change-Id: I0579cd61806133d90b8214c7d671e6f651038d87
2022-01-21 23:49:50 +09:00
Zuul 8ff2661983 Merge "compute: Add support for tunables parameters" 2022-01-08 05:41:40 +00:00
Takashi Kajinami ddeb43dacc Add support for rootwrap.conf
Change-Id: I2a6f5c4c6a185c1bf71c4136dee425afdfc0ce56
2022-01-03 09:28:20 +09:00
Zuul 6bdf24f750 Merge "Remove deprecated ceilometer::agent::auth" 2022-01-01 21:25:18 +00:00
Takashi Kajinami e25e6bd483 Remove deprecated ceilometer::agent::auth
... because it was deprecated during Wallaby cycle[1].

[1] 9a5323ad18

Change-Id: I2fd3ecdefc0181dd47e23f125d887e86228a99b6
2021-12-27 21:26:59 +09:00
Takashi Kajinami f9bd8755ed Load libraries in a single place
This change refactors how the dependent libraries are loaded during
unit tests, and load the libraries in the base spec_helper to avoid
duplicate and redundant implementations.

Change-Id: I440118c0a63a19cde453c2ba5b73b23604d868cd
2021-12-27 11:58:53 +09:00