cmueller/ansible
1
0
Fork 0
Code Issues Pull requests Projects Releases Wiki Activity

lvg: fix default value type, should be str (#57723)

Browse source
This commit is contained in:
Alexey Kamenskiy 2019-06-12 20:52:43 +08:00 committed by René Moser
parent 33a26d31fd
commit 793c18506a
2 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
changelogs/fragments/57720-fix-lvg-warning-int-to-str-conversion.yml Normal file
View file

@ -0,0 +1,2 @@
bugfixes:
- lvg - Fixed warning shown when using default value for pesize about conversion from int to str.

4
lib/ansible/modules/system/lvg.py
View file

@ -38,7 +38,7 @@ options:
- The size of the physical extent. pesize must be a power of 2, or multiple of 128KiB. - The size of the physical extent. pesize must be a power of 2, or multiple of 128KiB.
- Since Ansible 2.6, pesize can be optionally suffixed by a UNIT (k/K/m/M/g/G), default unit is megabyte. - Since Ansible 2.6, pesize can be optionally suffixed by a UNIT (k/K/m/M/g/G), default unit is megabyte.
type: str type: str
default: 4 default: "4"
pv_options: pv_options:
description: description:
- Additional options to pass to C(pvcreate) when creating the volume group. - Additional options to pass to C(pvcreate) when creating the volume group.
@ -143,7 +143,7 @@ def main():
argument_spec=dict( argument_spec=dict(
vg=dict(type='str', required=True), vg=dict(type='str', required=True),
pvs=dict(type='list'), pvs=dict(type='list'),
pesize=dict(type='str', default=4), pesize=dict(type='str', default='4'),
pv_options=dict(type='str', default=''), pv_options=dict(type='str', default=''),
vg_options=dict(type='str', default=''), vg_options=dict(type='str', default=''),
state=dict(type='str', default='present', choices=['absent', 'present']), state=dict(type='str', default='present', choices=['absent', 'present']),