mirror of https://github.com/ceph/ceph-ansible.git
osd: validate devices variable input
Fail with a sane message if the devices or raw_journal_devices variables are strings instead of lists during manual device assignment. Signed-off-by: Douglas Fuller <dfuller@redhat.com>pull/1644/head
parent
035846217e
commit
e5d06a449f
|
@ -55,6 +55,15 @@
|
||||||
- not osd_auto_discovery
|
- not osd_auto_discovery
|
||||||
- devices|length == 0
|
- devices|length == 0
|
||||||
|
|
||||||
|
- name: make sure the devices variable is a list
|
||||||
|
fail:
|
||||||
|
msg: "devices: must be a list, not a string, i.e. [ \"/dev/sda\" ]"
|
||||||
|
when:
|
||||||
|
- osd_group_name is defined
|
||||||
|
- osd_group_name in group_names
|
||||||
|
- not osd_auto_discovery
|
||||||
|
- devices is string
|
||||||
|
|
||||||
- name: verify journal devices have been provided
|
- name: verify journal devices have been provided
|
||||||
fail:
|
fail:
|
||||||
msg: "please provide devices and raw journal devices to your osd scenario"
|
msg: "please provide devices and raw journal devices to your osd scenario"
|
||||||
|
@ -66,6 +75,18 @@
|
||||||
- raw_journal_devices|length == 0
|
- raw_journal_devices|length == 0
|
||||||
or devices|length == 0
|
or devices|length == 0
|
||||||
|
|
||||||
|
- name: make sure the raw_journal_devices variable is a list
|
||||||
|
fail:
|
||||||
|
msg: "raw_journal_devices: must be a list, not a string, i.e. [ \"/dev/sda\" ]"
|
||||||
|
when:
|
||||||
|
- osd_group_name is defined
|
||||||
|
- osd_group_name in group_names
|
||||||
|
- not containerized_deployment
|
||||||
|
- raw_multi_journal
|
||||||
|
- raw_journal_devices is string
|
||||||
|
- raw_journal_devices|length == 0
|
||||||
|
or devices|length == 0
|
||||||
|
|
||||||
- name: check if bluestore is supported by the selected ceph version
|
- name: check if bluestore is supported by the selected ceph version
|
||||||
fail:
|
fail:
|
||||||
msg: "bluestore is not supported by the selected Ceph version, use Luminous or above."
|
msg: "bluestore is not supported by the selected Ceph version, use Luminous or above."
|
||||||
|
|
Loading…
Reference in New Issue