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

Jump to bottom

[WIP] [CASSANDRA-522] add cassandra.ssl_storage_port config var #343

Merged
merged 4 commits into from Dec 15, 2016
Merged

[WIP] [CASSANDRA-522] add cassandra.ssl_storage_port config var #343

merged 4 commits into from Dec 15, 2016

Conversation

mgummelt
Copy link
Contributor

fixes #339

@mgummelt
Copy link
Contributor Author

Waiting for @andlaz to clarify in #339 before merging.

@mgummelt mgummelt changed the title [CASSANDRA-522] add cassandra.ssl_storage_port config var [WIP] [CASSANDRA-522] add cassandra.ssl_storage_port config var Dec 12, 2016
@mgummelt
Copy link
Contributor Author

cc @andlaz

Look good?

@nickbp
Copy link
Contributor

nickbp commented Dec 13, 2016

Failure is:

dcos.errors.DCOSException: Package file [stub-universe-cassandra/repo/packages/C/cassandra/0/config.json] is not JSON: expected } or , got " (line 318, column 12935)

@mgummelt mgummelt force-pushed the CASSANDRA-522-ssl-storage-port branch from a684a0a to 2fd9c32 Compare December 15, 2016 01:05
@mgummelt mgummelt merged commit 6c30e95 into master Dec 15, 2016
@mgummelt mgummelt deleted the CASSANDRA-522-ssl-storage-port branch December 15, 2016 19:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@nickbp nickbp nickbp approved these changes

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

Successfully merging this pull request may close these issues.

SSL storage port hard-coded in marathon json
2 participants
@mgummelt @nickbp