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
[CORE-824] Disallow configuring logs in text format #8112
Conversation
Codecov Report
@@ Coverage Diff @@
## 2.3.x #8112 +/- ##
==========================================
- Coverage 22.28% 22.25% -0.03%
==========================================
Files 415 415
Lines 112171 112171
==========================================
- Hits 24995 24962 -33
- Misses 82900 82922 +22
- Partials 4276 4287 +11
📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more |
@@ -10,6 +10,10 @@ pachctl config set active-context local | |||
pachctl port-forward | |||
{{- end -}} | |||
|
|||
{{- if eq .Values.pachd.logFormat "text" }} | |||
WARNING: setting log format to text is not supported. Defaulting to json | |||
{{- end -}} |
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.
I think I'd warn that it's going away in 2.4 and not remove it in 2.3. Seems odd for a patch release to make a change this large, even if "text" isn't that great.
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.
Ahh okay, can do!
db292f4
to
5198237
Compare
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.
Very nice!
No description provided.