search

swisstopo / swissgeol-boreholes-suite

Bohrdatenmanagementsystem (BDMS)
MIT License
4 stars 1 forks source link

Add health check endpoint for the .NET API #1431

Closed danjov closed 1 month ago

danjov commented 1 month ago

Working on #1235

flenny commented 1 month ago

Q: Da der Health-Endpunkt (bspw. https://example.com/health) in einem Monitoring verwendet werden könnte, würde es dann Sinn machen den legacy-api/Proxy Aspekt auch zu prüfen? Im Kontext des Monitorings gesehen? Für die Docker/Kubernetes Prüf-Mechanismen wäre ein health-Enpoint im legacy-api Container aber weiterhin sinnvoll (nicht in diesem PR). Oder lassen wir diesen Aspekt gleich weg, da der legacy-api Container demnächst wegfällt?

danjov commented 1 month ago

Q: Da der Health-Endpunkt (bspw. https://example.com/health) in einem Monitoring verwendet werden könnte, würde es dann Sinn machen den legacy-api/Proxy Aspekt auch zu prüfen? Im Kontext des Monitorings gesehen? Für die Docker/Kubernetes Prüf-Mechanismen wäre ein health-Enpoint im legacy-api Container aber weiterhin sinnvoll (nicht in diesem PR). Oder lassen wir diesen Aspekt gleich weg, da der legacy-api Container demnächst wegfällt?

@flenny Im Kontext des Monitorings wäre es sicher sinnvoll und ich würde die Entscheidung gerne nächste/übernächste Woche fällen. Denn wir schätzen gerade den Aufwand für den kompletten Rückbau (siehe #1405) und wissen nächste Woche vermutlich mehr, wann wir das angehen können.