Skip to content

Commit

Permalink
feat: Add support for modifying edge probe values (#97)
Browse files Browse the repository at this point in the history
* feat: Add support for modifying edge probe values

This allows BASE_PATH enviornment variable changes to be reflected
in the values.yaml file. Without this, k8s will attempt to hit
/internal-backstage/health, while the helm-edge application is
reporting health values at /BASE_PATH/internal-backstage/health.

* Bump chart minor version

* Set up defaults for unleash-edge nodes
  • Loading branch information
I3uckwheat authored Oct 20, 2023
1 parent 044d794 commit 2c2bf04
Show file tree
Hide file tree
Showing 3 changed files with 26 additions and 3 deletions.
2 changes: 1 addition & 1 deletion charts/unleash-edge/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -4,7 +4,7 @@ name: unleash-edge
description: A Helm chart for deploying Unleash Edge to kubernetes
icon: https://docs.getunleash.io/img/logo.svg
type: application
version: 2.0.2
version: 2.0.3

appVersion: "v11.0.2"
maintainers:
Expand Down
14 changes: 12 additions & 2 deletions charts/unleash-edge/templates/deployment.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -52,14 +52,24 @@ spec:
- name: http
containerPort: 3063
protocol: TCP
{{- if .Values.livenessProbe.enabled | default true }}
livenessProbe:
httpGet:
path: /internal-backstage/health
path: {{ .Values.livenessProbe.path | default "/internal-backstage/health" }}
port: http
initialDelaySeconds: {{ .Values.livenessProbe.initialDelaySeconds | default 30 }}
timeoutSeconds: {{ .Values.livenessProbe.timeoutSeconds | default 10 }}
{{- end }}
{{- if .Values.readinessProbe.enabled | default true}}
readinessProbe:
httpGet:
path: /internal-backstage/health
path: {{ .Values.readinessProbe.path | default "/internal-backstage/health" }}
port: http
initialDelaySeconds: {{ .Values.readinessProbe.initialDelaySeconds | default 30 }}
timeoutSeconds: {{.Values.readinessProbe.timeoutSeconds | default 10 }}
successThreshold: {{.Values.readinessProbe.successThreshold | default 5 }}
periodSeconds: {{.Values.readinessProbe.periodSeconds | default 10 }}
{{- end }}
resources:
{{- toYaml .Values.resources | nindent 12 }}
{{- with .Values.nodeSelector }}
Expand Down
13 changes: 13 additions & 0 deletions charts/unleash-edge/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -64,6 +64,19 @@ ingress:
# hosts:
#

livenessProbe: {}
# enabled: true
# path: /internal-backstage/health
# initialDelaySeconds: 30
# timeoutSeconds: 10

readinessProbe: {}
# enabled: true
# path: /internal-backstage/health
# initialDelaySeconds: 30
# timeoutSeconds: 10
# periodSeconds: 10
# successThreshold: 5

autoscaling:
enabled: false
Expand Down

0 comments on commit 2c2bf04

Please sign in to comment.