Fix order of logic to determine required args

This commit is contained in:
Matt Martz 2014-12-18 13:02:44 -06:00
parent b0dcff214a
commit d0e82fa3af

View file

@ -437,14 +437,14 @@ def cloudservers(module, state=None, name=None, flavor=None, image=None,
'incorrectly capitalized region name.')
if state == 'present' or (state == 'absent' and instance_ids is None):
if not boot_from_volume and not boot_volume and not image:
module.fail_json(msg='image is required for the "rax" module')
for arg, value in dict(name=name, flavor=flavor).iteritems():
if not value:
module.fail_json(msg='%s is required for the "rax" module' %
arg)
if not boot_from_volume and not boot_volume and not image:
module.fail_json(msg='image is required for the "rax" module')
if boot_from_volume and not image and not boot_volume:
module.fail_json(msg='image or boot_volume are required for the '
'"rax" with boot_from_volume')