Skip to content

Commit

Permalink
feat: working with OTEL collector
Browse files Browse the repository at this point in the history
  • Loading branch information
helio-frota committed Jan 14, 2025
1 parent dcddc11 commit eb7fbc1
Show file tree
Hide file tree
Showing 6 changed files with 43 additions and 4 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/lint-test.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,7 @@ jobs:
- name: Set up Helm
uses: azure/setup-helm@v4.2.0
with:
version: v3.13.0
version: v3.13.3

# Python is required because `ct lint` runs Yamale (https://github.com/23andMe/Yamale) and
# yamllint (https://github.com/adrienverge/yamllint) which require Python
Expand Down
3 changes: 2 additions & 1 deletion .github/workflows/release.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -26,9 +26,10 @@ jobs:
- name: Install Helm
uses: azure/setup-helm@v4.2.0
with:
version: v3.13.0
version: v3.13.3

- run: |
helm repo add open-telemetry https://open-telemetry.github.io/opentelemetry-helm-charts
helm repo add jaeger https://jaegertracing.github.io/helm-charts
- name: Run chart-releaser
Expand Down
7 changes: 5 additions & 2 deletions charts/trustify-infrastructure/Chart.lock
Original file line number Diff line number Diff line change
Expand Up @@ -8,5 +8,8 @@ dependencies:
- name: jaeger
repository: https://jaegertracing.github.io/helm-charts
version: 3.3.3
digest: sha256:1af1b7a450ff013caa7b815c14fe1a34040770a01aa765ff8b15cd41e840ad27
generated: "2024-12-17T10:22:56.132130532+01:00"
- name: opentelemetry-collector
repository: https://open-telemetry.github.io/opentelemetry-helm-charts
version: 0.110.3
digest: sha256:3f19b3ab66618999059dc55e5c21f4ea9f14197b9510db96214d57fd23900ef0
generated: "2025-01-14T08:55:44.404252021-03:00"
4 changes: 4 additions & 0 deletions charts/trustify-infrastructure/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -34,3 +34,7 @@ dependencies:
repository: https://jaegertracing.github.io/helm-charts
version: 3.3.3
condition: jaeger.enabled
- name: opentelemetry-collector
repository: https://open-telemetry.github.io/opentelemetry-helm-charts
version: 0.110.3
condition: opentelemetry-collector.enabled
29 changes: 29 additions & 0 deletions charts/trustify-infrastructure/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -27,3 +27,32 @@ keycloakPostInstall:

oidc:
clients: {}

opentelemetry-collector:
image:
repository: "otel/opentelemetry-collector-contrib"
enabled: true
nameOverride: otelcol
mode: deployment
presets:
kubernetesAttributes:
enabled: true
service:
type: ClusterIP
config:
receivers:
otlp:
protocols:
grpc:
endpoint: "infrastructure-otelcol:4317"
exporters:
otlp:
endpoint: "infrastructure-jaeger-collector:4317"
tls:
insecure: true
service:
pipelines:
traces:
exporters: [otlp]
processors: [batch]
receivers: [otlp]
2 changes: 2 additions & 0 deletions charts/trustify/templates/helpers/_infrastructure.tpl
Original file line number Diff line number Diff line change
Expand Up @@ -44,6 +44,8 @@ Arguments (dict):
value: parentbased_traceidratio
- name: OTEL_TRACES_SAMPLER_ARG
value: "0.1"
- name: OTEL_EXPORTER_OTLP_ENDPOINT
value: "http://infrastructure-otelcol:4317"
{{- end }}

{{- end }}
Expand Down

0 comments on commit eb7fbc1

Please sign in to comment.