This refactors resource dependencies to improve the following points.
- Avoid unnecessary dependencies across services. For example aodh
service does not require cinder db.
- Restart only api service when config files like paste.ini, which
are used only be api service is changed.
Change-Id: I6a4c65e81b97235d0ce8a142245927c92ab6e48d
This looses dependency using the new openstackclient tag, which
requires only packages actually related to openstack CLI.
Also, glanceclient is not required by glance itself so the support
package tag can be removed.
Depends-on: https://review.opendev.org/899594
Change-Id: I50f0775a2f20b9b4f3aca51840cb4ab48ab14120
The glance-cache.conf is used by the glance-cache-prefetcher command
which requires access to backend store. Thus all store options should
be added to the glance-cache.conf file so that backend store can be
loaded properly.
Closes-Bug: #2034893
Change-Id: I8f0b2be8b138d267364f20ff81876902b225d29e
Setting an empty string('') to [paste_deploy] flavor results in
the exactly same behavior as removing the [paste_deploy] flavor option.
Thus we don't really need the special treatment for an empty string.
Change-Id: Ib50da09ac6fd2499fc922d5d0e1115cd600ecb69
the validate_legacy function is marked for deprecation in
v9.0.0 from puppetlabs-stdlib.
Note that this change removes the pattern validation about [DEFAULT]
pipeline. The existing pattern check is incomplete, because of
the following two reasons.
- the pattern does not guarantee that the requested pipeline is
really supported or not
- the pattern accepts a+b+c but the existing pipelines in glance
does not have any entries with this pattern
Change-Id: I156d9a3a30ba6c523a0b16a14bf8a1839b61fb78
... instead of using the actual package name. This helps users to
identify the resource name more easily.
Change-Id: Icc6a27685f3b3747b4e0562eb8066aa3cce5516c
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: I55a19eb7ed16f87f51ca472e658fc845b1a86dac
The rbd backend implementation of glance_store requires not only
python3-rbd but also python3-rados. In CentOS the python3-rados package
is installed as a dependency of python3-rbd, but in Debian and Ubuntu
that dependency is not defined and we should install python3-rados
explicitly.
This fixes the missing python3-rados package in Ubuntu, by using
the meta python3-ceph package which installs all python libraries.
Closes-Bug: #2002934
Change-Id: I620c607f09e98f1ce1d7ba34d1b797dbef8402ad
When running glance-api behind a proxy or load balancer that hides
glance behind an extended base URL, it is necessary to provide
information about the base URL to glance-api to ensure that clients
know where to pass subsequent requests.
Change-Id: I9bebee01c2580f0500a069f5ab57ac2a8c329389
The apache class is already included by the openstacklib::wsgi::apache
defined types and the apache::wsgi class and the apache::ssl class is
automatically included by the apache::vhost defined type.
This removes the redundant inclusion of these classes.
Change-Id: If132d46ec1929ed56e16a9b615010ea0535282ba
These were deprecated during Xena cycle in favor of the new separate
classes to customize key manager[1].
[1] bc7651fa5f
Change-Id: Id3395fd58cbdd7e686a08c8369cd38c3e322628b
The puppetlabs-apache module is enforcing more strict data type
validation[1].
This change updates the default values to adapt to that change.
[1] f41251e336
Closes-Bug: #1983300
Depends-on: https://review.opendev.org/851652
Change-Id: I652d82c449371bf4df73d1384ddf1633a76224e9
This reverts commit ae04e3ac34.
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: I6cfea1ca8dbf7633ea6b3805666237ec43533b4b
This change introduces a basic implementation to run glance-api as
an application managed by httpd+mod_wsgi, instead of a standalone
eventlet server.
Change-Id: I9421ec891c535503ff61ad6107a1935a32b4b263
... so that users can customize that option using the native interface.
In TripleO we append additional arguments(to load glance-cache.conf)
and we need to set the option explicitly to ensure the correct file
path is used.
Also, the pipeline parameter is confusing because of its naming
different from the actual parameter in Glance, so this change renames
the parameter as well.
Change-Id: I3d6ce07f44ae526207a30d4b57c2fec7cd6fa3ca
In some test cases for glance::api, we set most of the parameters by
actual default values explicitly, instead of letting the class to set
these by the defaults. This is redundant and makes it hard to find out
which parameters are actually set during the tests.
This change removes the redundant "default" values and makes test cases
pass minimum set of the parameters.
Change-Id: I7cfc4a17e0e42d1929098673b654d72b11a8e86b
A recent change in os-brick [1], which is used by cinder backends,
requires external file locks. This patch adds support for configuring
the lock path, with a default value that matches the corresponding
lock paths used by other services that use os-brick (namely cinder
and nova).
[1] I6f7f7d19540361204d4ae3ead2bd6dcddb8fcd68
Closes-Bug: #1980539
Change-Id: I5ea6766656f4227a1c514777c03bbf78d4ac59cd
This change introduces support for the following two reserved backends,
which were added by [1].
[1] 6dba83ba3a7c047df282f2edb7217548f43d6251
Change-Id: I9e809f5ba25516b0974015ea5259d1ab7f5bb1ca
... so that users can avoid executing the same cron command at the same
time in multiple nodes.
Technically this can be done without this change by setting a different
cron job schedule in each node but it requires complicated parameter
setting.
Change-Id: I596879d5d46f23e54fe344d5bf93a57837a6d287
The parameter was already deprecated, but it is still required in
some setup (eg. to leverage in-storage copy in a deployment with Ceph
used for cinder/glance).
Considering the parameter is still valid, this change ensures it is
removed by default, so that old value is properly removed from config
file.
Change-Id: I2477d5e271b017ee12546c67f4c3f3f9be89c062
Glance removed native ssl support during Ussuri cycle[1]. This change
deprecates the remaining parameters to set up the feature, so that
we can remove these ineffective parameters in a future release.
[1] 06b2465f598c014451c244d47c2ded4f1e2c6947
Closes-Bug: #1965776
Change-Id: Ie4a984a4b7136304425588856eabff2e60d1cbba
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: I3c85c7e0976ff353090cc2b7737112cf119b6f38
Packages managed by Puppet OpenStack modules should have 'openstack'
tag. This change fixes one package resource without that tag.
Also, this change replaces ensure_resource by ensure_packages, so that
more specific function for package resource is used.
Change-Id: I281c7f4504f7205e7f23458364288179da89d445
This change allows usage of system scope credentials in addition to
project scope credentials, to use service user token when accessing
Barbican API.
Depends-on: https://review.opendev.org/823883
Change-Id: I836f8ca32ed8c3352e05f6ab805b6d8f311c061d