Skip to content
This repository has been archived by the owner on Jan 10, 2023. It is now read-only.

Add repo permissions support #99

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

drrk
Copy link

@drrk drrk commented Jun 25, 2015

Add the ability to control the permissions of a repo that is managed with the LWRP.

Adds 6 attributes for admin, write and read permissions for users and groups in stash. It will ensure that the permissions set in the LWRP are applied, both to add and remove users/groups.

In addition this PR slightly alters the behaviour of the chef-stash install, to only override the gem source from the system default if specified in the attributes.

@bflad
Copy link
Owner

bflad commented Jul 12, 2015

I copied the other maintainers on #109 to see if we should worry about adding tests before/after this PR so you don't have to wait. Have you been using this in a branch okay? If you have ideas for LWRP testing, please feel free to comment on that other issue. Thank you!

@drrk
Copy link
Author

drrk commented Jul 12, 2015

I have been using it at a customers site, although that was before I cleaned up the branch for the PR. Sorry i've not looked at this in the last couple of weeks, I have been busy with another customer. I will have another look at fixing the remaining rubocop errors this week, and will discuss with some colleagues about LWRP testing strategies.

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

Successfully merging this pull request may close these issues.

2 participants