mirror of
https://github.com/ansible-collections/community.general.git
synced 2024-09-14 20:13:21 +02:00
f6db0745fc
* revamp filesystem module to prepare next steps * pass all commands to module.run_command() as lists * refactor grow() and grow_cmd() to not need to override them so much * refactor all existing get_fs_size() overrides to raise a ValueError if not able to parse command output and return an integer. * override MKFS_FORCE_FLAGS the same way for all fstypes that require it * improve documentation of limitations of the module regarding FreeBSD * fix indentation in DOCUMENTATION * add/update function/method docstrings * fix pylint hints filesystem: refactor integration tests * Include *reiserfs* and *swap* in tests. * Fix reiserfs related code and tests accordingly. * Replace "other fs" (unhandled by this module), from *swap* to *minix* (both mkswap and mkfs.minix being provided by util-linux). * Replace *dd* commands by *filesize* dedicated module. * Use FQCNs and name the tasks. * Update main tests conditionals. * add a changelog fragment * Apply suggestions from code review Co-authored-by: Felix Fontein <felix@fontein.de> * declare variables as lists when lists are needed * fix construction without useless conversion Co-authored-by: Felix Fontein <felix@fontein.de>
110 lines
3.1 KiB
YAML
110 lines
3.1 KiB
YAML
---
|
|
- name: "Create filesystem"
|
|
community.general.filesystem:
|
|
dev: '{{ dev }}'
|
|
fstype: '{{ fstype }}'
|
|
register: fs_result
|
|
|
|
- name: "Assert that results are as expected"
|
|
ansible.builtin.assert:
|
|
that:
|
|
- 'fs_result is changed'
|
|
- 'fs_result is success'
|
|
|
|
- name: "Get UUID of created filesystem"
|
|
ansible.builtin.command:
|
|
cmd: 'blkid -c /dev/null -o value -s UUID {{ dev }}'
|
|
changed_when: false
|
|
register: uuid
|
|
|
|
- name: "Check that filesystem isn't created if force isn't used"
|
|
community.general.filesystem:
|
|
dev: '{{ dev }}'
|
|
fstype: '{{ fstype }}'
|
|
register: fs2_result
|
|
|
|
- name: "Get UUID of the filesystem"
|
|
ansible.builtin.command:
|
|
cmd: 'blkid -c /dev/null -o value -s UUID {{ dev }}'
|
|
changed_when: false
|
|
register: uuid2
|
|
|
|
- name: "Assert that filesystem UUID is not changed"
|
|
ansible.builtin.assert:
|
|
that:
|
|
- 'fs2_result is not changed'
|
|
- 'fs2_result is success'
|
|
- 'uuid.stdout == uuid2.stdout'
|
|
|
|
- name: "Check that filesystem is recreated if force is used"
|
|
community.general.filesystem:
|
|
dev: '{{ dev }}'
|
|
fstype: '{{ fstype }}'
|
|
force: yes
|
|
register: fs3_result
|
|
|
|
- name: "Get UUID of the new filesystem"
|
|
ansible.builtin.command:
|
|
cmd: 'blkid -c /dev/null -o value -s UUID {{ dev }}'
|
|
changed_when: false
|
|
register: uuid3
|
|
|
|
- name: "Assert that filesystem UUID is changed"
|
|
# libblkid gets no UUID at all for this fstype on FreeBSD
|
|
when: not (ansible_system == 'FreeBSD' and fstype == 'reiserfs')
|
|
ansible.builtin.assert:
|
|
that:
|
|
- 'fs3_result is changed'
|
|
- 'fs3_result is success'
|
|
- 'uuid.stdout != uuid3.stdout'
|
|
|
|
|
|
- when: 'grow|bool and (fstype != "vfat" or resize_vfat)'
|
|
block:
|
|
- name: "Increase fake device"
|
|
community.general.filesize:
|
|
path: '{{ image_file }}'
|
|
size: '{{ fssize | int + 1 }}M'
|
|
|
|
- name: "Resize loop device for LVM"
|
|
ansible.builtin.command:
|
|
cmd: 'losetup -c {{ dev }}'
|
|
when: fstype == 'lvm'
|
|
|
|
- name: "Expand filesystem"
|
|
community.general.filesystem:
|
|
dev: '{{ dev }}'
|
|
fstype: '{{ fstype }}'
|
|
resizefs: yes
|
|
register: fs4_result
|
|
|
|
- name: "Get UUID of the filesystem"
|
|
ansible.builtin.command:
|
|
cmd: 'blkid -c /dev/null -o value -s UUID {{ dev }}'
|
|
changed_when: false
|
|
register: uuid4
|
|
|
|
- name: "Assert that filesystem UUID is not changed"
|
|
ansible.builtin.assert:
|
|
that:
|
|
- 'fs4_result is changed'
|
|
- 'fs4_result is success'
|
|
- 'uuid3.stdout == uuid4.stdout' # unchanged
|
|
|
|
- when:
|
|
- (grow | bool and (fstype != "vfat" or resize_vfat)) or
|
|
(fstype == "xfs" and ansible_system == "Linux" and
|
|
ansible_distribution not in ["CentOS", "Ubuntu"])
|
|
block:
|
|
- name: "Check that resizefs does nothing if device size is not changed"
|
|
community.general.filesystem:
|
|
dev: '{{ dev }}'
|
|
fstype: '{{ fstype }}'
|
|
resizefs: yes
|
|
register: fs5_result
|
|
|
|
- name: "Assert that the state did not change"
|
|
ansible.builtin.assert:
|
|
that:
|
|
- 'fs5_result is not changed'
|
|
- 'fs5_result is succeeded'
|