Skip to content

Add support for K8s TLS terminated reverse-proxy scenarios #8

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

Open
wants to merge 9 commits into
base: main
Choose a base branch
from
Open

Add support for K8s TLS terminated reverse-proxy scenarios #8

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
9 commits
Select commit Hold shift + click to select a range
b1194a3
Add support for reverse-proxy scenarios
bcarlock-mycarrier Apr 23, 2025
3463a19
refactor(helm): restructure hyperdx configuration for better organiza…
bcarlock-mycarrier Apr 24, 2025
c10caa6
refactor(helm): update hyperdx deployment configuration for improved …
bcarlock-mycarrier Apr 24, 2025
e626463
fix(helm): correct app port variable reference in hyperdx service con…
bcarlock-mycarrier Apr 24, 2025
0610eff
fix(helm): update api-key secret generation to use dynamic value from…
bcarlock-mycarrier Apr 24, 2025
9f0bd61
fix(helm): correct variable references for app and API ports in Confi…
bcarlock-mycarrier Apr 24, 2025
66e3a0a
fix(helm): update OTEL_SERVICE_NAME to use dynamic fullname reference…
bcarlock-mycarrier Apr 24, 2025
d44586f
fix(helm): correct variable reference for hyperdx API key secret gene…
bcarlock-mycarrier Apr 24, 2025
2671d3a
fix(helm): add missing api port configuration in hyperdx service
bcarlock-mycarrier Apr 24, 2025
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
11 changes: 7 additions & 4 deletions charts/hdx-oss-v2/templates/hyperdx-deployment.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ metadata:
{{- end -}}
{{- end }}
spec:
replicas: {{ .Values.hyperdx.replicas | default 1 }}
replicas: 1
Copy link
Contributor

@teeohhem teeohhem Apr 23, 2025

Choose a reason for hiding this comment

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

We probably want to keep the replica setting for flexibility.

Copy link
Contributor Author

@bcarlock-mycarrier bcarlock-mycarrier Apr 24, 2025

Choose a reason for hiding this comment

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

Thanks, that was an accidental regression.

selector:
matchLabels:
{{- include "hdx-oss.selectorLabels" . | nindent 6 }}
Expand All @@ -34,10 +34,13 @@ spec:
command: ['sh', '-c', 'until nc -z {{ include "hdx-oss.fullname" . }}-mongodb {{ .Values.mongodb.port }}; do echo waiting for mongodb; sleep 2; done;']
containers:
- name: app
image: "{{ .Values.hyperdx.image }}"
image: "{{ .Values.images.hdx.repository }}:{{ .Values.images.hdx.tag }}"
securityContext:
capabilities:
add: ["NET_ADMIN", "NET_BIND_SERVICE"]
ports:
- name: app-port
containerPort: {{ .Values.hyperdx.appPort }}
containerPort: {{ .Values.hyperdx.app.port }}
- name: api-port
containerPort: {{ .Values.hyperdx.apiPort }}
envFrom:
Expand All @@ -51,4 +54,4 @@ spec:
key: api-key
{{- with .Values.hyperdx.env }}
{{- toYaml . | nindent 12 }}
{{- end }}
{{- end }}
6 changes: 4 additions & 2 deletions charts/hdx-oss-v2/values.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,8 +6,10 @@ hyperdx:
image: "hyperdx/hyperdx:2-beta"
apiKey: "xxxxxxxx-xxxx-xxxx-xxxx-xxxxxxxxxxxx"
apiPort: 8000
appPort: 3000
appUrl: "http://localhost"
app:
port: 3000
url: "http://localhost"
appendPort: false # Set to true if you want to append the port to the URL in generated links
logLevel: "info"
usageStatsEnabled: true
annotations: {}
Expand Down