-
Notifications
You must be signed in to change notification settings - Fork 95
docs: add logging section for control/data planes & DNS #2198
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
base: master
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for kuma ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
bartsmykla
left a comment
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.
tabs needs to be adjusted here after: #2216
f620d8a to
883d9cf
Compare
* feat(tf): render tf tab Signed-off-by: slonka <[email protected]> * feat(tf): arrays sort of work Signed-off-by: slonka <[email protected]> * feat(tf): commas in arrays work Signed-off-by: slonka <[email protected]> * feat(tf): seems to render quite correctly now Signed-off-by: slonka <[email protected]> * feat(tf): top part looks good Signed-off-by: slonka <[email protected]> * feat(tf): render from uni style Signed-off-by: slonka <[email protected]> * feat(tf): render only when not kuma Signed-off-by: slonka <[email protected]> * feat(tf): update golden files Signed-off-by: slonka <[email protected]> * feat(tf): make all keys snake_case Signed-off-by: slonka <[email protected]> * feat(tf): use snake_case fn everywhere Signed-off-by: slonka <[email protected]> * feat(tf): no need to if else on tf tab Signed-off-by: slonka <[email protected]> * feat(tf): add a guard for 2.10.x onwards Signed-off-by: slonka <[email protected]> --------- Signed-off-by: slonka <[email protected]> Signed-off-by: dascole <[email protected]>
This commit adds a new section to the troubleshooting guide for logging. It includes information on how to enable debug logging for the control and data planes, as well as how to enable debug logging for DNS. This partially addresses issue kumahq#1302 by providing initial documentation for logging. Further work is needed to complete the full scope of the issue. Signed-off-by: dascole <[email protected]>
Resolve lint errors in troubleshooting.md Signed-off-by: dascole <[email protected]>
Address the linting issue from 'Adjusting Logging Levels for Kuma-DP' Signed-off-by: dascole <[email protected]>
Address 'Adjusting Logging Levels for Kuma-DP' Signed-off-by: dascole <[email protected]>
Signed-off-by: dascole <[email protected]>
76c24ec to
4b1d4cc
Compare
Signed-off-by: dascole <[email protected]>
|
Tabs have been adjusted to align with #2216 |
Signed-off-by: Lukasz Dziedziak <[email protected]>
Signed-off-by: dascole <[email protected]>
Signed-off-by: dascole <[email protected]>
| metadata: | ||
| annotations: | ||
| kuma.io/envoy-log-level: debug | ||
| kuma.io/envoy-component-log-level: dns:debug |
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 line is not for "all components"
| kuma.io/envoy-component-log-level: dns:debug |
Co-authored-by: Jay Jijie Chen <[email protected]> Signed-off-by: dascole <[email protected]>
| {% endtab %} | ||
| {% tab Universal %} | ||
|
|
||
| In Universal mode, logging can be enabled by setting the environment variable `KUMA_RUNTIME_KUBERNETES_INJECTOR_BUILTIN_DNS_LOGGING`. |
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.
Universal DPs use KUMA_DNS_ENABLE_LOGGING, insead of KUMA_RUNTIME_KUBERNETES_INJECTOR_BUILTIN_DNS_LOGGING
jijiechen
left a comment
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.
Thanks for the contribution and I think we do need docs helping people diagnostic the mesh.
|
|
||
| use_meshservice = @params["use_meshservice"] == "true" && Gem::Version.new(release.value.dup.sub "x", "0") >= TARGET_VERSION | ||
|
|
||
| show_tf = Gem::Version.new(release.value.dup.sub "x", "0") >= TARGET_VERSION |
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 change is not needed, right?
| ```yaml | ||
| containers: | ||
| - name: control-plane | ||
| image: "docker.io/kong/kuma-cp:2.9.3" |
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.
the image field is not correct for kuma. We may just remove it.
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.
The same suggestion applys to other code snippets underneath.
| group: true | ||
| items: | ||
| - text: Logging | ||
| url: /guides/logging |
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 it's better to put it under "Using Kuma". WDYT @bartsmykla ?

This commit adds a new section to the troubleshooting guide for logging. It includes information on how to enable debug logging for the control and data planes, as well as how to enable debug logging for DNS.
This partially addresses issue #1302 by providing initial documentation for logging. Further work is needed to complete the full scope of the issue.
Did you sign your commit? Instructions
Have you read Contributing guidelines?