Skip to content

Commit

Permalink
Pin the opentelemetry chart (#1490)
Browse files Browse the repository at this point in the history
Signed-off-by: Matt Moore <[email protected]>
  • Loading branch information
mattmoor authored Sep 29, 2023
1 parent 8470c06 commit cf77166
Show file tree
Hide file tree
Showing 2 changed files with 15 additions and 17 deletions.
28 changes: 11 additions & 17 deletions images/opentelemetry-collector-contrib/tests/custom-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -51,11 +51,6 @@ exporters:
endpoint: "http://zipkin-all-in-one:9411/api/v2/spans"
format: proto

jaeger:
endpoint: jaeger-all-in-one:14250
tls:
insecure: true

# datadog:
# api:
# site: us5.datadoghq.com
Expand All @@ -68,7 +63,7 @@ processors:
timeout: 10s
resourcedetection:
# These warn when not applicable, so just include everything
detectors:
detectors:
- env
- eks
- ec2
Expand All @@ -89,46 +84,45 @@ extensions:
size_in_percentage: 5

service:
extensions:
extensions:
- pprof
- zpages
- health_check
- memory_ballast
pipelines:
metrics:
receivers:
receivers:
- otlp
- hostmetrics
processors:
processors:
- resourcedetection
- k8sattributes
- batch
exporters:
exporters:
- logging
- prometheus
# - datadog

traces:
receivers:
receivers:
- otlp
processors:
processors:
- resourcedetection
- k8sattributes
- batch
exporters:
exporters:
- logging
- zipkin # fails (with warning) without zipkin deployed
- jaeger # fails (with warning) without jaeger deployed
# - datadog

logs:
receivers:
receivers:
- filelog
processors:
processors:
- resourcedetection
- k8sattributes
- batch
exporters:
exporters:
- logging
# - datadog

4 changes: 4 additions & 0 deletions images/opentelemetry-collector-contrib/tests/main.tf
Original file line number Diff line number Diff line change
Expand Up @@ -20,6 +20,10 @@ resource "helm_release" "open-telemetry-deploy" {
namespace = "otelc-deploy"
create_namespace = true

// After https://github.com/open-telemetry/opentelemetry-helm-charts/pull/892 we
// fail to install because of unrecognized exporters.
version = "0.69.0"

values = [jsonencode({
mode = "deployment"
image = {
Expand Down

0 comments on commit cf77166

Please sign in to comment.