The uriescape function from puppetlabs-stdlib is no longer functional
in Puppet 8, because the URI.escape, which is internally used by that
function, is no longer available in Ruby 3+.
This replaces the function by own function to avoid the failure in
Puppet 8.
Closes-Bug: #2057860
Change-Id: I7b4db4c1e64416e20d8470cbff0b8497c6a0cfc9
The device_addresses option defaults to an empty list ([]), so we don't
have to replace an empty string value by the os_service_default fact.
Change-Id: I244e80ff8a5d393fb94e8a4e635401eb88936b39
The oslo.cache 3.7.0 release introduced some options for redis backend
and redis sentinel backend. This introduces for these parameters.
Depneds-on: https://review.opendev.org/910629
Change-Id: I0e7fa1e2134ec2cfcf1ae078e3fcf75b946dd699
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: Id4916244bdf4cd4ff2b45a45ec3e25fb0d115249
This introduces parameters to customize cert_file and key_file used for
tls connections in libvirt, in addition to ca_file and crl_file which
are currently supported.
Closes-Bug: #2055160
Change-Id: I597b4d61661a21470ed76ba259f276c2ceffc5ba
This introduces support for a few more migration_* options for QEMU,
which are used to determine the address/network used during migration.
Change-Id: I5cd32eeee428b0bef53a79f5d8bfe9900e64db22
Nova disables vTPM support by default and users have to set
the [libvirt] swtpm_enabled option to enable it. So the package
installation should be optional.
Note:
The parameter was introduced during this cycle[1] so we can safely
change the default value.
[1] e8a14766c8
Change-Id: I2e1791dcc64a844691a380686e695efeaaa0e343
The option defaults to False, so we don't need the explicit default and
can replace it by os_service_default fact.
Change-Id: Idd35189e279f3b6990f80c2eca80b7fa16c38e1f
This option was deprecated in 23.0.0 release[1], in favor of the native
TLS support by QEMU.
[1] 14071dfb118a32b26c6ebc068c66558d37466a5a
Change-Id: Ie563aa627a9cff5601570e38d981976f1bbfeaed
The dbus service is fundamental in CentOS/RHEL systems and we can
assume the service is always running.
Change-Id: I7775effaed6d7b476131675ee8ccdfd0a99cdff9
Restarting the dbus service is not necessary and is now triggering
a bug[1] in systemd-logind in CentOS Stream 9.
[1] https://issues.redhat.com/browse/RHEL-21004
Change-Id: Iae47044d7a58ba6316ff5f4b5ffdc01b03eddeba
The hard-coded value is exactly same as the default value in
libvirt-guests script.
Also, we don't have to replace the existing comment lines.
Change-Id: Iaf85adac38f5c62d961c63c8f6535c05e81ebaab
This addresses several problems in the existing implementation.
- The libvirt-guests service is available in Ubuntu and Debian.
There is no specific reason to support only RHEL and CentOS.
- The libvirt-guests package installs the client package in fact.
- Updating libvirt client does not need restarting nova services.
- Any change in config files should not restart the service. Restarting
the libvirt-guests service shutdown all guests in that host.
Change-Id: I2529925c439d7c22a6ada34e6da71bc29a79de12
This looses dependency using the new openstackclient tag, which
requires only packages actually related to openstack CLI.
Also, novaclient is not required by nova itself so the support
package tag can be removed.
Depends-on: https://review.opendev.org/899594
Change-Id: I6c43e57251d1520bc7622f0135a12f2a5bebaa62