Skip to content
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

opentelemetry documentation #9144

Closed
wants to merge 10 commits into from
Closed

opentelemetry documentation #9144

Changes from 1 commit
Commits
Show all changes
10 commits
Select commit Hold shift + click to select a range
ea3f215
opentelemetry doc
esigo Oct 11, 2022
ac09fcc
zipkin
esigo Oct 16, 2022
d9df16c
zipkin
esigo Oct 16, 2022
f6d0492
Merge remote-tracking branch 'upstream/main' into otel-9016-3
esigo Jan 15, 2023
4e71867
use opentelemetry value
esigo Jan 15, 2023
f3e609a
Merge remote-tracking branch 'upstream/main' into otel-9016-3
esigo Feb 5, 2023
dfb26e9
revert merge conflict
esigo Feb 5, 2023
bfdff0b
fix
esigo Feb 5, 2023
a1d9aab
format
esigo Feb 5, 2023
6b8f0c6
review comments
esigo Mar 21, 2023
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
Next Next commit
use opentelemetry value
  • Loading branch information
@esigo
esigo committed Jan 15, 2023
commit 4e7186701f1035a9255559d71a90f6b4d7afdffb
12 changes: 7 additions & 5 deletions docs/user-guide/third-party-addons/opentelemetry.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -135,12 +135,14 @@ graph TB

To install the example and collectors run:

1. Enable Ingress addon with extra module:
1. Enable Ingress addon with:

```yaml
extraModules:
- name: opentelemetry
image: registry.k8s.io/ingress-nginx/opentelemetry:v20220906-g981ce38a7@sha256:aa079daa7efd93aa830e26483a49a6343354518360929494bad1d0ad3303142e
opentelemetry:
enabled: true
image: registry.k8s.io/ingress-nginx/opentelemetry:v20230107-helm-chart-4.4.2-2-g96b3d2165@sha256:331b9bebd6acfcd2d3048abbdd86555f5be76b7e3d0b5af4300b04235c6056c9
containerSecurityContext:
allowPrivilegeEscalation: false
```

2. Enable OpenTelemetry and set the otlp-collector-host:
Expand All @@ -151,7 +153,7 @@ To install the example and collectors run:
kind: ConfigMap
data:
enable-opentelemetry: "true"
opentelemetry-config: "/etc/nginx/opentelemtry.toml"
opentelemetry-config: "/etc/nginx/opentelemetry.toml"
opentelemetry-operation-name: "HTTP $request_method $service_name $uri"
OpentelemetryTrustIncomingSpan: "true"
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should OpentelemetryTrustIncomingSpan be updated to opentelemetry-trust-incoming-span?

otlp-collector-host: "otel-coll-collector.otel.svc"
Expand Down