-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Updated version of #50 #104
base: main
Are you sure you want to change the base?
Conversation
Note that disk volumes mean filesystems directly on disks. Currently the test fails: on first invocation the filesystem gets mounted by device name and only on the second by UUID. This should be also aught by a real idempotence test. XXX this should be part of the existing verification task lists: there is already a task called "Verify that the device identifier appears in /etc/fstab", but it does not check for this condition.
d92ad0d
to
4ea9545
Compare
tests/tests_change_disk_fs.yml
Outdated
@@ -6,7 +6,7 @@ | |||
mount_location: '/opt/test' | |||
volume_size: '5g' | |||
fs_type_after: "{{ 'ext3' if (ansible_distribution == 'RedHat' and ansible_distribution_major_version == '6') else 'ext4' }}" | |||
|
|||
pat: "{{ '^([^#\\s]+)\\s+' + mount_location + '\\s.*' }}" | |||
tasks: | |||
- include_role: | |||
name: storage |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
please use name: linux-system-roles.storage
[citest pending] |
[citest bad] |
[citest pending] |
use `linux-system-roles.storage` - fix indentation
[citest pending] |
3 similar comments
[citest pending] |
[citest pending] |
[citest pending] |
|
||
- name: print fstab device | ||
debug: | ||
msg: device is: {{ ( _test_change_disk_fs_fstab1.stdout_lines|select('search', pat) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This string must be quoted because jinja2/yaml doesn't like the colon :
character in is:
:
Exception when running tests: mapping values are not allowed here
in "<unicode string>", line 37, column 23:
msg: device is: {{ ( _test_change_disk_fs_fsta ...
^ (line: 37)
up to but not including the commit that probably fixed the problem.