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: If2cbbc392bd54d906c7a4f51f1c7cfca69463aaf
The sizelimit middleware was added to pipelines defined by the default
api-paste.ini file [1]. This adds support for the parameter of
the sizelimit middleware.
[1] c0a5572814e96f90d3d4b9ea0f10d35d88f5cd68
Change-Id: Ic3659225f4445c8b33e05157af76af4d11b45194
the validate_legacy function is marked for deprecation in
v9.0.0 from puppetlabs-stdlib.
Note that this change looses validation about pool ids. Alternatively
we can define a new data type to keep the validation, but these defined
types for pool-manager were already deprecated and useless and will be
removed in a near future release so we avoid that redundant step.
Change-Id: I85d7058bf379c6aa0e787b2f806f14aebf0693d3
This ensures the parameters used by if-else logic accept only boolean
values because non-boolean can result in unexpected behavior.
Change-Id: I703a2a64ce15ea6adf7cf233e635baf3ad963d31
... because the logic was added during the previous cycle[1] and
the option should be removed when a deployment is upgraded to Zed.
[1] f39567e17b
Change-Id: Iecc650fe94846a2f643ff2bf0b613026a1fb6570
This change makes designate::policy included by the api class
automatically to enable oslo.policy settings automatically in
the designate service.
Closes-Bug: #1980698
Change-Id: Ided1306c3c7154861c7c55257ea2bdd35fa6b370
The [service:api] max_header_line parameter has had no effect since
Train and it was effectively replaced by the [DEFAULT] max_header_line
parameter provided by oslo.service.
This updates the parameter definition accordingly.
Closes-Bug: #1967825
Change-Id: Ia969cc31b380950a87c0be0b911cc214b4219d82
This change introduces some parameters to enable verification of
the given project id while updating quota definitions.
Change-Id: If45d66fa2f0bc13a77fa95ba2995ae0118f79ac6
These parameters were deprecated and renamed during the previous cycle
because of the confusing behavior which we can hardly guess from their
names[1].
[1] b7cfb61a6d
Change-Id: I93aba377d20da0733f17199707197d4e31a1606f
Rename confusing service_ensure parameter.
Based on:
https://review.opendev.org/c/openstack/puppet-designate/+/773372
Added manage service parameter to worker.pp, so it will have the same
service management functionality as the rest of the services.
Change-Id: Ibe58d1395ee0b79af7535714ef5c9d2b26b52780
The following parameters are defined as "ListOpt"s which accept
comma-separated lists.
service:api/enabled_extensions_admin
service:api/enabled_extensions_v2
This change makes sure that list values are converted to string
properly to accept list values in the corresponding designate::api
parameters.
Change-Id: I6c18016bf33791e953f708b0f6abf772a434d74f
This change removes the enable_extentions_v1 parameter which was left
when support for v1 api was removed[1].
[1] a5d78457d6
Change-Id: I1f5ce5c70af4636c1176029bf76ffdfc83300871
The service_ensure parameter is supposed to accept service status and
ensure that service is in the given status, but in designate::api class
the parameter has been used wrongly.
This change renames the parameter to clearly show that the parameter
defines whether the service is managed or not.
Change-Id: I0984f74c5f6c863464be6599252d38cf1d321aa2
Designate has already removed its v1 API, so we should deprecate
and remove all parameters for it.
This change also fixes the outdated default value for keystone endpoint
url, which still includes '/v1'.
Change-Id: I645af15a9825ad000fc951d6c8334e715e645c8a
... because it was deprecated long ago in puppet-designate, and actual
implementation was already removed from designate[1].
[1] c318106c01b2b3976049f2c3ba0c8502a874242b
Change-Id: I121e06e5586d574aa63e4fd0b6553d414e227caf
The parameter name used to enforce package state is inconsistent across
manifests and modules. I settle on package_ensure since it's already
widely used.
Change-Id: I9868f954a032a0014b9304526daf5476b0922c5d
Closes-Bug: #1668732
This adds defined anchor points for external modules to hook into the
software install, config and service dependency chain. This allows
external modules to manage software installation (virtualenv,
containers, etc) and service management (pacemaker) without needing rely
on resources that may change or be renamed.
Change-Id: If9fa188b7dca47e6724a737d66e34c0c85668c36
The "api_host" and "api_port" options from group "service:api" have been deprecated
in favor of the listen option setting.
Change-Id: I1011e890adeb36e6908af7c183f74161d442a777
Since we are in ocata lets remove all old parameters in api
to configure the keystone_authtoken section
Change-Id: Idd1bc92b6a68e93911042784d9edb366f85547a6
This change updates our previous implementation of the keystone
authtoken setting to use the new designate::keystone::authtoken class to
do the configuration for the api.
Change-Id: Ib2a1d20d4e29353fc139835ce1c010a801506b9c
Closes-Bug: #1604463
Update designate::api to leverage keystone::resource::authtoken to
configure the keystone middleware. This patch also allows designate to
be able to leverage Keystone v3.
Some deprecations:
- designate::api::keystone_tenant is deprecated in favor of
designate::api::project_name.
- designate::api::keystone_user is deprecated in favor of
designate::api::username.
- designate::api::keystone_password is deprecated in favor of
designate::api::password.
- designate::api::keystone_host, designate::api::keystone_port,
designate::api::keystone_protocol are deprecated in favor of
designate::api::auth_uri and designate::api_auth_url.
- designate::api::keystone_memcached_servers is deprecated in favor of
designate::api::memcached_servers.
The patch is backward compatible.
Change-Id: I2cb4076b3e8679adac2722871f8391e750e8b4b0
Closes-Bug: #1604463
Added enable_api_admin to enable the (experimental) Admin API.
Added api_base_uri to set the base uri used in responses
Change-Id: I8f385fc3f97ba818954a614f9e8d767099f19057
This patch is aim to add the following params in to designate::api
- api_host
- api_port
Change-Id: I876e153aadecc44b82d3ed003212a7a23256a4ab
closes-Bug: #1523746
This patch introduces designate::generic_service which is already
exists in other modules to manage designate packages and services.
Change-Id: Ic05539fbbcfb88ed95f35d5eb5ca95b7f7141ed1
Close-Bug: #1486860
This adds defined anchor points for external modules to hook into the
software install, config and service dependency chain. This allows
external modules to manage software installation (virtualenv,
containers, etc) and service management (pacemaker) without needing rely
on resources that may change or be renamed.
Change-Id: I2b5707a0fe3dd72a56bbf1d840b4197bfa3012db
Some users wish to override the default package provider by their own.
Tag all packages with the 'openstack' to allow mass resource attributes
override using resource collectors.
Closes-bug: #1391209
Change-Id: Idcd296c754c4c66efcbdd1672c8367ce3539b268
Signed-off-by: Gael Chamoulaud <gchamoul@redhat.com>
This changes the puppet-lint requirement to 1.1.x, so that we can use
puppet-lint plugins. Most of these plugins are for 4.x compat, but some
just catch common errors.
Change-Id: I3b16baf063746024cb8dc89a931dae2f604258f6
This makes all package names and package ensure values customizable as
parameters. This is helpful for people who are not using vendor
provided packaging. This also normalizes the package ensure value to be
'present' in all case.
Change-Id: Iee186f753b2800e59addd3f38591675fd5af35ad
* Fix designate package name require, openstack-designate is the final
package name, designate-common name is override by params class.
* Remove designate-common dependency on central/sink/api services.
* Fix designate::api/keystone_user default user.
Change-Id: I441565d39ba5d425c6b93db071237a85f1eb4b1f
Currently secrets like rabbit_password or admin_password are laked
puppet logs when changed. This commit changes designate_*_config and
designate_*_ini types adding a new parameter that triggers obfuscation
the values in puppet logs.
Change-Id: I54e7c0bb27e46928db1a7f0125783c02d00d0e69
Closes-Bug: #1328448