users module should allow smb parameter when password_disabled is defined about ansible-truenas HOT 13 CLOSED

I apologize for taking this long to get back to this project. I've just pushed out release 1.9.1, which should fix this problem. Please try it out.

from ansible-truenas.

Error is

• user_create.password_disabled: Password authentication may not be disabled for SMB users

from ansible-truenas.

@steven-ellis Do you have a playbook that illustrates the problem you're seeing?

from ansible-truenas.

Here is an example

   - name: Make sure  mythtv user exists
      arensb.truenas.user:
        name: mythtv
        uid: 1000
        groups: 'users,mythtv'
        password_disabled: yes

from ansible-truenas.

I haven't been able to reproduce this problem under TrueNAS Core. In my own testing, I was able to create users with password_disabled: yes, both with and without SMB enabled. Is this something that happens when a user is created? When an existing user is changed? Both?

And what flavor and version of TrueNAS are you using?

from ansible-truenas.

TrueNAS Scale 23.10.2

This is happening when I try to create a new user. As a default a new user has smb defaulting to true which conflicts with password_disabled.

from ansible-truenas.

Sorry I've been neglecting these bugs. I've been dealing with family matters.

from ansible-truenas.

Sorry, I created a duplicate issue with #17

Have created a PR with a suggested fix.

This is an issue with TrueNAS scale and is still present with 24.4.0

from ansible-truenas.

Have linked PR to this issue now #17 has been closed as duplicate

from ansible-truenas.

@arensb Worked good for me!

I had some issues installing the updated collection via ansible-galaxy, but overwriting the installed user module with the updated file worked a treat.

(ansible-galaxy collection install arensb.truenas --force --pre installed 1.9.0 again 🤦 )

from ansible-truenas.

Thank you for contributing a fix 😄

from ansible-truenas.

@MozzieBytes Oh, sorry, yeah, I marked this as a tentative release on github, and haven't uploaded it to Ansible Galaxy. I was hoping to get some time to work on issues #13 and following in the coming days (we'll see if that works. I think the universe has other plans for me).

from ansible-truenas.

@MozzieBytes Oh, sorry, yeah, I marked this as a tentative release on github, and haven't uploaded it to Ansible Galaxy. I was hoping to get some time to work on issues #13 and following in the coming days (we'll see if that works. I think the universe has other plans for me).

Sorry about that! I'm still new to Ansible so am learning on the go with the nuances of how ansible-galaxy and collections work.

I'm happy to give the NFSv4 issue a go if that's at all helpful? Your description of how you see the issue being handled is enough that I could probably put together an implementation.

from ansible-truenas.