Improve the ec2_snapshot argument spec and docs

ec2_snapshot got missed when moving to a common argument spec.
It could already make use of the capabilities (as it uses
ec2_connect) and the documentation suggested it supported the
common argument spec (thanks to the documentation fragment work)
so it was just a matter of fixing the argument spec.

Removed unnecessary documentation for profile and security_token
that is covered by documentation fragment

Also removed spurious documentation flags (default: null, aliases: [])
which aren't needed.
reviewable/pr18780/r1
willthames 11 years ago
parent b08ce87429
commit 44eb585e74

@ -26,52 +26,27 @@ options:
description: description:
- The AWS region to use. If not specified then the value of the EC2_REGION environment variable, if any, is used. - The AWS region to use. If not specified then the value of the EC2_REGION environment variable, if any, is used.
required: false required: false
default: null
aliases: ['aws_region', 'ec2_region'] aliases: ['aws_region', 'ec2_region']
volume_id: volume_id:
description: description:
- volume from which to take the snapshot - volume from which to take the snapshot
required: false required: false
default: null
aliases: []
description: description:
description: description:
- description to be applied to the snapshot - description to be applied to the snapshot
required: false required: false
default: null
aliases: []
instance_id: instance_id:
description: description:
- instance that has the required volume to snapshot mounted - instance that has the required volume to snapshot mounted
required: false required: false
default: null
aliases: []
device_name: device_name:
description: description:
- device name of a mounted volume to be snapshotted - device name of a mounted volume to be snapshotted
required: false required: false
default: null
aliases: []
profile:
description:
- uses a boto profile. Only works with boto >= 2.24.0
required: false
default: null
aliases: []
version_added: "1.6"
security_token:
description:
- security token to authenticate against AWS
required: false
default: null
aliases: []
version_added: "1.6"
snapshot_tags: snapshot_tags:
description: description:
- a hash/dictionary of tags to add to the snapshot - a hash/dictionary of tags to add to the snapshot
required: false required: false
default: null
aliases: []
version_added: "1.6" version_added: "1.6"
author: Will Thames author: Will Thames
@ -112,21 +87,19 @@ except ImportError:
sys.exit(1) sys.exit(1)
def main(): def main():
module = AnsibleModule( argument_spec = ec2_argument_spec()
argument_spec = dict( argument_spec.update(
dict(
volume_id = dict(), volume_id = dict(),
description = dict(), description = dict(),
instance_id = dict(), instance_id = dict(),
device_name = dict(), device_name = dict(),
region = dict(aliases=['aws_region', 'ec2_region'], choices=AWS_REGIONS),
ec2_url = dict(),
ec2_secret_key = dict(aliases=['aws_secret_key', 'secret_key'], no_log=True),
ec2_access_key = dict(aliases=['aws_access_key', 'access_key']),
wait = dict(type='bool', default='true'), wait = dict(type='bool', default='true'),
wait_timeout = dict(default=0), wait_timeout = dict(default=0),
snapshot_tags = dict(type='dict', default=dict()), snapshot_tags = dict(type='dict', default=dict()),
) )
) )
module = AnsibleModule(argument_spec=argument_spec)
volume_id = module.params.get('volume_id') volume_id = module.params.get('volume_id')
description = module.params.get('description') description = module.params.get('description')

Loading…
Cancel
Save