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: I789537557b60d9a408cec9fde65ac9ddc1cb743b
... which were added recently to ironic. Note that the specific default
value is defined for rbac_service_project_name, because in Puppet
OpenStack modules 'services' is widely used for service project instead
of 'service', which is the default in ironic.
Depends-on: https://review.opendev.org/907148
Change-Id: I28485f7946d5a040a039e6c200de264a840ba7c4
Some parameters are used to determine file path (or name) so should
accept only absolute path (or a non-empty string).
Change-Id: I74a137693c18a187d12cd53462c36759828c5b5b
The pxe_timeout parameter should be set to an integer value (or
a string representing an integer) and can't be set to
the os_service_default fact.
Closes-Bug: #2058229
Change-Id: I7b04e5586c82fc528f960beddfe50a7b355f6580
The ironic-dbsync command by default load service config files.
Removing the override allows us to use additional paths such as
ironic.conf.d in the future.
Note that we still need the hard-coded path for ironic-inspector-dbsync
because the service config file is installed at a path different from
the default one oslo.config attempts to load (the file name should be
"ironic-inspector.conf")
Change-Id: I07e066d240dbe4da3eb2fce9040685cc15997ca0
Currently even ironic-inspector services use the ironic keystone user
though the ironic-inspector keystone user is created. This adds
a warning message to make users aware of the future change to use
the own service user.
Change-Id: I62b7cf88496a811f4068244d7a3fb563a53aa1d6
... because these parameters are actually required for authentication.
Also, fix the inconsistent default values of auth_url parameter.
Depends-on: https://review.opendev.org/903851
Change-Id: I41cdf7f04b8ced4f591d7b1ceb5b1138462ff4fd
... because these parameters are actually required for authentication.
Also, fix the inconsistent default values of auth_url parameter.
Change-Id: Ic2c0e5d51825b7baa5c11be5c8bbf842f60b2f22
The password parameter is not really optional. This makes it
a required parameter to give more sensible validation error.
Change-Id: I38d4f000905ac01ac1cf50bfbc48cb30af609cf7
... using the operating system name. This may allow the module to work
with untested distributions such as Rocky.
Change-Id: Idd73ccd4d5366ccbb9373d7e3bfcba120fc238bb
The ironic-inspector service can be deployed using separate api service
and conductor service. This introduces the initial support for that
deployment pattern. Note that the non-standalone mode is supported in
only RHEL and CentOS. We may add Debian support later.
Change-Id: Ic53d454841252cfb6802cd7856faaab2435b984b
The ironic-inspector service has api workers and rpc workers and these
communicate over rabbitmq.
Closes-Bug: #2043931
Change-Id: I759c856d9f19c970390a10ca3b19ba0458190203
The new openstackclient tag was added so that we can get all resources
about openstack CLI more easily. This adds this tag to ironicclient
because the plugin provides some sub-commands.
This also removes the support package tag because the client package is
not at all used by ironic itself.
In addition, openstackclient is needed to query keystone api when
transform in ironic_config is used.
Depends-on: https://review.opendev.org/899594
Change-Id: I46dcf0f8e7f9b72d27317d310bffdd25653a25cf
... so that the credentials in clouds.yaml file can be used if
available instead of openrc.
Also, the transformation from project name to project uuid is still
needed so the transform_to method has been undeprecated.
Change-Id: I0e0d42d92e8272f3f7faf809e07e599805bbedaa
This introduces a new class to manage [pxe] options, which are used by
the iptables PXE filter options.
The configuration of [pxe] dnsmasq_interface option will be moved to
the new class, following the existing implementation to configure
the [dnsmasq_pxe_filter] dhcp_hostsdir option.
Change-Id: If0ec9eecb70d9976de76899fc115fc9facc75fdd
The dnsmasq_dhcp_hostsdir should be set to a specific path when dnsmasq
PXE filter is used. This adds a warning message to inform possible
mis-configuration.
Note that dhcp hostsdir is not needed when the other PXE filter such
as iptables PXE filter is used, thus we avoid updating the default
value.
Closes-Bug: #2042731
Change-Id: If127bd4661709835ab0d9d39e0341367fdef51b7
This [pxe] ip_version parameter was deprecated in ironic during Xena
cycle and has been unused by in-tree drivers since Ussuri[1].
This also removes cleanup of [pxe] ipxe_enabled which was added a few
cycles back.
[1] b2a249d186b77def6cd449d34fd600520c159a50
Change-Id: I67ecf2e0bebc93ad20733677662c51ad31619f01