-
Notifications
You must be signed in to change notification settings - Fork 683
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
group resource: Modified DarwinGroup to collect all users properly on macos #4343
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You'll need to chefstyle -a
=)
And probably to convert everything over to command resources? This was sloppy and only tested locally. Not sure HOW to test this tbh.. BK doesn't do mac yet. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
While it may seem unlikely someone would use a Mac machine as a remote scan target, we have to be careful to keep local execution separate from remote execution - so we need to check the backticks.
0a9cbf8
to
800eca4
Compare
I noticed that my profile targeting group "staff" on OSX wasn't returning *me*. I'm awesome! So that seemed wrong. And it turns out it is wrong. We were not collecting any users whose primary group was that group. (almost all regular users are in group staff and they're all missing). Signed-off-by: Ryan Davis <zenspider@chef.io>
800eca4
to
374e365
Compare
Code Climate has analyzed commit 374e365 and detected 2 issues on this pull request. Here's the issue category breakdown:
View more on Code Climate. |
I noticed that my profile targeting group "staff" on OSX wasn't
returning ME. That seemed wrong. And it turns out it is wrong. We were
not collecting any users whose primary group was that group. (almost
all regular users are in group staff and they're all missing).
I ALSO found out that our MockLoader-based testing is piss-poor and,
through much pain and suffering, figured out how to test the group
resource directly.
Signed-off-by: Ryan Davis zenspider@chef.io