728 Commits

Author SHA1 Message Date
Takashi Kajinami
c8810f340a Do not test detail of keystone::resource::service_identity
... but test interfaces of that resource type. This helps us avoid
direct breakage caused by any change in puppet-keystone.

Change-Id: Iae924a809e6aabeebb46a07494ae21d996ba93a4
2021-09-29 12:12:42 +09:00
Takashi Kajinami
1432fd8a8c Fix unit tests incompatible with puppetlab-stdlib 8.0.0
Since puppetlabs-stdlib 8.0.0, ensure_packages automatically translates
'ensure => present'  to 'ensure => installed' and that translation
breaks existing assertions in unit tests.

Change-Id: I23d6e6d23b5f4aa691731daea6b0318ec46c5610
2021-09-24 00:29:28 +09:00
ZhongShengping
d20eb43ec8 Add watch_log_file option
Add support for Using logging handler designed to watch file system.

Change-Id: I86fe2ea8facfe5363ffb343978c3db20620599e1
Closes-Bug: #1943212
2021-09-14 16:02:08 +08:00
Takashi Kajinami
fab7f496da Allow purging policy files
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: I8e84349dd75908d22ba9cc570b633f2f57cff1bd
2021-09-04 21:56:22 +09:00
Zuul
d54167cbc2 Merge "Do not test service_user parameters directly" 2021-09-02 19:23:35 +00:00
Zuul
654af68ad4 Merge "Do not test authtoken parameters directly" 2021-09-02 19:23:33 +00:00
Zuul
ecaf65298c Merge "Use a 'params' hash for authtoken parameters" 2021-09-02 19:23:32 +00:00
Takashi Kajinami
688e39654c Do not test service_user parameters directly
The service_user parameters are not managed directly but managed by
the keystone::resource::service_user resource type. Thus we should
avoid testing parameters directly otherwise any change in the resource
type can cause test failures.

Change-Id: Idc1324b4eb683a0ec0e57e20bca25238149adb0f
2021-08-27 22:11:18 +09:00
Takashi Kajinami
1ae066c60b Do not test authtoken parameters directly
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: I04286fac26f1f8787a3eb1304fc689d304ba5d50
2021-08-27 06:31:33 +09:00
Takashi Kajinami
c65c7c3782 Use a 'params' hash for authtoken parameters
This change adds the 'params' hash in authtoken class, to implement
the same functionality as the one recently introduced into
puppet-nova[1].

[1] 5c38281e1b698f157f03bf1815733277c541c30b

Change-Id: Id291959862bfe82cc1800e9bc7759b52006a10b7
2021-08-26 13:24:10 +00:00
Takashi Kajinami
40885d15bb Clean up v2 API support
This change removes the remaining parameters for v2 API support, which
were deprecated during the previous Wallaby cycle.
Also, enable_v3_api parameter is deprecated because the parameter was
deprecated in cinder[1] as part of removal of volume v2 API.

[1] e05b261af7dcd24096b229860df65dff1d385910

Change-Id: I20e6952d215d4d1886f09021a12d6c57c59a5d66
2021-08-18 14:15:08 +00:00
Zuul
28edb4ebad Merge "Remove deprecated support for GlusterFS volume driver" 2021-07-12 18:15:13 +00:00
Takashi Kajinami
c7719d1015 Remove deprecated support for GlusterFS volume driver
... because it was deprecated during Victoria cycle[1] and driver
code was already removed from cinder.

[1] f3bba521d32ffbdfa2dcfbf7ee58567b1f02f14e

Change-Id: Id9d97fbc2020f97244686777fb16071bbc15863d
2021-07-11 16:11:59 +09:00
Takashi Kajinami
75949efb1d Remove support for the Veritas HyperScale Driver
... because it was deprecated during Victoria cycle[1] and driver
code was already removed from cinder.

[1] 0da2b260eb6961222a23f7728c41593bcf349030

Change-Id: I9af225f714bf2df6b4c579ff12392d5c30769cb1
2021-07-07 03:52:15 +00:00
Zuul
e42724cc89 Merge "Add support for [cors] options" 2021-07-04 22:02:40 +00:00
Takashi Kajinami
a432ad63b4 Add support for [cors] options
Change-Id: Ide37b4ef6a03a673976df5c0e070fba0f867b524
2021-07-01 18:22:24 +09:00
Tobias Urdin
5e86972b77 Add backup_swift_create_storage_policy to cinder::backup::swift
Adds the backup_swift_create_storage_policy config option
that was introduced in [1].

