-
Notifications
You must be signed in to change notification settings - Fork 15
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
[GITHUB-35] Give bigger control over server.properties #34
Conversation
ef7a9de
to
d66767c
Compare
@@ -1,24 +1,17 @@ | |||
ANSIBLE_INSTALL_VERSION ?= 2.5.9 | |||
ANSIBLE_INSTALL_VERSION ?= 2.6.7 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we definitely need these changes to the Makefile? It's fine if so but we will need to push these updates to the other roles at some point.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
if the changes are ok, I will send PR to scaffold repo
2f2cc02
to
0c03e78
Compare
It's now possible to specify any server property using `sansible_kafka_server_properties` dictionary. ``` sansible_kafka_server_properties: offsets.topic.replication.factor: 1 ``` This change also REMOVES: * a lot of previous defaults for `server.properties`. * AWS autodiscovery scripts fixes: #23 fixes: #29
0c03e78
to
edd8426
Compare
LGTM 👍 |
It's now possible to specify any server property using
sansible_kafka_server_properties
dictionary.This change also REMOVES:
server.properties
.fixes: #23
fixes: #29