Skip to content

HAProxy startupProbe and named ports #437

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.

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

Merged
merged 2 commits into from
May 12, 2025
Merged

HAProxy startupProbe and named ports #437

merged 2 commits into from
May 12, 2025

Conversation

benbz
Copy link
Member

@benbz benbz commented May 12, 2025

We always want a startupProbe for HAProxy. We also always want to use named ports (so that changing the port number doesn't cause breakages in the probes.

The periodSeconds and failureThreshold are going to be extreme with no Synapse but this is a problem after #430 that we push onto users.

Doing as a separate PR to minimise dyff & increase reviewability

@benbz benbz requested a review from a team as a code owner May 12, 2025 14:28
Copy link

dyff of changes in rendered templates of CI manifests

example-default-enabled-components-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
matrix-authentication-service-keep-auth-in-synapse-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
matrix-authentication-service-synapse-secrets-externally-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
matrix-authentication-service-synapse-secrets-in-helm-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
pytest-matrix-authentication-service-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
pytest-matrix-rtc-synapse-wellknown-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
pytest-synapse-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
pytest-well-known-values.yaml (changed)
@@ spec.template.spec.containers.haproxy @@
# apps/v1/Deployment/default/release-name-haproxy
! + one map entry added:
+   startupProbe:
+     httpGet:
+       path: /haproxy_test
+       port: haproxy-metrics
+     periodSeconds: 2
+     failureThreshold: 150

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
quick-setup-certificates-pg-external-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
quick-setup-certificates-pg-with-helm-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
quick-setup-external-cert-pg-external-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
quick-setup-external-cert-pg-with-helm-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
quick-setup-letsencrypt-pg-external-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
quick-setup-letsencrypt-pg-with-helm-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
quick-setup-wildcard-cert-pg-external-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
quick-setup-wildcard-cert-pg-with-helm-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
synapse-checkov-with-workers-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
synapse-ingress-additional-paths-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
synapse-minimal-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
synapse-postgres-secrets-externally-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
synapse-postgres-secrets-in-helm-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
synapse-secrets-externally-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
synapse-secrets-in-helm-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
synapse-worker-example-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
well-known-checkov-values.yaml (changed)
@@ spec.template.spec.containers.haproxy @@
# apps/v1/Deployment/default/release-name-haproxy
! + one map entry added:
+   startupProbe:
+     httpGet:
+       path: /haproxy_test
+       port: haproxy-metrics
+     periodSeconds: 2
+     failureThreshold: 150

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
well-known-element-web-values.yaml (changed)
@@ spec.template.spec.containers.haproxy @@
# apps/v1/Deployment/default/release-name-haproxy
! + one map entry added:
+   startupProbe:
+     httpGet:
+       path: /haproxy_test
+       port: haproxy-metrics
+     periodSeconds: 2
+     failureThreshold: 150

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
well-known-mas-values.yaml (changed)
@@ spec.template.spec.containers.haproxy @@
# apps/v1/Deployment/default/release-name-haproxy
! + one map entry added:
+   startupProbe:
+     httpGet:
+       path: /haproxy_test
+       port: haproxy-metrics
+     periodSeconds: 2
+     failureThreshold: 150

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
well-known-minimal-values.yaml (changed)
@@ spec.template.spec.containers.haproxy @@
# apps/v1/Deployment/default/release-name-haproxy
! + one map entry added:
+   startupProbe:
+     httpGet:
+       path: /haproxy_test
+       port: haproxy-metrics
+     periodSeconds: 2
+     failureThreshold: 150

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
well-known-synapse-mas-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics
well-known-synapse-values.yaml (changed)
@@ spec.template.spec.containers.haproxy.startupProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8406
+ synapse-ready

@@ spec.template.spec.containers.haproxy.livenessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@@ spec.template.spec.containers.haproxy.readinessProbe.httpGet.port @@
# apps/v1/Deployment/default/release-name-haproxy
! ± type change from int to string
- 8405
+ haproxy-metrics

@benbz benbz merged commit 0f75841 into main May 12, 2025
49 checks passed
@benbz benbz deleted the bbz/haproxy-startupProbe branch May 12, 2025 14:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants