Commit Graph

1998 Commits

Author SHA1 Message Date
Joel Capitao 3aaa45b6db Get rid of removed class parameter of nova::api
This parameter was deprecated and have had no affect since [1],
and removed since [2].

[1] https://review.opendev.org/c/openstack/puppet-nova/+/582621
[2] https://review.opendev.org/c/openstack/puppet-nova/+/916325

Change-Id: I48bfb0c9bc77366e8a1d30f8e4848088f6401178
2024-04-24 14:41:20 +02:00
Takashi Kajinami 4c7254d282 Nova: Enable service token
Now service token is required so that nova can update volume
attachments in cinder.

Resolves: rhbz#2228733
Change-Id: I2e873f82245ff4b1e64f2d3fbb2589e03abda8d9
2024-04-16 10:32:22 +00:00
Zuul 3850a45069 Merge "Clean up unnecessary workaround related to puppetlabs-apache" 2024-03-06 19:38:08 +00:00
Zuul e6e8c93d95 Merge "Remove sahara support" 2024-03-06 19:38:07 +00:00
Takashi Kajinami 9af0a8e844 Clean up unnecessary workaround related to puppetlabs-apache
Since v5.6.0, mod_packages[1] and mod_libs[2] are automatically set
to the appropriate value in CentOS >= 8, so we don't need to set these
parameters explicitly.

Also, the mod_ssl package is now installed automatically when ssl is
enabled in vhost.

[1] https://github.com/puppetlabs/puppetlabs-apache/commit/d346b76d1
[2] https://github.com/puppetlabs/puppetlabs-apache/commit/f31e392a6

Change-Id: I28feb2b5d4cde4c430806e5fc8b9b7c98695214a
2024-03-05 04:52:43 +00:00
Zuul 33770d44b5 Merge "Remove reference to monolithic puppet-openstack" 2024-02-29 09:21:23 +00:00
Zuul 7be707f051 Merge "Prohibit retired/unmaintained network plugins" 2024-02-29 09:20:38 +00:00
Alfredo Moralejo 08a27a2ef5 Add enable-chassis-as-gw option for OVN CMS cluster
After recent neutron change [1] we need to add option
enable-chassis-as-gw so that the chassis acts as gateway.

I am also implementing workaround for known issue on mariadb [2].

[1] https://review.opendev.org/c/openstack/neutron/+/908325
[2] https://issues.redhat.com/browse/RDO-241

Change-Id: Ie6a9a9287f64256999c05f7a3ab273c436f1cf54
2024-02-22 18:17:58 +01:00
Takashi Kajinami baf74924e7 Prohibit retired/unmaintained network plugins
- The networking-hyperv plugin[1] was retired because the Winstackers
  project was retired.

- The ncs plugin was already removed[2]

- The cisco_nexues plugin still exists but the networking-cisco project
  is no longer maintained and has not been updated for 5 years[3].

[1] https://github.com/openstack-archive/networking-hyperv
[2] 31e4880299
[3] https://opendev.org/x/networking-cisco

Change-Id: Ide554b21cfb029cd863a348dc13ef5328ac83906
2024-01-07 15:39:25 +09:00
Takashi Kajinami 892fec52d8 Remove sahara support
The sahara project was marked inactive[1] during this cycle. Health of
the project has not been recovered until the promised deadline (m-2)
and 2024.1 release won't be created for this project.

Change-Id: I15dca648cd84ae128171d70a194281eebe32bd8d
2024-01-07 15:30:43 +09:00
Takashi Kajinami a64a08ede4 Remove reference to monolithic puppet-openstack
The monolithic module[1] was retired in favor of separate modules per
service.

Also the puppet-openstack_extras module is not used.

[1] https://github.com/puppetlabs-toy-chest/puppetlabs-openstack

Change-Id: I42dbc51851b5635fb16b566a88e23bc8eed8fbe7
2024-01-07 15:27:46 +09:00
karolinku b00b07af7e The `action` attribute within the `firewall` type has been removed.
v7.0.0 of firwall module is introducing several changes, as described
in [1]. This commit is adjusting manifests to this change.

[1] 8ea5f45d00

Change-Id: I31a04a83e2844fa1ec53a27fccd15fb64cafacb0
2023-10-16 15:16:35 +02:00
Zuul bd7855ffca Merge "Heat: Use separate class to manage [trustee] options" 2023-04-11 12:03:39 +00:00
Joel Capitao 15bbe1f942 Move to project_ terminology used in current Keystone v3 API
tenant_* properties were deprecated long time ago with [1]
and removed recently with [2].

