Skip to content

Commit

Permalink
Fixed director readiness probe failing
Browse files Browse the repository at this point in the history
Readiness probe was failing when using BASE_PATH  env
  • Loading branch information
shkuviak committed Sep 27, 2023
1 parent 3b261d3 commit fd54494
Show file tree
Hide file tree
Showing 4 changed files with 10 additions and 2 deletions.
2 changes: 1 addition & 1 deletion charts/sorry-cypress/Chart.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,7 @@ apiVersion: v2
name: sorry-cypress
description: A Helm chart for Sorry Cypress
type: application
version: 1.18.0
version: 1.18.1
appVersion: 2.5.9
home: https://sorry-cypress.dev/
sources:
Expand Down
3 changes: 3 additions & 0 deletions charts/sorry-cypress/changelog.md
Original file line number Diff line number Diff line change
@@ -1,3 +1,6 @@
# 1.18.1
- Fixed director readiness probe failing when using BASE_PATH env

# 1.18
- Update app to version `2.5.9`

Expand Down
4 changes: 3 additions & 1 deletion charts/sorry-cypress/templates/deployment-director.yml
Original file line number Diff line number Diff line change
Expand Up @@ -161,6 +161,8 @@ spec:
value: {{ .Values.director.environmentVariables.inactivityTimeoutSeconds | quote }}
- name: GITLAB_JOB_RETRIES
value: {{ .Values.director.environmentVariables.gitlabJobRetries | quote }}
- name: BASE_PATH
value: {{ .Values.director.environmentVariables.basePath | quote }}
{{- with .Values.director.extraEnv }}
{{- toYaml . | nindent 8 }}
{{- end }}
Expand All @@ -175,7 +177,7 @@ spec:
{{- toYaml .Values.director.securityContext | nindent 10 }}
readinessProbe:
httpGet:
path: /health-check-db
path: {{ .Values.director.environmentVariables.basePath }}/health-check-db
port: 1234
periodSeconds: {{ .Values.director.readinessProbe.periodSeconds }}
timeoutSeconds: {{ .Values.director.readinessProbe.timeoutSeconds }}
Expand Down
3 changes: 3 additions & 0 deletions charts/sorry-cypress/values.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -300,6 +300,9 @@ director:
# Enable gitlab job retries
gitlabJobRetries: false

# BASE_PATH serving path, usefull when using reverse proxy ex: /cypress/dashboard
basePath: ""

# Set annotations for pods
podAnnotations: {}

Expand Down

0 comments on commit fd54494

Please sign in to comment.