Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

user.add problem to windows #53363

Closed
Dtengfei opened this issue Jun 4, 2019 · 4 comments
Closed

user.add problem to windows #53363

Dtengfei opened this issue Jun 4, 2019 · 4 comments
Assignees
Labels
Bug broken, incorrect, or confusing behavior severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around Windows
Milestone

Comments

@Dtengfei
Copy link

Dtengfei commented Jun 4, 2019

Description of Issue

I executed "salt win15-ky.cq3 user.add name=" 2019123" groups=" Remote Desktop Users" on salt-master to prompt users to be int type. I don't know how pure digital users like me can add it? It seems that all operations involving Windows user are problematic. User. update is no good either.

Setup

(Please provide relevant configs and/or SLS files (Be sure to remove sensitive info).)
https://docs.saltstack.com/en/latest/ref/modules/all/salt.modules.useradd.html
salt.modules.useradd.add(name, uid=None, gid=None, groups=None, home=None, shell=None, unique=True, system=False, fullname='', roomnumber='', workphone='', homephone='', other='', createhome=True, loginclass=None, root=None, nologinit=False)

Steps to Reproduce Issue

(Include debug logs if possible and relevant.)
` Passed invalid arguments to user.add: Objects of type 'int' can not be converted to Unicode.

    Add a user to the minion.

    Args:
        name (str): User name

        password (str, optional): User's password in plain text.

        fullname (str, optional): The user's full name.

        description (str, optional): A brief description of the user account.

        groups (str, optional): A list of groups to add the user to.
            (see chgroups)

        home (str, optional): The path to the user's home directory.

        homedrive (str, optional): The drive letter to assign to the home
            directory. Must be the Drive Letter followed by a colon. ie: U:

        profile (str, optional): An explicit path to a profile. Can be a UNC or
            a folder on the system. If left blank, windows uses it's default
            profile directory.

        logonscript (str, optional): Path to a login script to run when the user
            logs on.

    Returns:
        bool: True if successful. False is unsuccessful.

    CLI Example:

    .. code-block:: bash

        salt '*' user.add name password`

Versions Report

(Provided by running salt --versions-report. Please also mention any differences in master/minion versions.)
`Salt Version:
Salt: 2019.2.0

Dependency Versions:
cffi: 1.5.2
cherrypy: 3.2.3
dateutil: Not Installed
docker-py: Not Installed
gitdb: Not Installed
gitpython: Not Installed
ioflo: Not Installed
Jinja2: 2.7.2
libgit2: 0.26.0
libnacl: Not Installed
M2Crypto: Not Installed
Mako: Not Installed
msgpack-pure: Not Installed
msgpack-python: 0.4.8
mysql-python: Not Installed
pycparser: 2.14
pycrypto: 2.6.1
pycryptodome: Not Installed
pygit2: 0.26.2
Python: 2.7.5 (default, Nov 6 2016, 00:28:07)
python-gnupg: Not Installed
PyYAML: 3.10
PyZMQ: 15.3.0
RAET: Not Installed
smmap: Not Installed
timelib: Not Installed
Tornado: 4.2.1
ZMQ: 4.1.4

System Versions:
dist: centos 7.3.1611 Core
locale: UTF-8
machine: x86_64
release: 3.10.0-514.21.1.el7.x86_64
system: Linux
version: CentOS Linux 7.3.1611 Core`

win15-ky.cq3: 2019.2.0

@cmcmarrow cmcmarrow added the Bug broken, incorrect, or confusing behavior label Jun 4, 2019
@cmcmarrow cmcmarrow added this to the Approved milestone Jun 4, 2019
@stale
Copy link

stale bot commented Jan 8, 2020

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

If this issue is closed prematurely, please leave a comment and we will gladly reopen the issue.

@stale stale bot added the stale label Jan 8, 2020
@Dtengfei
Copy link
Author

Dtengfei commented Jan 8, 2020

This problem hasn't been solved!

@stale
Copy link

stale bot commented Jan 8, 2020

Thank you for updating this issue. It is no longer marked as stale.

@stale stale bot removed the stale label Jan 8, 2020
@sagetherage sagetherage added the severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around label Jun 15, 2021
@twangboy twangboy self-assigned this Mar 4, 2024
@twangboy
Copy link
Contributor

The Above PR should fix this issue, among other issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug broken, incorrect, or confusing behavior severity-medium 3rd level, incorrect or bad functionality, confusing and lacks a work around Windows
Projects
Status: Done
Development

No branches or pull requests

4 participants