diff options
author | James Laska <jlaska@ansible.com> | 2014-03-12 19:28:57 -0400 |
---|---|---|
committer | Michael DeHaan <michael.dehaan@gmail.com> | 2014-03-13 07:50:11 -0400 |
commit | f0a89d5bdf6e96d870247cdd023a4d9e78794745 (patch) | |
tree | 4b9b9e784fd894bcb2c40301a8b93b6840076b5c | |
parent | 763235a83da51932f5c92c32be27807347db89bd (diff) | |
download | ansible-f0a89d5bdf6e96d870247cdd023a4d9e78794745.tar.gz |
Correct test integration ec2 cleanup script
-rw-r--r-- | test/integration/cleanup_ec2.py | 12 |
1 files changed, 7 insertions, 5 deletions
diff --git a/test/integration/cleanup_ec2.py b/test/integration/cleanup_ec2.py index 08d54751aa..0e974c2089 100644 --- a/test/integration/cleanup_ec2.py +++ b/test/integration/cleanup_ec2.py @@ -15,12 +15,14 @@ def delete_aws_resources(get_func, attr, opts): for item in get_func(): val = getattr(item, attr) if re.search(opts.match_re, val): - prompt_and_delete("Delete object with %s=%s? [y/n]: " % (attr, val), opts.assumeyes) + prompt_and_delete(item, "Delete object with %s=%s? [y/n]: " % (attr, val), opts.assumeyes) -def prompt_and_delete(prompt, assumeyes): - while not assumeyes: - assumeyes = raw_input(prompt) - obj.delete() +def prompt_and_delete(item, prompt, assumeyes): + if not assumeyes: + assumeyes = raw_input(prompt).lower() == 'y' + assert hasattr(item, 'delete'), "Class <%s> has no delete attribute" % item.__class__ + if assumeyes: + item.delete() def parse_args(): # Load details from credentials.yml |