-
Notifications
You must be signed in to change notification settings - Fork 39
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
[config] Update default configuration template #250
Conversation
assets/default-config.yaml
Outdated
discovery: | ||
config: | ||
service: | ||
env: "{{ .Env }}" |
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.
This should stay as default_env
since it's not a templated file
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.
LGTM pending 1 comment
Codecov Report
@@ Coverage Diff @@
## master #250 +/- ##
=======================================
Coverage 76.16% 76.16%
=======================================
Files 30 30
Lines 2240 2240
=======================================
Hits 1706 1706
Misses 393 393
Partials 141 141 Continue to review full report at Codecov.
|
This is a breaking change as the operator will not be able to create a working default configuration for m3db v1.0.0+. To use the new version of the operator with an older version of m3db you will have to provide a custom config map.
Please see m3db changelog for details.