Skip to content
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

Jump to bottom

Add health checks to services #111

Closed
TWilkin opened this issue Feb 23, 2022 · 5 comments · Fixed by #303, #305, #310 or #392
Closed

Add health checks to services #111

TWilkin opened this issue Feb 23, 2022 · 5 comments · Fixed by #303, #305, #310 or #392
Assignees
@TWilkin
Labels
api Relating to the API service energenie-controller Relating to the energenie controller service enhancement New feature or request harmony-controller Relating to the harmony controller service kubernetes Relating to the Kubernetes deployment lifx-controller Relating to the LIFX controller service mqtt Relating to the mosquitto service node-controller Relating to the node controller service persistence Relating to the persistence service scheduler Relating to the scheduler service ui Relating to the UI virtual-controller Relating to the virtual controller service voice-assistant Relating to the voice-assistant service zigbee-controller Relating to the ZigBee controller service
Milestone
Kubernetes

Comments

@TWilkin
Copy link
Owner

TWilkin commented Feb 23, 2022 •
edited
Loading

Very occasionally a service will stop working and oftentimes docker swam doesn't restart it.
Investiage HEALTHCHECK in Dockerfile for each service, or setting them in docker-compose.yaml.
@TWilkin TWilkin added enhancement New feature or request energenie-controller Relating to the energenie controller service zigbee-controller Relating to the ZigBee controller service virtual-controller Relating to the virtual controller service ui Relating to the UI voice-assistant Relating to the voice-assistant service config-server Relating to the config-server service scheduler Relating to the scheduler service harmony-controller Relating to the harmony controller service lifx-controller Relating to the LIFX controller service freedns Relating to the freedns service api Relating to the API service energy-monitor Relating to the energy-monitor service persistence Relating to the persistence service certbot Relating to the certbot service device-mapper Relating to the device-mapper service labels Feb 23, 2022
@TWilkin TWilkin added node-controller Relating to the node controller service mqtt Relating to the mosquitto service and removed certbot Relating to the certbot service device-mapper Relating to the device-mapper service labels Mar 13, 2023
@TWilkin TWilkin added this to the Kubernetes milestone Mar 21, 2023
TWilkin added a commit that referenced this issue Mar 23, 2023
@TWilkin
#111 Add simple health endpoint to deep-thought
2dc229f
TWilkin added a commit that referenced this issue Mar 23, 2023
@TWilkin
#111 Add readiness and liveness probes to deep-thought
3ece950
TWilkin added a commit that referenced this issue Mar 23, 2023
@TWilkin
#111 Probes were under wrong element
87d296e
TWilkin added a commit that referenced this issue Mar 23, 2023
@TWilkin
#111 Add probe to UI
ffeaf5a
TWilkin added a commit that referenced this issue Mar 23, 2023
@TWilkin
#111 Add probe to mosquitto
b9a794b
TWilkin added a commit that referenced this issue Mar 23, 2023
@TWilkin
#111 Chart version increases
3937fe0
TWilkin added a commit that referenced this issue Mar 23, 2023
@TWilkin
#111 Add probes to database
56df8ae
TWilkin added a commit that referenced this issue Mar 23, 2023
@TWilkin
#111 Add a connected method to common MqttService
f9966d7
TWilkin added a commit that referenced this issue Mar 23, 2023
@TWilkin
#111 Update health endpoint to check MQTT and db connections
bacad26
TWilkin added a commit that referenced this issue Mar 27, 2023
@TWilkin
#111 Add HealthService to persistence
e8b9282
TWilkin added a commit that referenced this issue Mar 27, 2023
@TWilkin
#111 Add FileService with touch to common
ffe8049
TWilkin added a commit that referenced this issue Mar 27, 2023
@TWilkin
#111 Touch the file each time the health is okay
f5b1a8b
TWilkin added a commit that referenced this issue Mar 27, 2023
@TWilkin
#111 Also get the health status of the database
0bfb295
TWilkin added a commit that referenced this issue Mar 27, 2023
@TWilkin
#111 Missed underscore types dependency
fb181de
TWilkin added a commit that referenced this issue Mar 27, 2023
@TWilkin
#111 Include health check shell script in common dist
e75a437
TWilkin added a commit that referenced this issue Mar 27, 2023
@TWilkin
#111 Add probes to persistence
26d7737
TWilkin added a commit that referenced this issue Mar 27, 2023
@TWilkin
#111 Fixes to health service
c77679a
TWilkin added a commit that referenced this issue Mar 27, 2023
@TWilkin
#111 Add tests for FileService to common library
78ec564
TWilkin added a commit that referenced this issue Mar 27, 2023
@TWilkin
#111 Add tests for HealthService
c0140ba
TWilkin added a commit that referenced this issue Mar 27, 2023
@TWilkin
#111 Also add a test for HealthService.start
bb85402
@TWilkin
Copy link
Owner Author

