Merge pull request #2947 from mattclay/user-check-mode-freebsd
Make modify_user honor check mode on FreeBSD.
This commit is contained in:
commit
50bfacb0da
1 changed files with 4 additions and 0 deletions
|
@ -864,6 +864,8 @@ class FreeBsdUser(User):
|
||||||
|
|
||||||
# modify the user if cmd will do anything
|
# modify the user if cmd will do anything
|
||||||
if cmd_len != len(cmd):
|
if cmd_len != len(cmd):
|
||||||
|
if self.module.check_mode:
|
||||||
|
return (0, '', '')
|
||||||
(rc, out, err) = self.execute_command(cmd)
|
(rc, out, err) = self.execute_command(cmd)
|
||||||
if rc is not None and rc != 0:
|
if rc is not None and rc != 0:
|
||||||
self.module.fail_json(name=self.name, msg=err, rc=rc)
|
self.module.fail_json(name=self.name, msg=err, rc=rc)
|
||||||
|
@ -872,6 +874,8 @@ class FreeBsdUser(User):
|
||||||
|
|
||||||
# we have to set the password in a second command
|
# we have to set the password in a second command
|
||||||
if self.update_password == 'always' and self.password is not None and info[1] != self.password:
|
if self.update_password == 'always' and self.password is not None and info[1] != self.password:
|
||||||
|
if self.module.check_mode:
|
||||||
|
return (0, '', '')
|
||||||
cmd = [
|
cmd = [
|
||||||
self.module.get_bin_path('chpass', True),
|
self.module.get_bin_path('chpass', True),
|
||||||
'-p',
|
'-p',
|
||||||
|
|
Loading…
Reference in a new issue