[1] https://review.opendev.org/c/openstack/cinder/+/761113

Change-Id: I26398350ab24b60603563b025319165dcd02c669
2021-06-29 14:16:27 +02:00
Zuul
d9ccac4f21 Merge "Use oslo::coordination to manage coordination parameters" 2021-06-28 14:16:53 +00:00
Zuul
a36d428df6 Merge "Pure Storage driver: Add support for pure_host_personality" 2021-06-28 10:47:18 +00:00
Zuul
7491fc94a8 Merge "Fix a wrong spec file name" 2021-06-28 07:52:03 +00:00
Takashi Kajinami
2c869fdeb1 Fix a wrong spec file name
All spec files should be named as *_spec.rb otherwise these files are
ignored.

Change-Id: Id00162678c1191489aafec637d980a8913e9feb7
2021-06-28 09:52:42 +09:00
Zuul
10215f63b8 Merge "Fix ignored unit test case of cinder::backend::pure" 2021-06-24 19:35:54 +00:00
Zuul
74ca27890b Merge "Add support for MultiStrOpt" 2021-06-23 18:51:38 +00:00
Takashi Kajinami
714c56b2a0 Pure Storage driver: Add support for pure_host_personality
This change introduces support for the pure_host_personality parameter
in Pure Storage volume driver.

Change-Id: I933ccfbd779719056ec05196c3d4165c6882f03b
2021-06-23 22:43:45 +09:00
Takashi Kajinami
bf00282852 Fix ignored unit test case of cinder::backend::pure
Change-Id: I7b540f9d1f604fc8c63e9cc7cbe5edfff82a055d
2021-06-23 22:42:14 +09:00
Zuul
d73ebdd432 Merge "Add support for cinder_internal_tenant_* parameters" 2021-06-20 17:09:18 +00:00
Erik Berg
62899eba67 Allow for '-' in volume type property values
There's no validation going on (yet) in cinder as to which characters
are allowed in properties. So this change alters the regex to cast a
wide net catching '-' and other possible valid variations.

Also update unit-test to cover this scenario.

Closes-Bug: #1932272
Change-Id: I564528663a3d2f83136b98ed46e09e373a9ad6de
2021-06-19 02:37:34 +00:00
Takashi Kajinami
a19b83c650 Add support for cinder_internal_tenant_* parameters
Change-Id: I2db1eb941615e5b773096c7a0b9df3eb1f11ee24
2021-06-16 12:42:13 +09:00
Takashi Kajinami
24dbff6a5d Set fixture paths for unit tests
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: I40391cecb877d948a30a480b080a10895b431d0b
2021-06-02 13:54:22 +09:00
Thomas Goirand
6874113187 Get rid of the $pyvers variable
Since everyone has switched to Python3, it's time for the removal of the
$pyvers variable.

Change-Id: I152451ef1be492bfa96868fdc1d89f4eabce3371
2021-05-21 11:09:06 +02:00
Takashi Kajinami
9078af8559 Use oslo::coordination to manage coordination parameters
This change replaces current implementation to manage coordination
parameters by the base class in puppet-oslo. With this change
the required backend package is installed according to the backend
driver used.

Depends-on: https://review.opendev.org/791628
Change-Id: I210f207d4f05862c98a2ef6d04162880d1cfc4ae
2021-05-18 10:05:02 +09:00
Takashi Kajinami
e73bf6a864 Deprecate the powerstore_appliances parameter
... because it was already deprecated[1] in cinder and has no effect.

[1] f328341ed0eb920a6471f7c65e06b27c3f719fbe
Change-Id: Ic25a2a88ccfc5eddf180d44cb109d69378c5277f
2021-05-16 21:27:22 +09:00
Zuul
496eb152f9 Merge "Add support for the use_forwarded_for parameter" 2021-05-13 08:20:46 +00:00
Zuul
00d6526168 Merge "Create independent classes for key managers" 2021-05-11 21:42:44 +00:00
Takashi Kajinami
6dc69ab67f Create independent classes for key managers
This change introduces the new independent classes to manage options
for key managers, which are implemented in the castellan library.