[1] https://review.opendev.org/c/openstack/puppet-neutron/+/828942
[2] https://review.opendev.org/c/openstack/puppet-neutron/+/877189

Change-Id: I8a324fdab89e474ad2a8e6df1560a5c16edbb866
2023-03-21 16:06:51 +01:00
Alfredo Moralejo bd1e892301 Enable snapshots in NFS backends and add nova section
Snapshots feature is disabled in nfs by default in cinder because of issues with old
versions of libvirt [1] which are not found in centos8 and centos9, which
are the versions supported since long time ago.

Also, we are missing the [nova] section in cinder.conf which is required
in some cinder operations, as snapshots with nfs backend.

[1] 2d77a7a87d

Change-Id: Ie70d0f242220b513bab37befedb9ab785e08d077
2023-02-27 15:35:25 +01:00
Joel Capitao 8726503615 Replace deprecated defined resource type of Manila
Since [1], 'manila::service_instance' has been replaced
by 'manila::backend::service_instance'.

We have also to remove the use of
'use_guestagent_template' as it was removed in [2]

Finally, since tox4 we need to set skipdist to False
in order to install the package (needed for pbr to
get the version). Previously with tox3, it was set to
True but it was a bug apprently.

[1] https://review.opendev.org/c/openstack/puppet-manila/+/873479
[2] https://review.opendev.org/c/openstack/puppet-trove/+/874367

Change-Id: I464675a5f78b1efcc3a277da4772dca5690dd842
2023-02-22 12:36:31 +01:00
Takashi Kajinami 261bc218e7 Heat: Use separate class to manage [trustee] options
This change ensures the heat::trustee class, which was introduced
during Yoga cycle[1], is used to manage the [trustee] options.

[1] 6e8b799ba8f8761616d52928e479a914ef899ea2

Change-Id: I8fc4504a76e9bb0725a60f3bcfab8355b4f52ac4
2022-12-05 16:14:55 +09:00
Alfredo Moralejo e8078a4f40 Remove enable_ssl from keystone.pp
It has been removed in puppet-keystone [1] and actually it has no effect
since some releases ago as the ssl termination is managed in the wsgi
server.

[1] https://review.opendev.org/c/openstack/puppet-keystone/+/864675

Change-Id: Iffb87bfda58e38809e244ea84eb925fe3aec3c25
2022-11-28 10:36:28 +01:00
Takashi Kajinami 1f804e515b Ceilometer: Remove redundant installation of pythonN-redis
The pythonN-redis is now indirectly required by the ceilometer-common
package (ceilmeter-common -> pythonN-tooz -> pythonN-redis) so we no
longer need to install the package explicitly.

Change-Id: I45b17579c2fcbd8f1f8027043b57ee3b14950831
2022-10-20 09:46:29 +09:00
Zuul 535aaf736d Merge "Establish order between redis and aodh and ceilometer" 2022-10-19 16:45:07 +00:00
Zuul 5f489446eb Merge "Fix dependency between redis and gnocchi services" 2022-10-18 17:21:32 +00:00
Zuul 034d10c8f6 Merge "Drop firewall rule for ceilometer-api" 2022-10-18 16:51:07 +00:00
Alfredo Moralejo 83ca93d962 Establish order between redis and aodh and ceilometer
Follow-up of [1]

[1] https://review.opendev.org/c/x/packstack/+/861757

Change-Id: I2438360b7ef1d051e46b69298b9ee6ca462a1afd
2022-10-18 16:57:03 +02:00
Takashi Kajinami faf5329941 Fix dependency between redis and gnocchi services
This is follow-up of d044e10afc and is
an attempt to fix the dependency between redis and gnocchi services.

- Use title for resource collector according to the definition in
  puppet-redis. We might want to use the public API such as the redis
  class itself but it makes it difficult to handle the deployment with
  redis and gnocchi running in different nodes.

- Make all gnocchi services dependent on redis.

Change-Id: I70f5ab186b5635b364648f4e4ffb11ba5d328f4c
2022-10-18 22:18:26 +09:00
Takashi Kajinami 2c05591d9c Ceilometer: Use built-in class to run ceilometer-upgrade
The puppet-ceilometer module provides the class to run the command, so
you can use the class instead of defining the raw exec resource.

Change-Id: I9e6d2e21fcfb4847e4d54079cbb50e699931a07f
2022-10-18 11:47:55 +09:00
Takashi Kajinami bcad045b42 Drop firewall rule for ceilometer-api
... because the ceilometer-api service was removed long ago.

