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

[stable-2.8] Omit -A and -G options in local mode since luseradd does not support these (#55401) #58480

Open
wants to merge 2 commits into
base: stable-2.8
from

Conversation

Projects
None yet
3 participants
@samdoran
Copy link
Member

commented Jun 27, 2019

SUMMARY

Backport of #55401 for Ansible 2.8

ISSUE TYPE
  • Bugfix Pull Request
COMPONENT NAME

lib/ansible/modules/system/user.py

[stable-2.8] Omit -A and -G options in local mode since luseradd does…
… not support these (#55401)

Add integration tests
(cherry picked from commit 20ad120)

Co-authored-by: Sam Doran <sdoran@redhat.com>
@ansibot

This comment has been minimized.

Copy link
Contributor

commented Jun 27, 2019

@abadger

This comment has been minimized.

Copy link
Member

commented Jun 29, 2019

This doesn't look complete. Looking at #48722 (comment) we still need to add something to argspec to error out if local and the group options are both specified.

Also probably a good idea to open a new feature request to add lgroupmod to support those parameters with local=True at a later time.

@ansibot ansibot removed the needs_triage label Jun 29, 2019

@samdoran

This comment has been minimized.

Copy link
Member Author

commented Jul 1, 2019

Roger. I'll add another PR upstream for the mutually exclusive params.

User - make groups and append mutually exclusive with local (#59309)
* Update intigration tests

(cherry picked from commit 8edad83)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.