Skip to content

Remove overrides in manifest command; use property files #77

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.

Sign up for GitHub

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
solsson wants to merge 6 commits into from
Closed

Remove overrides in manifest command; use property files #77

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
fba19e0
Default conf from 1.0.0-rc1, except our Server Basics
solsson Oct 15, 2017
ed9fe4e
Continues the transition to property files
solsson Oct 15, 2017
2b446b5
Makes indefinite retention the default
solsson Oct 15, 2017
8a01ab6
localhost for zookeeper isn't a sensible default in k8s
solsson Oct 15, 2017
96045b8
I'm hesitant because the manifest must duplicate this storage path
solsson Oct 15, 2017
cc0334c
We don't reflect the default config anymore, so use git blame
solsson Oct 15, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
We don't reflect the default config anymore, so use git blame
  • Loading branch information
@solsson
solsson committed Oct 15, 2017
commit cc0334c98d7668660f4a7ebdf096ee3a46cff366
1 change: 0 additions & 1 deletion 10broker-config.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -35,7 +35,6 @@ data:
# Switch to enable topic deletion or not, default value is false
delete.topic.enable=true

# Yolean/kubernetes-kafka
auto.create.topics.enable=false

############################# Socket Server Settings #############################
Expand Down