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.

Sign up for GitHub

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

Jump to bottom

fix chuid argument order #7616

Merged
merged 1 commit into from
Nov 17, 2016
Merged

fix chuid argument order #7616

merged 1 commit into from
Nov 17, 2016

Conversation

ccasey
Copy link

@ccasey ccasey commented Nov 10, 2016 •
edited
Loading

Required for all non-trivial PRs

( think this is trivial)

  • Rebased/mergable
  • Tests pass
  • CHANGELOG.md updated
  • Sign CLA (if not already signed)

Changing GROUP ends up ultimately changing the USER:

# User and group
USER=influxdb
GROUP=daemon

Current:

drwxr-xr-x 2 daemon   influxdb 4096 Nov 10 22:15 data/
drwxr-xr-x 2 daemon   influxdb 4096 Nov 10 22:15 meta/

w/fix:

drwxr-xr-x 2 influxdb daemon 4096 Nov 10 22:28 data/
drwxr-xr-x 2 influxdb daemon 4096 Nov 10 22:28 meta/

@jwilder
Copy link
Contributor

jwilder commented Nov 15, 2016

@ccasey Can you add this to the changelog for 1.2 section?

@jwilder jwilder added this to the 1.2.0 milestone Nov 15, 2016
@jwilder jwilder merged commit bdf389d into influxdata:master Nov 17, 2016
jwilder added a commit that referenced this pull request Nov 17, 2016
@jwilder
Update changelog
63554ce 
Add #7616
@ccasey
Copy link
Author

ccasey commented Nov 17, 2016

Sorry you beat me to the changelog, thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jwilder jwilder jwilder approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
1.2.0
Development

Successfully merging this pull request may close these issues.

None yet
2 participants
@ccasey @jwilder