diff options
author | Matt Martz <matt@sivel.net> | 2014-12-18 13:02:44 -0600 |
---|---|---|
committer | Matt Clay <matt@mystile.com> | 2016-12-08 11:22:43 -0500 |
commit | 724ae97898f8e4918e166e24624e0d37f8e9ad6d (patch) | |
tree | 700ff21bb3e1ff928e183dec835d2690a867af67 /lib/ansible/modules/cloud/rackspace | |
parent | a0dad8e683d20b306a57634bba35b2b1ad9f24ac (diff) | |
download | ansible-724ae97898f8e4918e166e24624e0d37f8e9ad6d.tar.gz |
Fix order of logic to determine required args
Diffstat (limited to 'lib/ansible/modules/cloud/rackspace')
-rw-r--r-- | lib/ansible/modules/cloud/rackspace/rax.py | 6 |
1 files changed, 3 insertions, 3 deletions
diff --git a/lib/ansible/modules/cloud/rackspace/rax.py b/lib/ansible/modules/cloud/rackspace/rax.py index 4ec49e9736..3f6f7f334b 100644 --- a/lib/ansible/modules/cloud/rackspace/rax.py +++ b/lib/ansible/modules/cloud/rackspace/rax.py @@ -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') |