Due to the bug [1] in CentOS packaging, systemd-udev is substituted with
systemd-boot-unsigned. So you need to use NVR to properly
install systemd-udev until the bug is fixed.
[1] https://bugzilla.redhat.com/show_bug.cgi?id=2183279
Change-Id: I3c112b74b4777b9443f3c3041a51ecb770d48021
With update of ansible-lint to version >=6.0.0 a lot of new
linters were added, that enabled by default. In order to comply
with linter rules we're applying changes to the role.
With that we also update metdata to reflect current state.
Change-Id: I8bd832c172ef87acca27f053c5223941d80f9ce1
Since ansible-core 2.14 you can't use warn as module argument.
It's being removed to avoid module failure.
Change-Id: I7287d449b8fd0ad970e37aa63b5cb25f88197858
We're relying on udev to exists for glusterfs since we're
applying overrides for it as well as attempting to restart.
While systemd-udev seems not being pre-installed in all CentOS
containers anymore.
Depends-On: https://review.opendev.org/c/openstack/openstack-ansible-openstack_hosts/+/878926
Change-Id: Ia03ee4aeb381da00a538e3775b824f2a5ce4e01e
In case of setting file permissions to 640 systemd will raise an
alert [1]. In order to avoid spamming in logs, we change default mode to
644 as no passwords or credentials are going to be stored in this file.
With that we provide a way to override file mode if needed.
[1] https://paste.openstack.org/show/b83tSrjni8iB2WNsvDfQ/
Change-Id: I73164e9d4b02d7b3fae21b912369afcd1cc64abf
Improve logic for making systemd mount dependent on
network-online.target when network filesystem is being used.
This change improves feature implemented in commit [1].
Currently when device is specified by UUID or LABEL, dependency for
network-online.target is added even it's not a network filesystem.
We should be careful with this as it may create circular dependency.
In my case, circular dependency looked like this:
network.target ->
network-pre.target ->
netfilter-persistent.service ->
local-fs.target ->
var-lib-nova.mount ->
network-online.target ->
network.target
Additionally, this commit was originally supposed to add dependency also
for CIFS mounts(these ones starting from '//') but it never worked in my
opinion. It was also fixed by this change.
[1] 6a9dcb6886
Change-Id: I5d2d9c074aa0390a6f8b7c8c0ba1c839fe26f6c5
With tox release of 4.0, some parameters were deprecated and are ignored now
which causes tox failures. One of the most spread issues we have is using
`whitelist_externals` isntead of `allowlist_externals`
Change-Id: I8db7ddabfd3586136f442da4a4807fafbb91b5d3
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: Ied3df03a365480d0d07a03e17b161efe1fcd699f
This line was introduced by I680359ca655d0f69a40e9d29dbf1694cd0aa4ca2
for convenience, and should already be covered by the
distribution_major_version line above.
Change-Id: I8aba13ef29571730f9260dc2f6af33c168185780
With sphinx release of 5.0.0, they changed default for language variable
to 'en' from None. With that current None valuable is not valid and should
not be used.
Change-Id: Ie1bbbbd73c3d33cb8b7bc99bf32b6497c7f40534
Currently "Set the state of the mount" task reports changed state
every time, which makes impression that role is not idempotent.
We fix that by adding `changed_when: false` to the task.
Change-Id: I3deca051530b3c5497d70b9bc06f39483e8ef4ce
In README file we're refferencing plugins repo for installing
config_template from it. However, we're using module from it's
collection.
We're updating documentation along with renaming a-r-r to more common
requirements.yml and adjusting rc file
to drop not required overrides from there.
Change-Id: I7bf4c0fca678fd50be2dc041fe16efcc67c44fa5
Closes-Bug: #1961621
Use a first_found lookup instead of a with_first_found loop so that
the 'paths' parameter can be used.
This ensures that only vars from the role are included, and not vars
from a parent calling role. This can happen when a parent role has
a higher priority vars file available for inclusion than the role
it calls.
Change-Id: Idbeb3e61d0398944af8d73db5d98352de95b91a7
Since we still use ceph-ansible that has their own implementation of
config_template module it's worth to use mentioned module as a collection
explicitly.
Depends-On: https://review.opendev.org/c/openstack/openstack-ansible/+/819814
Change-Id: I69c8682c40a61e87156c1615c3b3eab67a76abe8
All references to Gentoo, SUSE, Debian stretch and Centos-7 are removed.
Conditional tasks, ternary operators and variables are simplified where possible
OS specific variables files are generalised where possible
Change-Id: I55037805e59cb6d727593f467e98568a4c030e74
Since we copy all release notes to the integrated repo there is not need
in publishing release notes for each repository. We should only verify their
validity and linting.
Change-Id: I3a8975eb80fcbef96bf18475837d76a509560f64
Currently it's deployer resposibility to install required binaries for
mounts to succeed, which is not really convenient. Instead of this
we can try to provide at least some basic binaries installation in case
supported type is set.
Depends-On: https://review.opendev.org/755484
Change-Id: I680359ca655d0f69a40e9d29dbf1694cd0aa4ca2
We assume that the deployer knows what they are doing so remove
these tasks which add extra runtime to deployments and CI tests.
Change-Id: I4806ce34a38375461f4de1cc69c7e05cd86ff602
ansible_python_interpreter is now set to auto in the tests repo.
it doesn't need to be overwritten in the test anymore.
Depends-On: https://review.opendev.org/734676/
Change-Id: I310a1b3d81d726eb8c3401607a1306f44ee1a4d4
The sync from https://review.opendev.org/733244 updated to
openstackdocstheme 2.2.1 and reno 3.1.0 versions.
Set openstackdocs_pdf_link to link to PDF file. Note that
the link to the published document only works on docs.openstack.org
where the PDF file is placed in the top-level html directory. The
site-preview places the PDF in a pdf directory.
openstackdocstheme renames some variables, so follow the renames
before the next release removes them. A couple of variables are also
not needed anymore, remove them.
See also
http://lists.openstack.org/pipermail/openstack-discuss/2020-May/014971.html
Change-Id: I63feca3be7a797b2c2af8e650b938aef0f1cd782
Add file to the reno documentation build to show release notes for
stable/ussuri.
Use pbr instruction to increment the minor version number
automatically so that master versions are higher than the versions on
stable/ussuri.
Change-Id: If15c37aa648af362a9b301693f4c48ab78ff6225
Sem-Ver: feature
New version of openstackdocstheme (Victoria+) respects pygments_style.
Since this repo is using now Victoria (master) requirements but has
not branched for Ussuri yet, it uses the new version.
Change pygments_style to 'native' since old theme version always used
'native' and the theme now respects the setting and using 'sphinx' can
lead to some strange rendering.
Change-Id: I93ed87772eeea389ebf6d1cb95241091d54e9ae0