Change-Id: I010570cbf5a56629c532b726bf4c2db20615b800
2022-10-18 10:44:04 +09:00
Alfredo Moralejo d044e10afc Start redis before gnocchi-metricd when coordination uses redis
After [1] we need to make sure that redis is running before
gnocchi-metricd is started as it requires it up when doing any
operation.

[1] https://review.opendev.org/c/x/packstack/+/840314

Change-Id: I98b1f86c184a3a87b2d2dfd29059ec9f1cec8f8a
2022-10-17 17:19:44 +02:00
Zuul 56534af6a0 Merge "Gnocchi: Fix missing coordination backend" 2022-10-06 14:40:02 +00:00
Zuul 0c19ba40f2 Merge "Remove puppet-certmonger from dependencies" 2022-10-06 12:07:02 +00:00
Takashi Kajinami a809d1797c Gnocchi: Fix missing coordination backend
Gnocchi relies on coordination backend provided by the tooz library
for task distribution. This change ensures the coordination backend
is configured in Gnocchi, in addition to the other Telemetry services
(Ceilometer and Aodh).

Change-Id: I1588a2df2f49ac6335246b19a60e95bd436fd590
2022-10-06 13:17:38 +02:00
Zuul c85bd66dbe Merge "Replace own hiera_undef function by built-in lookup" 2022-10-06 10:56:00 +00:00
Zuul 3f559ceac4 Merge "Replace hiera by lookup" 2022-10-06 10:55:58 +00:00
Takashi Kajinami a85ba821c2 Replace own hiera_undef function by built-in lookup
Change-Id: If934b7ac2d5186d580814649994f3ee1da655cbc
2022-08-18 00:42:01 +09:00
Takashi Kajinami 6ff87dd6d4 Replace hiera by lookup
The hiera function is deprecated and does not work with the latest
hieradata version 5. It should be replaced by the new lookup
function[1].

[1] https://puppet.com/docs/puppet/7/hiera_automatic.html

With the lookup function, we can define value type and merge behavior,
but these are kept default at this moment to limit scope of this change
to just simple replacement. Adding value type might be useful to make
sure the value is in expected type (especially when a boolean value is
expected), but we will revisit that later.

example:
lookup(<NAME>, [<VALUE TYPE>], [<MERGE BEHAVIOR>], [<DEFAULT VALUE>])

This also replaces the hiera_array function which is also deprecated,
according to the guideline[2].

[2] https://puppet.com/docs/puppet/7/hiera_migrate.html#updated_classic_hiera_function_calls

Change-Id: Ic9930107fbc68cba3432f4424f113071325efcb7
2022-08-18 00:37:32 +09:00
Takashi Kajinami 05de67a535 Remove six
Because Python 2 support was removed, we no longer need the six
library which is used to make our code compatible with both Python 2
and Python 3.

Change-Id: I9c956ec5623a58fceff6854a878b4b48197c1ff3
2022-08-16 08:58:47 +00:00
Takashi Kajinami 3d41916261 Stop overriding placement::wsgi::apache::api_port
Default value of the api_port parameter has been updated to 8778. Thus
we no longer need to override the parameter.

This also resolves the type validation failure with puppetlabs-apache
8.0.0, which no longer accepts a string value for apache::vhost::port.

Change-Id: I3c29f018f6508f992181945de0cf6091dde44502
2022-08-16 03:11:40 +09:00
Joel Capitao 6f9aedc4a3 Trove: Use trove::db to set up database options
... because database options of the base trove class have been
deprecated.

Remove CentOS Stream 8 jobs

RDO Zed (currently master) will only support
CentOS Stream 9 [1].
The CS8 DLRN builder is about to be stopped
and removed, so we can get rid of the CS8 jobs.

Plus we need to require "six" explicitly.
Until now, it was installed as transitive dependency.

[1] https://lists.rdoproject.org/archives/list/dev@lists.rdoproject.org/thread/EKPN4RMQEKYS3ZAHRUHZQ47IF7QXOLZH/

Change-Id: Ib82083f328c3e23b07559652e31acb0153c61a43
2022-07-01 14:15:08 +02:00
Takashi Kajinami edcd363dc6 Trove: Remove trove::use_neutron
... because the parameter was deprecated during Yoga cycle[1] and has
had no effect since then.