TWilkin commented Mar 27, 2023

Completed for:

  • energenie-controller
  • harmony-controller
  • lifx-controller
  • macro-controller
  • node-controller
  • zigbee-controller

TWilkin added a commit that referenced this issue Apr 3, 2023
@TWilkin
Merge pull request #310 from TWilkin/294.persistence-indexes
0f637b5 
#111/#294/#297 Add health check and indexes to persistence
@TWilkin TWilkin linked a pull request Apr 3, 2023 that will close this issue
#111/#294/#297 Add health check and indexes to persistence #310
Merged
@TWilkin
Copy link
Owner Author

TWilkin commented Apr 3, 2023

Completed for persistence.

TWilkin added a commit that referenced this issue May 1, 2023
@TWilkin
#298, #111 Add health check probes
24e6013
@TWilkin TWilkin mentioned this issue May 4, 2023
Merged
@TWilkin
Copy link
Owner Author

TWilkin commented May 4, 2023

Completed for scheduler.

@TWilkin TWilkin added the kubernetes Relating to the Kubernetes deployment label May 4, 2023
TWilkin added a commit that referenced this issue Jul 9, 2023
@TWilkin
#111 Add health endpoint to voice-assistant
d5e0b09
TWilkin added a commit that referenced this issue Jul 9, 2023
@TWilkin
#111 Add probe to voice-assistant
772db91
TWilkin added a commit that referenced this issue Jul 9, 2023
@TWilkin
#111 Add test for HealthService
bd809da
TWilkin added a commit that referenced this issue Jul 9, 2023
@TWilkin
#111 Endpoint is just /health
c3cb587
@TWilkin
Copy link
Owner Author

TWilkin commented Jul 9, 2023 •
edited
Loading

Not going to apply for energy-monitor, config-server and freedns as they are scheduled with cron jobs and therefore don't run for long enough to justify the health check.

@TWilkin TWilkin removed config-server Relating to the config-server service freedns Relating to the freedns service energy-monitor Relating to the energy-monitor service labels Jul 9, 2023
@TWilkin TWilkin mentioned this issue Jul 9, 2023
Merged
TWilkin added a commit that referenced this issue Jul 9, 2023
@TWilkin
Merge pull request #392 from TWilkin/111.voice-assistant
202a911 
#111 Add health-check to voice-assistant
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@TWilkin TWilkin

Labels
api Relating to the API service energenie-controller Relating to the energenie controller service enhancement New feature or request harmony-controller Relating to the harmony controller service kubernetes Relating to the Kubernetes deployment lifx-controller Relating to the LIFX controller service mqtt Relating to the mosquitto service node-controller Relating to the node controller service persistence Relating to the persistence service scheduler Relating to the scheduler service ui Relating to the UI virtual-controller Relating to the virtual controller service voice-assistant Relating to the voice-assistant service zigbee-controller Relating to the ZigBee controller service
Projects
None yet
Milestone
Kubernetes
1 participant
@TWilkin