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

Auth related env variables for customizing cassandra.yaml #112

Closed

Conversation

dfinlay
Copy link

@dfinlay dfinlay commented Aug 17, 2017

Add support for customizing the authenticator, authorizer, role_manager values in cassandra.yaml via environmentals. This does not solve all of the pain points with initial auth setup, but it helps.

May be a start to addressing #106.

…er values in cassandra.yaml via environmentals. This does not solve some of the painpoints with initial auth setup, but it helps.
@tetafro
Copy link

tetafro commented Dec 21, 2017

Please merge this. How can anyone use the image, that gives no way to setup authentication process?

@dfinlay
Copy link
Author

dfinlay commented Dec 21, 2017

@tetafro I think the obvious answer is the image is even easier to use when there isn't any authentication :).

In less pithy news, we've been using my fork in Production since early August. Hundreds of ephemeral environments up and down without issue. It's not the ideal way to do it, so if anyone has a better suggestion, please throw it out there.

I was thinking about some sort of yaml merging facility, overlaying a provided yaml document on top of the defaults.

@tianon
Copy link
Member

tianon commented Oct 3, 2018

I'm sorry, but honestly if we're going to do anything more with environment variables it would likely be in the opposite direction (which we won't do due to the backwards compatibility implications) -- modifying a YAML file via shell script is very fragile, and setting complex configuration in this way is not great. For custom configuration, the only/best recommendation is to provide an appropriate cassandra.yaml file to your running instance (via a new image with FROM / COPY in a short Dockerfile, via Docker Configs, via bind mount, etc).

Closing in favor of #160 + docker-library/docs#1317.

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

Successfully merging this pull request may close these issues.

3 participants