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
... because this option was deprecated as part of deprecation of
glance-scrubber.
Depends-on: https://review.opendev.org/910042
Change-Id: I9862d999025dd63dcbac4f33faeaad4fa6c1ada9
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
Using legacy backend class (eg glance::backend::file) with multi_store
set to false(default) has been broken, because of duplicate declaration
of [glance_store] default_store option.
This fixes that problem.
Change-Id: Ibb12bdb8333fc91e2dc95c2c2cc4de79f2c82cc1
The purge_config parameters only accept boolean values. This enforces
that using the typed parameters.
Change-Id: I68260f896eb4b94661685a2ebf76500b0cf409a6
The default_store_real variable is not defined in some code paths and
that is causing the following error when Puppet 8 is used.
```
Evaluation Error: Unknown variable: 'default_store_real'.
```
Change-Id: I8220b646de922e44e08db49817b473789e2742f8
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
This ensures the parameters used by if-else logic accept only boolean
values because non-boolean can result in unexpected behavior.
Change-Id: Ie3fce5786d44be38fad176442a0a0f9b24faab77
This parameter was deprecated during the previous cycle[1] and has had
no effect since then.
[1] 89857141d1
Change-Id: I21e59483cef2847b02783a13072745a9f434a318
This parameter was deprecated during the past cycle because it is not
used by glance-api actually[1], and has had no effect since then.
[1] bd6fb8f175
Change-Id: Ic4f2ed44fa4e7263d2e4c6d3ae3e255fdf798aa2
These parameters were deprecated during the previous cycle by [1] and
have had no effect. These parameters are not really used by recent
glance.
[1] 2a2f349d8a
Depends-on: https://review.opendev.org/865924
Related-Bug: #1965776
Change-Id: Ia24b57966f6d8aee5cfa56cfe3496f1d81924ea1
... service validation was deprecated during Yoga cycle by [1].
The parameters were left for interface compatibility but have had no
effect since then.
[1] 3d5c217596daeafe7620273ab7a0b5dd9ee2d194
Change-Id: I4b2fa2f250576eb811d66c4925f0b9a5cf6bb3ee
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
These were deprecated during Xena cycle in favor of the new separate
classes to customize key manager[1].
[1] bc7651fa5f
Change-Id: Id3395fd58cbdd7e686a08c8369cd38c3e322628b
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
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
... because these options are used only when legacy store is used,
which has been deprecated in favor of multi store.
Change-Id: Ib5cef2869b241d26bbe2fc0a71b05942e03fe512
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 scrub_time parameter is not used by glance-api. The parameter is
used by glance-scrubber which is not yet supported by puppet-glance,
and should be added to glance-scrubber.conf instead.
This change deprecates the parameter because it has been ineffective.
Change-Id: I12ed8285a2a048880b5a263958f0aba62128ca58
Following the past deprecation in puppet-keystone[1], this change
deprecates service validation for glance-api.
[1] 3c95205e0253895e477b9135c10f5175d4166cfc
Change-Id: Icf44d49d68a345b0ac910918a3b4208e1c23aec9
Adding 'use_keystone_limits' parameter, which will allow Glance to
retrieve limits set in keystone for resource consumption and enforce
them against API users.
Depends-on: https://review.opendev.org/812766
Change-Id: I5977d0cd62454d70028a25aaa6206b490884f325
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: Id54b87472ae67a9005276e52de9dd68172e22eff
Glance has added support for project persona of secure RBAC as
an experimental feature where 'enforce_secure_rbac' has been introduced.
Adding support of 'enforce_secure_rbac' parameter here.
Partially Implements: blueprint secure-rbac
Change-Id: I1db7fa2694bc9a448a47e435cfd95264504086c6
This change drops remaining items about support for glance-registry.
These were deprecated by the removal patch[1] which was merged during
the previous cycle thus we are now ready to drop them.
[1] 11599ba429
Change-Id: If2d371a372cf4d95d84e3d71fb4ce272565ac659
To support distributed image import new configuration parameter
'worker_self_reference_url' has been added.
Adding support of 'worker_self_reference_url' parameter.
Change-Id: Ib14a126bde1ecc2e55b0f5bace06a81c9cfa1011