storage

storage copied to clipboard

storage: size doesn't work for partition pool

3

# playbook ``` --- - hosts: all become: true vars: mount_location: '/opt/test1' volume_group_size: '5g' volume_size_before: '4g' volume_size_after: '10g' storage_safe_mode: false tasks: - include_role: name: storage - include_tasks: get_unused_disk.yml vars: min_size:...

yizhanglinux

Be Exceedingly Clear and Explicit in Documenting Parameters

1

Make sure to include data type, acceptable values, examples, default values, and whether each parameter is required or optional. Also add a separate statement describing the minimum information for a...

dwlehman

storage: tests_swap failed with one 280GB disk

4

The tests failed when I use one 1TB disk, it reports "device is too large for new format" from the error output, but it works when I use "mkswap on...

yizhanglinux

multipath support

4

there is a role for configuring multipath: https://github.com/cdomingos/ansible-role-multipath , would it make sense to integrate it? @cdomingos

pcahyna

consider failing earlier when no device is provided

1

I accidentally executed a playbook using the storage role where I did not have a device defined. Would it be good to detect very early that a device name is...

tabowling

support identifying existing devices by UUID

2

The current codebase does not make any attempt to identify devices by descriptors other than `name`. While we cannot specify UUID for new devices, we can support lookup/id of existing...

dwlehman

support more disk lookup keys

1

The current support in blivet does not include support for: 1. `/dev/disk/` symlinks 2. WWN/WWID

dwlehman

Document default to destructive behavior

1

@dwlehman reminder that we need to clearly communicate to end users via upstream documentation that the current behavior is to default to destructively overwriting existing and conflicting volume groups and...

tabowling

LVM size is not supporting 100% option.

4

LVM size is not supporting 100% option. I tried multiple syntax combinations: "100%", '100%' but both failed. ``` TASK [linux-system-roles.storage : parse the specified size] ********************************************************************************************************************** fatal: [rhel7]: FAILED! =>...

tabowling

Make lvm size optional

2

Make lvm size option optional if only a single lvm is specified and assume 100% free space. If multiple lvms are specified, space should be required.

tabowling