Releases: linux-system-roles/storage
add back support for mount_options; support collection-requirements.yml
Use meta/collection-requirements.yml for collection dependencies (#267)
There is no way for a role to automatically require collections for
module/plugin dependencies. Use the non-standard file
meta/collection-requirements.yml for this. Note that
meta/requirements.yml is supposed to be a list of roles only, in
the format described at https://docs.ansible.com/ansible/latest/galaxy/user_guide.html#installing-multiple-roles-from-multiple-files
add support for mount_options (#270)
- add support for mount_options
When support for argument validation was added, that support did not
include the mount_options parameter. This fix adds back that
parameter. In addition, the volume module arguments are refactored
so that the common volume parameters such as mount_options can be
specified in one place.
This adds a test for the mount_options parameter, and adds
verification for that parameter.
- only checkout mount_options if requested
[citest skip] bump tox-lsr version to 2.11.0; remove py37; add py310 (#266)
Signed-off-by: Rich Megginson rmeggins@redhat.com
add xfsprogs to packages; support gather_facts: false; support setup-snapshot
add setup snapshot to install packages into snapshot (#260)
allow role to work with gather_facts: false (#262)
Allow the role to work if the playbook has set gather_facts: false.
The gather_subset: min includes all of the facts used by this
role. Other roles may need to gather more facts.
The role gathers the facts where they are needed. Other roles may
need to gather facts in a different place.
add xfsprogs for non-cloud-init systems (#263)
-
add xfsprogs for el8
-
add xfsprogs for non-cloud-init systems
There should be an explicit dependency on the xfsprogs packages.
The role works in CI because the CI systems are using cloud-init
based VMs which pull in xfsprogs. On non-cloud-init systems the
package is missing and the role fails.
Less verbosity by default (#256)
support LVM RAID volumes - LVM cache support - remove recursive symlinks
Refactored pool create function
- refactored BlivetLVMVolume._create
Added support for LVM RAID volumes
- raid_level can be used on pool volume
- raid_disks can specify disks to be used
- test added
- some changes to README-devel (more to come)
remove extra blank lines
remove extra blank lines
Refined safe_mode condition in create_members
- safe_mode condition in Blivet.Pool._create_members should now catch
formatting reported by blkid but not being recognized/handled by blivet
Parameter checking (#223)
Older versions of ansible do not validate nested modules parameters for
types etc. Ansible 2.11 and newer implement this functionality. This
commit adds type definitions for nested variables.
- added custom parameter checking
- parameters (including nested) are now checked for type and value
- values of all parameters are now normalized when applicable
- specifying list values (e.g. disks) as non-list is now allowed
- non-list value is converted to list
- original failure-expecting test removed
- functionality tested by modification of an existing test
Add LVM RAID specific parameters to module_args (#246)
These were not added after the nested parameters checks were
implemented.
change recursive role symlink to individual role dir symlinks (#248)
Signed-off-by: Rich Megginson rmeggins@redhat.com
Add support for creating and managing LVM cache volumes (#235)
-
Add support for creating and managing LVM cache volumes
-
Allow specifying disks as cache_devices instead of PVs
It might be hard for users to know names of the partitions we'll
create as PVs so we should let them disk names and get the PVs on
them ourselves when creating the cache.
- Add LVM cache specific parameters to module_args
bump tox-lsr version to 2.8.3 (#249)
Signed-off-by: Rich Megginson rmeggins@redhat.com
workaround service_facts issues; workaround udev, lvm issues on el9
Add workaround for the service_facts module for Ansible < 2.12
add tags to allow skipping lvm tests (#242)
lvm is currently broken on el9. We need to skip these tests until
lvm is fixed. See https://bugzilla.redhat.com/show_bug.cgi?id=1967212
remove py27 from github CI testing (#244)
py27 is no longer supported
See linux-system-roles/linux-system-roles.github.io#70
for more details
add support for storage_udevadm_trigger (#243)
On some platforms, udev is not updated immediately after blivet
completes its disk operations. The user can set
storage_udevadm_trigger: true in order to cause udev to ensure
those changes take effect immediately.
update tox-lsr version to 2.8.0 (#240)
New version adds check for proper commenting of the ansible_managed var
Signed-off-by: Sergei Petrosian spetrosi@redhat.com
add meta/requirements.yml; support Rocky Linux; test with ansible 2.12
add meta/requirements.yml (#231)
The storage role requires ansible.posix.mount. Add a requirement
for the ansible.posix collection.
support ansible-core 2.11 ansible-lint, ansible-test
Add support for Rocky Linux 8 (#229)
This file is a copy of RedHat_8.yml / CentOS_8.yml.
support python 39, ansible-core 2.12, ansible-plugin-scan (#234)
update tox-lsr version to 2.7.1 (#236)
update the tox-lsr version used in github actions tox CI
to 2.7.1
The only difference between this an 2.7.0 is that Ansible 2.12
is now GA.
Signed-off-by: Rich Megginson rmeggins@redhat.com
Support ansible-core; Improve wording in README files
use tox-lsr version 2.5.1
This version removes support for molecule until we can figure out
what to do about molecule. This should make all of the tox tests
pass (except for python 2.6).
Signed-off-by: Rich Megginson rmeggins@redhat.com
use apt-get install -y
Signed-off-by: Rich Megginson rmeggins@redhat.com
replace json_query with selectattr and map
Replace crypttab with lineinfile (#228)
- Replace crypttab with lineinfile
Crypttab is not available in ansible-core, hence need to replace it
- Add create: true to lineinfile to create the file if it doesn't exist
Use lower case title for consistency
Set mode properly
- Remove entries with regexp regardeless of password specified, fix mode
Improve wording (#230)
Added skip checks feature to speed up the tests (#218)
- Added skip checks feature to speed up the tests
- added new fact that allows to skip some checks in case the role is run
multiple times within the same playbook - this should help speed up the tests
- Added skip checks feature to speed up the tests
- removed "packages_installed", now it is part of "required_packages"
check - pylint fixes
- luks tests install extra packages and checks skipping causes race
condition - checks do not disabled in this case
(commit will be squashed into previous one)
revert fix for dm-vdo error - this is now fixed
revert fix for testing bz1991062
do not check for dm-vdo
drop support for ansible 2.8
Raise supported Ansible version to 2.9
min_ansible_version is now 2.9
Bug 1989197 - drop support for Ansible 2.8
https://bugzilla.redhat.com/show_bug.cgi?id=1989197
Fix tests_lvm_errors; skip vdo tests if dm-vdo not found
use volume1_size; check for expected error
https://bugzilla.redhat.com/show_bug.cgi?id=1990793
The test was using volume_size, which is undefined, instead
of volume1_size. Furthermore, the error was being masked by
the rescue block because the rescue was not checking for the
correct error message. The fix is to use volume1_size AND check
for the correct error message.
I'll note that there are several tests that do not look for the
correct error message, and we may run into this problem with other
tests.
skip test if kvdo and dm-vdo are not found
https://bugzilla.redhat.com/show_bug.cgi?id=1991062
Am getting this error when testing vdo:
Failed to commit changes to disk: Process reported exit code 3: modprobe: FATAL: Module dm-vdo not found in directory /lib/modules/5.14.0-0.rc4.35.el9.x86_64
/usr/sbin/modprobe failed: 1
vdo: Required device-mapper target(s) not detected in your kernel.
Run `lvcreate --help' for more information.
So I'll just skip vdo testing unless both kvdo and dm-vdo are found.