execute the mount after path validation
This avoids a stale situation where name/path contains some impossible path, but gets configured (faultly) in fstab, and the module only fails after that, when creating that path.
This commit is contained in:
parent
e90d2573d1
commit
25b696c2d2
1 changed files with 2 additions and 1 deletions
|
@ -307,7 +307,6 @@ def main():
|
||||||
module.exit_json(changed=changed, **args)
|
module.exit_json(changed=changed, **args)
|
||||||
|
|
||||||
if state in ['mounted', 'present']:
|
if state in ['mounted', 'present']:
|
||||||
name, changed = set_mount(**args)
|
|
||||||
if state == 'mounted':
|
if state == 'mounted':
|
||||||
if not os.path.exists(name):
|
if not os.path.exists(name):
|
||||||
try:
|
try:
|
||||||
|
@ -315,6 +314,8 @@ def main():
|
||||||
except (OSError, IOError), e:
|
except (OSError, IOError), e:
|
||||||
module.fail_json(msg="Error making dir %s: %s" % (name, str(e)))
|
module.fail_json(msg="Error making dir %s: %s" % (name, str(e)))
|
||||||
|
|
||||||
|
name, changed = set_mount(**args)
|
||||||
|
if state == 'mounted':
|
||||||
res = 0
|
res = 0
|
||||||
if os.path.ismount(name):
|
if os.path.ismount(name):
|
||||||
if changed:
|
if changed:
|
||||||
|
|
Loading…
Reference in a new issue