mirror of
https://github.com/ansible-collections/community.general.git
synced 2025-07-24 22:00:22 -07:00
Add elapsed return value to select modules (#37969)
* Add elapsed return value to select modules It can be quite useful to know exactly how much time has elapsed downloading/waiting. This improves existing modules or updates documentation. * Ensure elapsed is always returned * Added changelog fragment
This commit is contained in:
parent
af3bac1320
commit
b64e666643
11 changed files with 141 additions and 90 deletions
|
@ -1,7 +1,7 @@
|
|||
#!/usr/bin/python
|
||||
# -*- coding: utf-8 -*-
|
||||
|
||||
# (c) 2012, Jan-Piet Mens <jpmens () gmail.com>
|
||||
# Copyright: (c) 2012, Jan-Piet Mens <jpmens () gmail.com>
|
||||
# GNU General Public License v3.0+ (see COPYING or https://www.gnu.org/licenses/gpl-3.0.txt)
|
||||
|
||||
from __future__ import absolute_import, division, print_function
|
||||
|
@ -226,6 +226,11 @@ dest:
|
|||
returned: success
|
||||
type: string
|
||||
sample: /path/to/file.txt
|
||||
elapsed:
|
||||
description: The number of seconds that elapsed while performing the download
|
||||
returned: always
|
||||
type: int
|
||||
sample: 23
|
||||
gid:
|
||||
description: group id of the file
|
||||
returned: success
|
||||
|
@ -268,7 +273,7 @@ size:
|
|||
sample: 1220
|
||||
src:
|
||||
description: source file used after download
|
||||
returned: changed
|
||||
returned: always
|
||||
type: string
|
||||
sample: /tmp/tmpAdFLdV
|
||||
state:
|
||||
|
@ -327,17 +332,19 @@ def url_get(module, url, dest, use_proxy, last_mod_time, force, timeout=10, head
|
|||
else:
|
||||
method = 'GET'
|
||||
|
||||
start = datetime.datetime.utcnow()
|
||||
rsp, info = fetch_url(module, url, use_proxy=use_proxy, force=force, last_mod_time=last_mod_time, timeout=timeout, headers=headers, method=method)
|
||||
elapsed = (datetime.datetime.utcnow() - start).seconds
|
||||
|
||||
if info['status'] == 304:
|
||||
module.exit_json(url=url, dest=dest, changed=False, msg=info.get('msg', ''))
|
||||
module.exit_json(url=url, dest=dest, changed=False, msg=info.get('msg', ''), elapsed=elapsed)
|
||||
|
||||
# Exceptions in fetch_url may result in a status -1, the ensures a proper error to the user in all cases
|
||||
if info['status'] == -1:
|
||||
module.fail_json(msg=info['msg'], url=url, dest=dest)
|
||||
module.fail_json(msg=info['msg'], url=url, dest=dest, elapsed=elapsed)
|
||||
|
||||
if info['status'] != 200 and not url.startswith('file:/') and not (url.startswith('ftp:/') and info.get('msg', '').startswith('OK')):
|
||||
module.fail_json(msg="Request failed", status_code=info['status'], response=info['msg'], url=url, dest=dest)
|
||||
module.fail_json(msg="Request failed", status_code=info['status'], response=info['msg'], url=url, dest=dest, elapsed=elapsed)
|
||||
|
||||
# create a temporary file and copy content to do checksum-based replacement
|
||||
if tmp_dest:
|
||||
|
@ -345,9 +352,9 @@ def url_get(module, url, dest, use_proxy, last_mod_time, force, timeout=10, head
|
|||
tmp_dest_is_dir = os.path.isdir(tmp_dest)
|
||||
if not tmp_dest_is_dir:
|
||||
if os.path.exists(tmp_dest):
|
||||
module.fail_json(msg="%s is a file but should be a directory." % tmp_dest)
|
||||
module.fail_json(msg="%s is a file but should be a directory." % tmp_dest, elapsed=elapsed)
|
||||
else:
|
||||
module.fail_json(msg="%s directory does not exist." % tmp_dest)
|
||||
module.fail_json(msg="%s directory does not exist." % tmp_dest, elapsed=elapsed)
|
||||
else:
|
||||
tmp_dest = module.tmpdir
|
||||
|
||||
|
@ -358,7 +365,7 @@ def url_get(module, url, dest, use_proxy, last_mod_time, force, timeout=10, head
|
|||
shutil.copyfileobj(rsp, f)
|
||||
except Exception as e:
|
||||
os.remove(tempname)
|
||||
module.fail_json(msg="failed to create temporary content file: %s" % to_native(e), exception=traceback.format_exc())
|
||||
module.fail_json(msg="failed to create temporary content file: %s" % to_native(e), elapsed=elapsed, exception=traceback.format_exc())
|
||||
f.close()
|
||||
rsp.close()
|
||||
return tempname, info
|
||||
|
@ -418,6 +425,15 @@ def main():
|
|||
timeout = module.params['timeout']
|
||||
tmp_dest = module.params['tmp_dest']
|
||||
|
||||
result = dict(
|
||||
changed=False,
|
||||
checksum_dest=None,
|
||||
checksum_src=None,
|
||||
dest=dest,
|
||||
elapsed=0,
|
||||
url=url,
|
||||
)
|
||||
|
||||
# Parse headers to dict
|
||||
if isinstance(module.params['headers'], dict):
|
||||
headers = module.params['headers']
|
||||
|
@ -426,7 +442,7 @@ def main():
|
|||
headers = dict(item.split(':', 1) for item in module.params['headers'].split(','))
|
||||
module.deprecate('Supplying `headers` as a string is deprecated. Please use dict/hash format for `headers`', version='2.10')
|
||||
except Exception:
|
||||
module.fail_json(msg="The string representation for the `headers` parameter requires a key:value,key:value syntax to be properly parsed.")
|
||||
module.fail_json(msg="The string representation for the `headers` parameter requires a key:value,key:value syntax to be properly parsed.", **result)
|
||||
else:
|
||||
headers = None
|
||||
|
||||
|
@ -456,7 +472,7 @@ def main():
|
|||
# Ensure the checksum portion is a hexdigest
|
||||
int(checksum, 16)
|
||||
except ValueError:
|
||||
module.fail_json(msg="The checksum parameter has to be in format <algorithm>:<checksum>")
|
||||
module.fail_json(msg="The checksum parameter has to be in format <algorithm>:<checksum>", **result)
|
||||
|
||||
if not dest_is_dir and os.path.exists(dest):
|
||||
checksum_mismatch = False
|
||||
|
@ -472,10 +488,10 @@ def main():
|
|||
module.params['path'] = dest
|
||||
file_args = module.load_file_common_arguments(module.params)
|
||||
file_args['path'] = dest
|
||||
changed = module.set_fs_attributes_if_different(file_args, False)
|
||||
if changed:
|
||||
module.exit_json(msg="file already exists but file attributes changed", dest=dest, url=url, changed=changed)
|
||||
module.exit_json(msg="file already exists", dest=dest, url=url, changed=changed)
|
||||
result['changed'] = module.set_fs_attributes_if_different(file_args, False)
|
||||
if result['changed']:
|
||||
module.exit_json(msg="file already exists but file attributes changed", **result)
|
||||
module.exit_json(msg="file already exists", **result)
|
||||
|
||||
checksum_mismatch = True
|
||||
|
||||
|
@ -490,7 +506,10 @@ def main():
|
|||
force = True
|
||||
|
||||
# download to tmpsrc
|
||||
start = datetime.datetime.utcnow()
|
||||
tmpsrc, info = url_get(module, url, dest, use_proxy, last_mod_time, force, timeout, headers, tmp_dest)
|
||||
result['elapsed'] = (datetime.datetime.utcnow() - start).seconds
|
||||
result['src'] = tmpsrc
|
||||
|
||||
# Now the request has completed, we can finally generate the final
|
||||
# destination file name from the info dict.
|
||||
|
@ -504,44 +523,41 @@ def main():
|
|||
filename = url_filename(info['url'])
|
||||
dest = os.path.join(dest, filename)
|
||||
|
||||
checksum_src = None
|
||||
checksum_dest = None
|
||||
|
||||
# If the remote URL exists, we're done with check mode
|
||||
if module.check_mode:
|
||||
os.remove(tmpsrc)
|
||||
res_args = dict(url=url, dest=dest, src=tmpsrc, changed=True, msg=info.get('msg', ''))
|
||||
module.exit_json(**res_args)
|
||||
result['changed'] = True
|
||||
module.exit_json(msg=info.get('msg', ''), **result)
|
||||
|
||||
# raise an error if there is no tmpsrc file
|
||||
if not os.path.exists(tmpsrc):
|
||||
os.remove(tmpsrc)
|
||||
module.fail_json(msg="Request failed", status_code=info['status'], response=info['msg'])
|
||||
module.fail_json(msg="Request failed", status_code=info['status'], response=info['msg'], **result)
|
||||
if not os.access(tmpsrc, os.R_OK):
|
||||
os.remove(tmpsrc)
|
||||
module.fail_json(msg="Source %s is not readable" % (tmpsrc))
|
||||
checksum_src = module.sha1(tmpsrc)
|
||||
module.fail_json(msg="Source %s is not readable" % (tmpsrc), **result)
|
||||
result['checksum_src'] = module.sha1(tmpsrc)
|
||||
|
||||
# check if there is no dest file
|
||||
if os.path.exists(dest):
|
||||
# raise an error if copy has no permission on dest
|
||||
if not os.access(dest, os.W_OK):
|
||||
os.remove(tmpsrc)
|
||||
module.fail_json(msg="Destination %s is not writable" % (dest))
|
||||
module.fail_json(msg="Destination %s is not writable" % (dest), **result)
|
||||
if not os.access(dest, os.R_OK):
|
||||
os.remove(tmpsrc)
|
||||
module.fail_json(msg="Destination %s is not readable" % (dest))
|
||||
checksum_dest = module.sha1(dest)
|
||||
module.fail_json(msg="Destination %s is not readable" % (dest), **result)
|
||||
result['checksum_dest'] = module.sha1(dest)
|
||||
else:
|
||||
if not os.path.exists(os.path.dirname(dest)):
|
||||
os.remove(tmpsrc)
|
||||
module.fail_json(msg="Destination %s does not exist" % (os.path.dirname(dest)))
|
||||
module.fail_json(msg="Destination %s does not exist" % (os.path.dirname(dest)), **result)
|
||||
if not os.access(os.path.dirname(dest), os.W_OK):
|
||||
os.remove(tmpsrc)
|
||||
module.fail_json(msg="Destination %s is not writable" % (os.path.dirname(dest)))
|
||||
module.fail_json(msg="Destination %s is not writable" % (os.path.dirname(dest)), **result)
|
||||
|
||||
backup_file = None
|
||||
if checksum_src != checksum_dest:
|
||||
if result['checksum_src'] != result['checksum_dest']:
|
||||
try:
|
||||
if backup:
|
||||
if os.path.exists(dest):
|
||||
|
@ -551,10 +567,10 @@ def main():
|
|||
if os.path.exists(tmpsrc):
|
||||
os.remove(tmpsrc)
|
||||
module.fail_json(msg="failed to copy %s to %s: %s" % (tmpsrc, dest, to_native(e)),
|
||||
exception=traceback.format_exc())
|
||||
changed = True
|
||||
exception=traceback.format_exc(), **result)
|
||||
result['changed'] = True
|
||||
else:
|
||||
changed = False
|
||||
result['changed'] = False
|
||||
if os.path.exists(tmpsrc):
|
||||
os.remove(tmpsrc)
|
||||
|
||||
|
@ -563,29 +579,25 @@ def main():
|
|||
|
||||
if checksum != destination_checksum:
|
||||
os.remove(dest)
|
||||
module.fail_json(msg="The checksum for %s did not match %s; it was %s." % (dest, checksum, destination_checksum))
|
||||
module.fail_json(msg="The checksum for %s did not match %s; it was %s." % (dest, checksum, destination_checksum), **result)
|
||||
|
||||
# allow file attribute changes
|
||||
module.params['path'] = dest
|
||||
file_args = module.load_file_common_arguments(module.params)
|
||||
file_args['path'] = dest
|
||||
changed = module.set_fs_attributes_if_different(file_args, changed)
|
||||
result['changed'] = module.set_fs_attributes_if_different(file_args, result['changed'])
|
||||
|
||||
# Backwards compat only. We'll return None on FIPS enabled systems
|
||||
try:
|
||||
md5sum = module.md5(dest)
|
||||
result['md5sum'] = module.md5(dest)
|
||||
except ValueError:
|
||||
md5sum = None
|
||||
result['md5sum'] = None
|
||||
|
||||
res_args = dict(
|
||||
url=url, dest=dest, src=tmpsrc, md5sum=md5sum, checksum_src=checksum_src,
|
||||
checksum_dest=checksum_dest, changed=changed, msg=info.get('msg', ''), status_code=info.get('status', '')
|
||||
)
|
||||
if backup_file:
|
||||
res_args['backup_file'] = backup_file
|
||||
result['backup_file'] = backup_file
|
||||
|
||||
# Mission complete
|
||||
module.exit_json(**res_args)
|
||||
module.exit_json(msg=info.get('msg', ''), status_code=info.get('status', ''), **result)
|
||||
|
||||
|
||||
if __name__ == '__main__':
|
||||
|
|
|
@ -54,8 +54,8 @@ options:
|
|||
method:
|
||||
description:
|
||||
- The HTTP method of the request or response. It MUST be uppercase.
|
||||
choices: [ "GET", "POST", "PUT", "HEAD", "DELETE", "OPTIONS", "PATCH", "TRACE", "CONNECT", "REFRESH" ]
|
||||
default: "GET"
|
||||
choices: [ CONNECT, DELETE, GET, HEAD, OPTIONS, PATCH, POST, PUT, REFRESH, TRACE ]
|
||||
default: GET
|
||||
return_content:
|
||||
description:
|
||||
- Whether or not to return the body of the response as a "content" key in
|
||||
|
@ -234,6 +234,11 @@ EXAMPLES = r'''
|
|||
|
||||
RETURN = r'''
|
||||
# The return information includes all the HTTP headers in lower-case.
|
||||
elapsed:
|
||||
description: The number of seconds that elapsed while performing the download
|
||||
returned: always
|
||||
type: int
|
||||
sample: 23
|
||||
msg:
|
||||
description: The HTTP message from the request
|
||||
returned: always
|
||||
|
@ -275,7 +280,7 @@ from ansible.module_utils.urls import fetch_url, url_argument_spec
|
|||
JSON_CANDIDATES = ('text', 'json', 'javascript')
|
||||
|
||||
|
||||
def write_file(module, url, dest, content):
|
||||
def write_file(module, url, dest, content, resp):
|
||||
# create a tempfile with some test content
|
||||
fd, tmpsrc = tempfile.mkstemp(dir=module.tmpdir)
|
||||
f = open(tmpsrc, 'wb')
|
||||
|
@ -284,7 +289,7 @@ def write_file(module, url, dest, content):
|
|||
except Exception as e:
|
||||
os.remove(tmpsrc)
|
||||
module.fail_json(msg="failed to create temporary content file: %s" % to_native(e),
|
||||
exception=traceback.format_exc())
|
||||
exception=traceback.format_exc(), **resp)
|
||||
f.close()
|
||||
|
||||
checksum_src = None
|
||||
|
@ -293,10 +298,10 @@ def write_file(module, url, dest, content):
|
|||
# raise an error if there is no tmpsrc file
|
||||
if not os.path.exists(tmpsrc):
|
||||
os.remove(tmpsrc)
|
||||
module.fail_json(msg="Source '%s' does not exist" % tmpsrc)
|
||||
module.fail_json(msg="Source '%s' does not exist" % tmpsrc, **resp)
|
||||
if not os.access(tmpsrc, os.R_OK):
|
||||
os.remove(tmpsrc)
|
||||
module.fail_json(msg="Source '%s' not readable" % tmpsrc)
|
||||
module.fail_json(msg="Source '%s' not readable" % tmpsrc, **resp)
|
||||
checksum_src = module.sha1(tmpsrc)
|
||||
|
||||
# check if there is no dest file
|
||||
|
@ -304,15 +309,15 @@ def write_file(module, url, dest, content):
|
|||
# raise an error if copy has no permission on dest
|
||||
if not os.access(dest, os.W_OK):
|
||||
os.remove(tmpsrc)
|
||||
module.fail_json(msg="Destination '%s' not writable" % dest)
|
||||
module.fail_json(msg="Destination '%s' not writable" % dest, **resp)
|
||||
if not os.access(dest, os.R_OK):
|
||||
os.remove(tmpsrc)
|
||||
module.fail_json(msg="Destination '%s' not readable" % dest)
|
||||
module.fail_json(msg="Destination '%s' not readable" % dest, **resp)
|
||||
checksum_dest = module.sha1(dest)
|
||||
else:
|
||||
if not os.access(os.path.dirname(dest), os.W_OK):
|
||||
os.remove(tmpsrc)
|
||||
module.fail_json(msg="Destination dir '%s' not writable" % os.path.dirname(dest))
|
||||
module.fail_json(msg="Destination dir '%s' not writable" % os.path.dirname(dest), **resp)
|
||||
|
||||
if checksum_src != checksum_dest:
|
||||
try:
|
||||
|
@ -320,7 +325,7 @@ def write_file(module, url, dest, content):
|
|||
except Exception as e:
|
||||
os.remove(tmpsrc)
|
||||
module.fail_json(msg="failed to copy %s to %s: %s" % (tmpsrc, dest, to_native(e)),
|
||||
exception=traceback.format_exc())
|
||||
exception=traceback.format_exc(), **resp)
|
||||
|
||||
os.remove(tmpsrc)
|
||||
|
||||
|
@ -401,7 +406,7 @@ def uri(module, url, dest, body, body_format, method, headers, socket_timeout):
|
|||
})
|
||||
data = open(src, 'rb')
|
||||
except OSError:
|
||||
module.fail_json(msg='Unable to open source file %s' % src, exception=traceback.format_exc())
|
||||
module.fail_json(msg='Unable to open source file %s' % src, exception=traceback.format_exc(), elapsed=0)
|
||||
else:
|
||||
data = body
|
||||
|
||||
|
@ -463,7 +468,7 @@ def main():
|
|||
body=dict(type='raw'),
|
||||
body_format=dict(type='str', default='raw', choices=['form-urlencoded', 'json', 'raw']),
|
||||
src=dict(type='path'),
|
||||
method=dict(type='str', default='GET', choices=['GET', 'POST', 'PUT', 'HEAD', 'DELETE', 'OPTIONS', 'PATCH', 'TRACE', 'CONNECT', 'REFRESH']),
|
||||
method=dict(type='str', default='GET', choices=['CONNECT', 'DELETE', 'GET', 'HEAD', 'OPTIONS', 'PATCH', 'POST', 'PUT', 'REFRESH', 'TRACE']),
|
||||
return_content=dict(type='bool', default=False),
|
||||
follow_redirects=dict(type='str', default='safe', choices=['all', 'no', 'none', 'safe', 'urllib2', 'yes']),
|
||||
creates=dict(type='path'),
|
||||
|
@ -505,7 +510,7 @@ def main():
|
|||
try:
|
||||
body = form_urlencoded(body)
|
||||
except ValueError as e:
|
||||
module.fail_json(msg='failed to parse body as form_urlencoded: %s' % to_native(e))
|
||||
module.fail_json(msg='failed to parse body as form_urlencoded: %s' % to_native(e), elapsed=0)
|
||||
if 'content-type' not in [header.lower() for header in dict_headers]:
|
||||
dict_headers['Content-Type'] = 'application/x-www-form-urlencoded'
|
||||
|
||||
|
@ -525,35 +530,37 @@ def main():
|
|||
# and the filename already exists. This allows idempotence
|
||||
# of uri executions.
|
||||
if os.path.exists(creates):
|
||||
module.exit_json(stdout="skipped, since '%s' exists" % creates, changed=False, rc=0)
|
||||
module.exit_json(stdout="skipped, since '%s' exists" % creates, changed=False)
|
||||
|
||||
if removes is not None:
|
||||
# do not run the command if the line contains removes=filename
|
||||
# and the filename does not exist. This allows idempotence
|
||||
# of uri executions.
|
||||
if not os.path.exists(removes):
|
||||
module.exit_json(stdout="skipped, since '%s' does not exist" % removes, changed=False, rc=0)
|
||||
module.exit_json(stdout="skipped, since '%s' does not exist" % removes, changed=False)
|
||||
|
||||
# Make the request
|
||||
start = datetime.datetime.utcnow()
|
||||
resp, content, dest = uri(module, url, dest, body, body_format, method,
|
||||
dict_headers, socket_timeout)
|
||||
resp['elapsed'] = (datetime.datetime.utcnow() - start).seconds
|
||||
resp['status'] = int(resp['status'])
|
||||
|
||||
# Write the file out if requested
|
||||
if dest is not None:
|
||||
if resp['status'] == 304:
|
||||
changed = False
|
||||
resp['changed'] = False
|
||||
else:
|
||||
write_file(module, url, dest, content)
|
||||
write_file(module, url, dest, content, resp)
|
||||
# allow file attribute changes
|
||||
changed = True
|
||||
resp['changed'] = True
|
||||
module.params['path'] = dest
|
||||
file_args = module.load_file_common_arguments(module.params)
|
||||
file_args['path'] = dest
|
||||
changed = module.set_fs_attributes_if_different(file_args, changed)
|
||||
resp['changed'] = module.set_fs_attributes_if_different(file_args, resp['changed'])
|
||||
resp['path'] = dest
|
||||
else:
|
||||
changed = False
|
||||
resp['changed'] = False
|
||||
|
||||
# Transmogrify the headers, replacing '-' with '_', since variables don't
|
||||
# work with dashes.
|
||||
|
@ -588,9 +595,9 @@ def main():
|
|||
uresp['msg'] = 'Status code was %s and not %s: %s' % (resp['status'], status_code, uresp.get('msg', ''))
|
||||
module.fail_json(content=u_content, **uresp)
|
||||
elif return_content:
|
||||
module.exit_json(changed=changed, content=u_content, **uresp)
|
||||
module.exit_json(content=u_content, **uresp)
|
||||
else:
|
||||
module.exit_json(changed=changed, **uresp)
|
||||
module.exit_json(**uresp)
|
||||
|
||||
|
||||
if __name__ == '__main__':
|
||||
|
|
Loading…
Add table
Add a link
Reference in a new issue