Depends-on: https://review.opendev.org/c/openstack/puppet-trove/+/830620
Change-Id: Ic627bfa002c190706c49cd3ffe7f7b7a8cf1c27f
2022-06-08 13:54:10 +00:00
Takashi Kajinami d61fd5f228 Add required fixes to pass CI
Trove: update class usage

* trove::conductor::auth_url is deprecated and has no effect

* Template for guestagent.conf is deprecated in favor of the new
  trove::guestagent class.

* trove::conductor::debug is removed

* trove::taskmanager::debug is removed

(squash) Remove parameter change_password_available

It has been retired from puppet-tempest [1].

[1] https://review.opendev.org/c/openstack/puppet-tempest/+/842548

Also, replacing python3.6 by python3.9 in tox job.

Change-Id: I6a7df5deebe1f1808c08964fb43c00d84603907e
2022-06-07 17:12:32 +02:00
Takashi Kajinami 95e812dfec Remove puppet-certmonger from dependencies
... because it is not used in deployment process.

Change-Id: I5764e11a39cacef6951c9fe57bb4b1bd91a74b7d
2022-04-13 16:53:19 +09:00
Joel Capitao ecb6e851a8 Support CS9 when installing RDO repo
Change-Id: I5618353cb368e9d56188297dfdf87b1b127ee653
2022-04-08 15:18:59 +02:00
Takashi Kajinami f863661516 Cinder: Remove tenant_id templating from endpoint
Cinder no longer requires tenant_id templating[1]. We should remove it
so that system users can use API.

[1] https://review.opendev.org/c/openstack/cinder/+/822040

Change-Id: Id1eb2b1e4ad6f4ac2abe3dd7a7e81d555ca95d70
2022-03-16 00:22:15 +09:00
Takashi Kajinami 174bd7ddee Manila: Remove tenant_id templating from v2 endpint
Since Wallaby Manila no longer requires tenant_id templating in v2
endpoint and we should remove it to use API with system-scoped
credentials.

Change-Id: Iedc84742b09f02e34994de1c16330b650b250396
2022-03-16 00:21:07 +09:00
Alfredo Moralejo e3e27d7635 Fix broken jobs
This change fixes the following two problems currently blocking CI.

- Fix usage of the removed parameters[1] of puppet-tripleo.

- Remove tenant_id templating from nova endpoint as it is blocking
  API call by system-admin, which is currently required[2]

[1] https://review.opendev.org/c/openstack/puppet-trove/+/831872
[2] https://review.opendev.org/c/openstack/puppet-nova/+/828814

Change-Id: Icd8510d71fc479320eec2598a14abcec609b38eb
2022-03-16 00:19:06 +09:00
Takashi Kajinami 89fc7184ea Remove redundant inclusion of apache::listen
The apache::listen resource is now included automatically and it's not
necessary to define the resource in composition layer explicitly.

Change-Id: Ia7b80c42fd1d35e4c2b7c60fc53a01e969918f61
2022-03-07 18:40:42 +09:00
Takashi Kajinami 7f01e457b7 Do not configure unused tcp/8777
tcp/8777 was used for ceilometer-api but the api functionality was
removed from ceilometer long ago and the port is no longer used.

Change-Id: I77448859c4fac393f04a22cf13b8afe14dab2fc8
2022-03-07 18:34:31 +09:00
Alfredo Moralejo 6adc4339dc Use ecdsa key in CentOS 9 Stream
... as rsa key no longer works with the latest openssl in CentOS 9
Stream repo.

Co-Authored-By: Takashi Kajinami <tkajinam@redhat.com>

Related-Bug: #1962507
Change-Id: If6270b7f49529382c5b8147449eb631c6df420aa
2022-03-03 16:31:34 +01:00
Joel Capitao 88f91e8558 Remove Cinder::Type definition
We no longer need this definition as credentials are
pull from cinder.config file.
Plus cinder_type definitions are well configured for each
backend, so we can remove this safely.

Change-Id: Ia58f78e58114275797ebaf4988967606848f711c
2022-02-23 12:05:44 +01:00
Alfredo Moralejo e8dcebc1d7 Remove usage of deprecated parameters
This patch is removing usage of several deprecated parameters:

- keystone_v2: https://review.opendev.org/507570
- allow_versions: https://review.opendev.org/817350
- coordination_url: https://review.opendev.org/792509
- allow_overlaping_ips: https://review.opendev.org/817199

Change-Id: I728bfcb6bc7c9c37bdf74dc2e357eb87696bc523
2022-02-11 13:28:10 +01:00
Zuul e61867cb23 Merge "Fix outdated swift pipeline" 2022-02-10 17:06:28 +00:00