Skip to content

3.11 image CAN modify some port by envs Update docker-entrypoint.sh #214

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

Closed
wants to merge 3 commits into from

Conversation

binbinshi
Copy link

Add storage_port ssl_storage_port native_transport_port authenticator and JMX_PORT can modify in envs ;

for example modify cql port :
$ docker run --name some-cassandra -d -e CASSANDRA_BROADCAST_ADDRESS=10.42.42.42 -e CASSANDRA_NATIVE_TRANSPORT_PORT=19042 -p 7000:7000 cassandra:tag

Add storage_port ssl_storage_port  native_transport_port  authenticator and JMX_PORT can modify in envs ; 

for example  modify cql port  :
$ docker run --name some-cassandra -d -e CASSANDRA_BROADCAST_ADDRESS=10.42.42.42 -e CASSANDRA_NATIVE_TRANSPORT_PORT=19042 -p 7000:7000 cassandra:tag
support arm64 arch of Kunpeng
Supported arm64 arch of Kunpeng
@binbinshi binbinshi closed this Sep 14, 2020
@binbinshi binbinshi reopened this Sep 14, 2020
@tianon
Copy link
Member

tianon commented Sep 14, 2020

Duplicate of #122:

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).

See also docker-library/docs#1317

@binbinshi
Copy link
Author

You are right, but there are often port conflicts when we deploy c* with other services. so I hope some ports can be modified.
I sincerely hope that the community can consider the above

@binbinshi binbinshi closed this Sep 23, 2020
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.

2 participants