Depends-on: https://review.opendev.org/772676
Change-Id: I5fc9b4e2db56daf998294245465e8c5fcbd8a061
2021-05-11 12:35:28 +09:00
Takashi Kajinami
e63efab415 Support more Glance parmaeters
Change-Id: I1237cd3ce7490d4027e539ec0551ead0939dbb98
2021-05-11 12:31:30 +09:00
Takashi Kajinami
defeaf5370 Improve unit test coverage of cinder::glance
Currently unit tests of cinder::glance only test defaults and doesn't
have enough use case to test all parameters.
This change adds some more test cases so that all parameters are
properly tested.

Change-Id: I6e602e84c778413f4f655d732e012a95eedef184
2021-05-11 12:31:30 +09:00
Takashi Kajinami
2b17605c6a Update default of cinder::glance::glance_api_servers
This change ensures that the glance_api_servers is removed by default.
The current default(undef) makes an empty value set in config file,
which is effectively same as the service default(None).

Change-Id: If287c7f725d2d4c8e0e31e1267487fddce1fd510
2021-05-11 12:31:30 +09:00
Takashi Kajinami
fe5be161ef Add support for the use_forwarded_for parameter
Change-Id: If9f511b7a94501ad910a282c6ccd84f721d6fd64
2021-05-11 08:21:22 +09:00
Takashi Kajinami
8e233cfb92 Deprecate the redundant cinder::backend_host parameter
The cinder::backend_host parameter is redundant, because it is not used
in the cinder class where the parameter is defined but is used to only
override the cinder::backends::backend_host parameter.

Change-Id: Ife5f8bdcfd4e982b7ffa5532b2a23d48c65d501f
2021-05-06 08:22:04 +09:00
Takashi Kajinami
870ff62655 Add support for MultiStrOpt
This replaces the provider implementation of cinder_config type so that
MultiStrOpt, which is used by several options like
 - oslo_messaging_notifications/driver
 - oslo_policy/policy_dirs
is handled correctly.

Change-Id: If052cde4da84970df4af657136060532ccb422f7
2021-05-05 10:45:22 +09:00
Zuul
4e465db2aa Merge "Remove support for Nexenta Edge Driver" 2021-04-29 11:26:59 +00:00
Zuul
8244283aea Merge "Remove support for HPE Lefthand driver" 2021-04-23 18:38:36 +00:00
Takashi Kajinami
09aa718089 Remove support for Nexenta Edge Driver
... because it was deprecated during Victoria cycle[1].

[1] 38e35eddacf39be7a9474da6a622bda69851a5fb

Change-Id: Ia2f7aa9d99557038147ef5573bcc2a96084cfd12
2021-04-23 00:14:51 +09:00
Takashi Kajinami
51ef949abf Remove support for HPE Lefthand driver
... because it was deprecated during the previous cycle[1], and
the driver implementation has been removed from cinder.

[1] c5927c42afa8f73bcd7890a28446c1308901cfbc

Change-Id: I86a4e6e14f39a5bdb70d5443764f946b3f26130e
2021-04-22 20:13:57 +09:00
Zuul
3fc60e2b9e Merge "Improve cinder_type properties regex" 2021-04-21 18:01:01 +00:00
Zuul
f4d779cbaa Merge "Remove deprecated cinder::backend::dellemc_xtremio_iscsi" 2021-04-14 18:15:52 +00:00
Ivan Pchelintsev
92019855a7 Remove deprecated cinder::backend::dellemc_xtremio_iscsi
cinder::backend::dellemc_xtremio_iscsi is deprecated and should
have been removed in Victoria release.

Change-Id: I7c0473881b5065fdfacda8a327f02f12074b9b96
2021-04-13 11:36:10 +03:00
Thomas Goirand
003b8d6ab6 Add support for cinder_api_uwsgi_config in Debian
This patch is adding the configuration of the number of workers,
threads, and the size of the listen queue in Debian, which uses
uwsgi to run Cinder API. Therefore, this patch adds a new
cinder_api_uwsgi_config provider as well as a new
cinder::wsgi::uwsgi class.

Change-Id: I6f4c6a51f8e33616069411a926a63ba2b388e25d
2021-04-10 22:40:57 +02:00
Tobias Urdin
a032bec7d6 Improve cinder_type properties regex
This changes the regex for properties managed with
the cinder_type resource so that it supports the
format Cinder needs for multiattach and similar.

  multiattach=<is> True

This patch is needed so that cinder_type resources
can set the above.

See regex testing here [1]. Needs to be backported
to older versions so that cinder_type can be
properly used there as well.

[1] https://regexr.com/5imqk

Change-Id: If08fce9443281e65636aaaa99e3139732435f6f7
2021-04-08 21:33:15 +02:00