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

Add shell option to start the daemon on CentOS #3868

Merged
merged 1 commit into from
Aug 28, 2015
Merged

Add shell option to start the daemon on CentOS #3868

merged 1 commit into from
Aug 28, 2015

Conversation

SwannCroiset
Copy link

Unless a valid shell for the user running influxdb, the command su fails with
the error:
This account is currently not available.

The user shouldn't have a valid shell although the package creates influxdb
user with /bin/bash

Unless a valid shell for the user running influxdb, the command su fails with
the error:
 This account is currently not available.

The user shouldn't have a valid shell although the package creates influxdb
user with /bin/bash
@otoolep
Copy link
Contributor

otoolep commented Aug 27, 2015

Thanks @SwannCroiset -- under what circumstances would a shell not be available for the user?

@otoolep
Copy link
Contributor

otoolep commented Aug 27, 2015

Also @SwannCroiset -- you must sign the CLA before we can merge this.

@otoolep
Copy link
Contributor

otoolep commented Aug 27, 2015

@SwannCroiset
Copy link
Author

CLA signed.

About the user's shell, we deploy influxdb with puppet and configure influxdb user with /sbin/nologin and create a specific home directory before install influxdb package.

@otoolep
Copy link
Contributor

otoolep commented Aug 28, 2015

OK, good enough for me -- thanks.

+1

@toddboom ?

@toddboom
Copy link
Contributor

+1

otoolep added a commit that referenced this pull request Aug 28, 2015
Add shell option to start the daemon on CentOS
@otoolep otoolep merged commit dd0530b into influxdata:master Aug 28, 2015
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