image_id is not required to delete a vm from openstack
Since deletion do not check the type of image or anything, and since that's tedious to keep track of the image_id and just adding noise to add image_id for nothing, this commit just relax the requirement.
This commit is contained in:
parent
5847720746
commit
3f158a4688
1 changed files with 6 additions and 3 deletions
|
@ -213,7 +213,7 @@ def main():
|
|||
auth_url = dict(default='http://127.0.0.1:35357/v2.0/'),
|
||||
region_name = dict(default=None),
|
||||
name = dict(required=True),
|
||||
image_id = dict(required=True),
|
||||
image_id = dict(default=None),
|
||||
flavor_id = dict(default=1),
|
||||
key_name = dict(default=None),
|
||||
security_groups = dict(default='default'),
|
||||
|
@ -234,6 +234,9 @@ def main():
|
|||
except Exception as e:
|
||||
module.fail_json( msg = "Error in authenticating to nova: %s" % e.message)
|
||||
if module.params['state'] == 'present':
|
||||
if not module.params['image_id']:
|
||||
module.fail_json( msg = "Parameter 'image_id' is required if state == 'present'")
|
||||
else:
|
||||
_get_server_state(module, nova)
|
||||
_create_server(module, nova)
|
||||
if module.params['state'] == 'absent':
|
||||
|
|
Loading…
Reference in a new issue