set_value method of Puppet::Util::IniFile uses '' for separator when
called without the separator argument[1]. Set the argument explicitly
to avoid broken rendering. Note that the argument was added in
puppetlabs-inifile 2.0.0 so we don't need to bump the min version.
[1] https://github.com/puppetlabs/puppetlabs-inifile/issues/544
Closes-Bug: #2081303
Change-Id: I6112c6d12e860bd4b4030e5f784952a7358f617f
(cherry picked from commit 93f15aa78c378dd853e000156ad14dfff15fa394)
(cherry picked from commit 5769e2a9bd787966d444b7964921c2e8e373dca2)
(cherry picked from commit a0dbacabb44350c816e56924ff50fe74ee7da81c)
The ignore_projects option is not a multi-valued option but accepts
only comma-separated strings.
Change-Id: I7977ea4c84417078457afb0d0834934ba5481e5d
(cherry picked from commit e751dda37b549d26f4bf5c097e7d1ce8f1aa08b4)
(cherry picked from commit 642bd4006c03615e429d04fff8f6bc7b0505317d)
(cherry picked from commit a78079fcf32e2869f2976c85452c4d38387585fb)
The title of the package resource to manage dnspython is not python3-*
but python-*. Also the proxy-server service should be restarted when
the package is updated so that the updated version is loaded.
Backport note:
Because the swift-proxy-server tag is not used in stable/2023.1, this
modifies the way to search the swift-proxy-server service resource.
Change-Id: I3372c38dcbc7312395ddf643862da91b00262ad7
(cherry picked from commit f77b2c23c9407a7aa4c434959470b5b8deb95bcd)
(cherry picked from commit 134bce281286db7c3c772e179e2b1da2ea459e28)
We've seen release job failures caused by too large tarball, which
contains irrelevant files like git history.
Let's exclude these files to reduce size of the archive.
The file is copied from the puppetlabs repositories with some files
specific to OpenStack projects added.
Change-Id: I9ea8a542c0eb10f001ea00ff96896b3d7a67a20e
(cherry picked from commit 9cb5bb1b4fa8b85846d6089d59576dacc796db23)
Update the URL to the upper-constraints file to point to the redirect
rule on releases.openstack.org so that anyone working on this branch
will switch to the correct upper-constraints list automatically when
the requirements repository branches.
Until the requirements repository has as stable/2023.1 branch, tests will
continue to use the upper-constraints list on master.
Change-Id: I39c63806340ee5d1c6700bfd5e72c0bf35ef23e3
This adds a few more assertions in acceptance tests to ensure storage
servers(account-server, container-server and object-server) are
listening on their tcp ports as expected.
Change-Id: Idbcb3d6df962d99402a06f895369fe5f812d49ab
puppetlabs_spec_helper recommends rspec-mocks instead of mocha[1] and
it uses rspec-mocks by default instead of mocha since v 5.0.0[2]
This is the prep work to adapt to that migration.
[1] https://github.com/puppetlabs/puppetlabs_spec_helper/#mock_with
[2] 493f0cbc1c
Closes-Bug: #2004135
Change-Id: I47b660d1713b394a2dd72a24a87b614d551fe802
This parameter was deprecated during the previous cycle[1] and has had
no effect since then.
[1] 152587fd2a1d0c0bc7ad67585fe2ecdd6abde995
Change-Id: I968efcf3e25199f78559d1e1154a19f8772884c7
This change introduces the new class to configure cron job to run
the swift-drive-audit CLI tool periodically. The CLI tool checks
the log files to detect any device failure and umount faulty disks.
Change-Id: I8aded9b59939e5a2eab04424277d9082cbda0a3a
The log_address parameter has never been deprecated and it has been
actually configured in the config files. This fixes the wrong parameter
description.
Change-Id: I5ec835ce2febdd46e9706d7f927d39a24fdd576c
The auditor_disk_chunk_size parameter defaults to $::os_service_default
now so passing undef to the parameter is resolved to that default
value by Puppet.
Change-Id: Ie6026ba2fb1bf657d6b1b981027bc5a8590d0640
This introduces a new defined resource type to manage the backend
ratelimit middleware options.
The middleware was added recently so that users can ratelimit internal
requests. Details can be found in the dependent change.
Depends-on: https://review.opendev.org/836046
Change-Id: Ie7c33d0a7e75238be9638551779413d93206cc41
This change introduces the new parameters for the following options so
that users can customize fallocate calls in storage server processes.
- [DEFAULT] disable_fallocate
- [DEFAULT] fallocate_reserve
- [app:<type>-server] fallocate_reserve
Change-Id: I37164a2fda28e63604545db46fc925ecf04f9204
swift 2.10.0 was released in 2016. We no longer need to support even
older versions. The output format depends on the version currently
installed, thus the latest logic should work as long as the same
release (or even a bit older release) is installed.
This also updates the logic to skip the header lines. The new logic
is considered to be more robust, because it works even when additional
lines are added.
Change-Id: I694ca4c046f490e4ffecf2101f3f147a69045162
The IPv6 addresses in ring_*_device resource names should be surrounded
by []. This ensures the address is properly formatted when the resource
is define in swift::storage::node.
Closes-Bug: #1997295
Change-Id: I390f3c7bbfbbbc7217f81e8a0312e5db201ef409
This change ensures the shortened address representations are always
used for ring devices to avoid broken idempotency caused by different
representations (shortened[1] vs canonical[2]).
[1] ::1
[2] 0000:0000:0000:0000:0000:0000:0000:0001
Closes-Bug: #1997313
Change-Id: I762f0780ba25536aa616482a49d59c7db69924d6
Currently acceptance tests are consistently failing in Ubuntu, because
of broken idempotency with swiftinit service provider.
This change disables the test cases as a temporal workaround.
Change-Id: I03265d899ad04f2aa6e39fcb4b79352ac7fa56cd
Add file to the reno documentation build to show release notes for
stable/zed.
Use pbr instruction to increment the minor version number
automatically so that master versions are higher than the versions on
stable/zed.
Sem-Ver: feature
Change-Id: I69a3680164ec1bfc3182e463322edc5aaa93ff9f
The swift processes does not require write access to the mount base
directory (which is usually /srv/node), thus the directory can be owned
by root. This is more consistent with the current installation guide
of Swift.
Change-Id: I6da122c569e7a66a1933b7483fb8cf68a40dd94f
This change makes the root user, instead of the swift user, own
the mount path where the data disk is mounted. This allows us to
prevent the swift processes from writing files.
This follows the current installation doc(eg. [1]) which suggests
chown AFTER mounting the disk device.
[1] https://docs.openstack.org/swift/latest/install/storage-install-rdo.html
Change-Id: I1a612fcf658da8ed0cb1dc51fa5e534dc0341241
The parameter was deprecated during Victoria cycle[1] in favor of
the auth_type parameter.
[1] 5722f7149c678fcb79cf792a9eda53537e080cb8
Change-Id: Ibc261c2e2ba9fa9ba547ad5d0f736c1b7230c579
