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: I0a3be947704009a723bfc3181c5277afa12ec933
This reverts commit e3f44b4ef5.
Reason for revert:
The original change was wrong because ec2api uses own ssl options
instead of ones imported by oslo.service.
Conflicts:
manifests/api.pp
Closes-Bug: #2011569
Change-Id: I592fc1da6b364f017a7892a30ff42a61d86f4d9f
The option has been managed by the underlying puppet-oslo module but
has not been configurable. This introduces the parameter to customize
the option.
Change-Id: I2382514189617edb9454c3d9e8b5f15de9c96237
This parameter was deprecated during the previous cycle by [1] because
the actual parameter no longer exists.
[1] a72e6fe949
Change-Id: Ieb53368b269f3323f54a0d14caf00dcf9b136831
This reverts commit c12186f850.
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: I3f4b076bcd4754adfce0fba4117cf1bf26a1649f
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: I773e1982437b8d3e0fb8cda26bf35f0e48521481
The ssl_* parameters in the [DEFAULT] sections were deprecated and
the [ssl] parameters should be used instead[1].
[1] 2e2940d1e17706b9d6367ff1783bde9d41891bfa
Change-Id: If5ee27626b778bda4d8cb47162691f3cced35955
... because the parameter was already removed[1].
The ssl_ca_file parameter is left because it is still used by
the oslo_service library.
[1] 5fc752ca6fd1588cc04f3073d1c300d985581041
Change-Id: Ia1035589c68790239c6619f4f3f9b33d7b5ae66a
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
Co-Authored-By: Hervé Beraud <hberaud@redhat.com>
Depends-On: https://review.opendev.org/826869
Change-Id: I9a7a3ae77d2445cec5470e8bfc9b9b651d37f783
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: Ifbdbb09a4bf036421bd2bbe0eedb8dc929423738
This replaces the provider implementation of ec2api_config type so
that MultiStrOpt, which is used by several options like
- oslo_messaging_notifications/driver
- oslo_policy/policy_dirs
are handled correctly.
Change-Id: If2441f4bf68b8e43a072d979a1c34a60c9c48bde
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
Co-Authored-By: Grzegorz Grasza <xek@redhat.com>
Depends-On: https://review.opendev.org/807851
Change-Id: I6156abd69160113113d8e239f7204ba9274a1ee7
This change is the first step to support secure RBAC and allows usage
of system scope credentials for Keystone API request.
This change covers the following two items.
- assignment of system scope roles to system user
- credential parameters for authtoken middleware
Depends-on: https://review.opendev.org/804325
Change-Id: I3858399136f886841c14c2880918cb8e6ee86756
The authtoken parameters are not managed directly but managed by
the keystone::resource::authtoken class. Thus we should avoid testing
parameters directly otherwise any change in the resource type can
cause test failures.
Change-Id: I57471c563a4a72ada8f8611b94a8e2a1e7b2df3e
Ec2Api supports caching metadata using the olso.cache library[1]. This
change introduces the new parameter/class to configure that usage.
[1] 7939ce17e4e39cb7b0cd1a36d6626258f06dcc70
Change-Id: Idf122ad2744acdd020de3cedbd0fe099d5997c4a
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: I96e33813aaf07021b15d7ac47a9bc2f9d8308607
... but test interfaces of that resource type. This helps us avoid
direct breakage caused by any change in puppet-keystone.
Change-Id: If806a05db2ce8b3c059aeb6beb6096c34523d73f
This change introduces the new purge_config parameter to the policy
class so that any policy rules not managed by puppet manifests can be
cleared.
Co-Authored-By: Martin Schuppert <mschuppert@redhat.com>
Depends-On: https://review.opendev.org/802305
Change-Id: Ia4e3b30c1ad7b9aaae2bd9377a539c77899c4f47
This change defines manifest_dir and module_path expicitly in unit
tests so that modules installed under fixtures directory is properly
loaded.
Closes-Bug: #1930403
Change-Id: I64316e1bb130c9bba96cfa77ee2a2971363dcd97
This change fixes the following error which curently appears in unit
tests on Ubuntu Focal.
Failure/Error: let(:facts) { facts.merge! @default_facts }
TypeError:
no implicit conversion of nil into Hash
Change-Id: I2911d57e7f07cd07da33e816e998ef3c4d8300b5
As Openstack projects continue to have longer database migration
chains, the Puppet default timeout of 300 seconds for an execution
is becoming too short a duration on some hardware, leading to timeouts.
As projects continue to add more migration scripts without pruning
the base, timeouts will continue to become more frequent unless
this time can be expanded.
Change-Id: Ia353f278db9426d29617d6c274fef0c8b71c53e1
Closes-Bug: #1904962
This patch adds support for [keystone_authtoken] interface parameter,
so that operators can define which endpoint should be used by authtoken
middleware.
Change-Id: I7cdb181057ad8a126fbbdd5d0862827a2ed28062
... and migrate it to openstacklib so that all logics about database
configuration are implemented in one common place.
Depends-on: https://review.opendev.org/#/c/728595/
Change-Id: I900142c35e8fb20d3279f607cdd80ad53157531c