Skip to content

Commit 26ad81e

Browse files
authored
fix: Observability stack now works (#398)
* fix(stack/observability): Change tempo port: 3100 -> 3200 Changed in chart 1.21.1: > Breaking Change * In order to be consistent with other projects and documentations, the default port has been changed from 3100 to 3200. * fix(stack/observability): Change jaeger endpoint jaeger-collector -> jaeger It changed somewhere between chart 3.4.1 and 4.5.0 (massive restructure) * chore(stack/observability): Use FQDNs This allows otelcollector configs to be used by pods in any namespace and make it easier to use Secret Operator generated TLS certs so grafana can connect to datasources securely. * fix: This didn't get bumped at release branch time The script doesn't look for that pattern, so it was missed. We should consider using renovate instead of a script with special (though fragile) rules. See: [slack thread](https://stackable-workspace.slack.com/archives/C0AUHQDF1NE/p1776949735762449)
1 parent f0a4345 commit 26ad81e

4 files changed

Lines changed: 12 additions & 10 deletions

File tree

stacks/observability/grafana.yaml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -22,14 +22,14 @@ options:
2222
datasources:
2323
- name: Loki
2424
type: loki
25-
url: http://loki:3100
25+
url: http://loki.{{ NAMESPACE }}.svc.cluster.local:3100
2626
access: proxy
2727
isDefault: false
2828
jsonData:
2929
tlsAuthWithCACert: false
3030
- name: Tempo
3131
type: tempo
32-
url: http://tempo:3100
32+
url: http://tempo.{{ NAMESPACE }}.svc.cluster.local:3200
3333
access: proxy
3434
isDefault: false
3535
jsonData:

stacks/observability/opentelemetry-collector-deployment.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -38,15 +38,15 @@ spec:
3838
debug: {}
3939
# verbosity: detailed
4040
otlp/jaeger:
41-
endpoint: jaeger-collector:4317
41+
endpoint: jaeger.{{ NAMESPACE }}.svc.cluster.local:4317
4242
tls:
4343
insecure: true
4444
otlp/tempo:
45-
endpoint: tempo:4317
45+
endpoint: tempo.{{ NAMESPACE }}.svc.cluster.local:4317
4646
tls:
4747
insecure: true
4848
otlphttp/loki:
49-
endpoint: http://loki:3100/otlp
49+
endpoint: http://loki.{{ NAMESPACE }}.svc.cluster.local:3100/otlp
5050
tls:
5151
insecure: true
5252
# auth:

stacks/observability/opentelemetry-collector-sidecar.yaml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -36,15 +36,15 @@ spec:
3636
debug: {}
3737
# verbosity: detailed
3838
otlp/jaeger:
39-
endpoint: jaeger-collector:4317
39+
endpoint: jaeger.{{ NAMESPACE }}.svc.cluster.local:4317
4040
tls:
4141
insecure: true
4242
otlp/tempo:
43-
endpoint: tempo:4317
43+
endpoint: tempo.{{ NAMESPACE }}.svc.cluster.local:4317
4444
tls:
4545
insecure: true
4646
otlphttp/loki:
47-
endpoint: http://loki:3100/otlp
47+
endpoint: http://loki.{{ NAMESPACE }}.svc.cluster.local:3100/otlp
4848
tls:
4949
insecure: true
5050
# auth:

stacks/stacks-v2.yaml

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -104,7 +104,8 @@ stacks:
104104
default: 3.4.0
105105
- name: stackableReleaseVersion
106106
description: The Stackable release to be used for the OpenSearch Dashboards image tag
107-
default: 0.0.0-dev
107+
# TODO (@NickLarsenNZ): Consider using renovate to bump the SDP versions at release branch time.
108+
default: 26.3.0
108109
observability:
109110
description: >-
110111
An observability stack with auto-injection of the opentelemetry-collector sidecar to receive traces/logs/metrics via OTLP, and send them to Jaeger/Tempo/Loki.
@@ -688,7 +689,8 @@ stacks:
688689
default: 3.4.0
689690
- name: stackableReleaseVersion
690691
description: The Stackable release to be used for the OpenSearch Dashboards image tag
691-
default: 0.0.0-dev
692+
# TODO (@NickLarsenNZ): Consider using renovate to bump the SDP versions at release branch time.
693+
default: 26.3.0
692694
- name: jupyterLabToken
693695
description: Token for JupyterLab UI access
694696
default: adminadmin

0 commit comments

Comments
 (0)