Set Cassandra listen_address and rpc_address #1383
Conversation
marko-asplund
force-pushed
the
cass-inst-fixes
branch
from
845f998 to
f860e5b
Compare
|
@msmith-techempower @hamiltont This is a bug fix only PR that should take care of the Cassandra installation problems. Please review. |
toolset/setup/linux/installer.py
Outdated
There was a problem hiding this comment.
I'd vote to use export e.g. l.append("export %s=%s" % (k,v)) - there's not much security risk and it ensures that even if there's a future change this variable will always be available
|
This LGTM. Mike mentioned in IRC that he's hoping to not take any more PRs, so I'm not sure if it's going to go in before R10 |
marko-asplund
force-pushed
the
cass-inst-fixes
branch
from
f860e5b to
08a535a
Compare
|
@hamiltont I updated this PR to address your review comments. @msmith-techempower I re-checked this PR after the change in a Vagrant development environment and it seems to work as before. Please consider merging this fix for R10. |
|
We need this to work before R10 as cassandra is still giving us trouble. |
Set Cassandra listen_address and rpc_address
3 participants
Set Cassandra listen_address and rpc_address param values during installation #1382