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

New instance parameter 'additional_groups' #27

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ruriky
Copy link
Contributor

@ruriky ruriky commented Nov 10, 2017

User can define additional groups for instance and fenced users.
Multiple groups can be defined as an array.

This change was needed because our db2 users need be able to become instance and fenced users using sudo command. There is an easy way of allowing users to become other users which are in the same group.

@crayfishx
Copy link
Owner

Thanks @ruriky - seems a good feature to have - I find the naming convension of "additional groups" a bit confusing though - it suggests there is somewhere else to set groups too, which there isn't - does this make more sense to just call the parameter groups??

User can define supplementary groups for instance and fenced users.
Multiple groups can be defined as an array.
@ruriky
Copy link
Contributor Author

ruriky commented Nov 13, 2017

Thanks for the comment @crayfishx - that makes sense. I changed the parameter name to 'groups'. Also in README.md file I switched 'additional groups' to 'supplementary groups' as that is how the man pages seem to describe them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants