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

account for unique user group #664

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

Conversation

lattisimo
Copy link

@lattisimo lattisimo commented Oct 6, 2020

Use id -gn to get the user group for curl|bash stackstorm installer.


The issue I had was installing on a box that had LDAP configured and my user group is 'domain users' since the script just assumed the group matched the user so the chown command fails and the script stops executing. I can avoid the issue by 'sudo su' but I realized that if I get the user's group with 'id -gn' then there is no assumption of what that value should be. This would likely be relevant on the other scripts.

use 'id -gn' to get the user group
Copy link
Member

@nmaludy nmaludy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@arm4b
Copy link
Member

arm4b commented Oct 7, 2020

@lattisimo Would be good to know the more context around this issue: what led you to this solution, what's the issue, error message, etc.

@lattisimo lattisimo requested a review from nmaludy October 8, 2020 01:56
@lattisimo
Copy link
Author

Since this is my first interaction of this sort. I want to follow up if there is any additional action needed on my part? I am not confident that I accomplished what @nmaludy has asked of me.

@nmaludy
Copy link
Member

nmaludy commented Oct 12, 2020

@lattisimo Your changes look good, we're just in a change freeze right now due to a pending 3.3.0 release. I'm going to target this for the 3.4.0 milestone, does that sound good?

@nmaludy nmaludy added this to the 3.4.0 milestone Oct 12, 2020
@lattisimo
Copy link
Author

Sounds good. Thanks.

@CLAassistant
Copy link

CLAassistant commented May 11, 2022

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you all sign our Contributor License Agreement before we can accept your contribution.
1 out of 2 committers have signed the CLA.

✅ lattisimo
❌ Shawn Robinson


Shawn Robinson seems not to be a GitHub user. You need a GitHub account to be able to sign the CLA. If you have already a GitHub account, please add the email address used for this commit to your account.
You have signed the CLA already but the status is still pending? Let us recheck it.

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.

4 participants