Skip to content
This repository has been archived by the owner on Apr 26, 2024. It is now read-only.

Docker healthcheck for workers with no http listener #8780

Closed
JuniorJPDJ opened this issue Nov 19, 2020 · 4 comments · Fixed by #14747
Closed

Docker healthcheck for workers with no http listener #8780

JuniorJPDJ opened this issue Nov 19, 2020 · 4 comments · Fixed by #14747
Labels
A-Workers Problems related to running Synapse in Worker Mode (or replication) T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements.

Comments

@JuniorJPDJ
Copy link

appservice, fed sender and pusher doesn't listen on any ports on which I could do healthchecks for docker :<

@clokep clokep added z-p3 (Deprecated Label) A-Workers Problems related to running Synapse in Worker Mode (or replication) labels Nov 19, 2020
@clokep
Copy link
Contributor

clokep commented Nov 19, 2020

Seems reasonable to have a healthcheck endpoint for each worker. Probably could do it via a health resource type in the config.

@richvdh richvdh changed the title Docker healthcheck for workers Docker healthcheck for workers with no http listener Feb 11, 2021
@JuniorJPDJ

This comment has been minimized.

@erikjohnston erikjohnston added T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements. and removed z-p3 (Deprecated Label) labels Jul 26, 2021
@ShadowJonathan
Copy link
Contributor

This issue is also triggered when the http listener port is non-"default"

See #11472 for a potential fix for this

@dklimpel
Copy link
Contributor

An idea for a workaround.

Did you try enable a metrics endpoint? The metrics entdpoint also have an health-listener.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
A-Workers Problems related to running Synapse in Worker Mode (or replication) T-Enhancement New features, changes in functionality, improvements in performance, or user-facing enhancements.
Projects
None yet
5 participants