mirror of
https://github.com/ansible-collections/community.general.git
synced 2024-09-14 20:13:21 +02:00
BUGFIX 1178: Adding better existence check in one place. db_import now fails fast if target does not exist b/c no point going further without the target.
This commit is contained in:
parent
b09a144724
commit
28c9104ed5
1 changed files with 5 additions and 2 deletions
|
@ -140,6 +140,9 @@ def db_dump(module, host, user, password, db_name, target, port, socket=None):
|
||||||
return rc, stdout, stderr
|
return rc, stdout, stderr
|
||||||
|
|
||||||
def db_import(module, host, user, password, db_name, target, port, socket=None):
|
def db_import(module, host, user, password, db_name, target, port, socket=None):
|
||||||
|
if not os.path.exists(target):
|
||||||
|
return module.fail_json(msg="target %s does not exist on the host" % target)
|
||||||
|
|
||||||
cmd = module.get_bin_path('mysql', True)
|
cmd = module.get_bin_path('mysql', True)
|
||||||
cmd += " --user=%s --password=%s" % (pipes.quote(user), pipes.quote(password))
|
cmd += " --user=%s --password=%s" % (pipes.quote(user), pipes.quote(password))
|
||||||
if socket is not None:
|
if socket is not None:
|
||||||
|
@ -148,9 +151,9 @@ def db_import(module, host, user, password, db_name, target, port, socket=None):
|
||||||
cmd += " --host=%s --port=%s" % (pipes.quote(host), pipes.quote(port))
|
cmd += " --host=%s --port=%s" % (pipes.quote(host), pipes.quote(port))
|
||||||
cmd += " -D %s" % pipes.quote(db_name)
|
cmd += " -D %s" % pipes.quote(db_name)
|
||||||
if os.path.splitext(target)[-1] == '.gz':
|
if os.path.splitext(target)[-1] == '.gz':
|
||||||
cmd = 'test -e ' + pipes.quote(target) + ' && gunzip < ' + pipes.quote(target) + ' | ' + cmd
|
cmd = 'gunzip < ' + pipes.quote(target) + ' | ' + cmd
|
||||||
elif os.path.splitext(target)[-1] == '.bz2':
|
elif os.path.splitext(target)[-1] == '.bz2':
|
||||||
cmd = 'test -e ' + pipes.quote(target) + ' && bunzip2 < ' + pipes.quote(target) + ' | ' + cmd
|
cmd = 'bunzip2 < ' + pipes.quote(target) + ' | ' + cmd
|
||||||
else:
|
else:
|
||||||
cmd += " < %s" % pipes.quote(target)
|
cmd += " < %s" % pipes.quote(target)
|
||||||
rc, stdout, stderr = module.run_command(cmd, use_unsafe_shell=True)
|
rc, stdout, stderr = module.run_command(cmd, use_unsafe_shell=True)
|
||||||
|
|
Loading…
Reference in a new issue