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.

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

Add health check endpoint for the .NET API #1431

Merged
merged 7 commits into from
Aug 8, 2024
Merged

Add health check endpoint for the .NET API #1431

merged 7 commits into from
Aug 8, 2024

Conversation

danjov
Copy link
Contributor

@danjov danjov commented Aug 6, 2024

Working on #1235

@danjov danjov requested a review from flenny August 6, 2024 13:16
@flenny
Copy link
Contributor

flenny commented Aug 8, 2024

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
Copy link
Contributor Author

danjov commented Aug 8, 2024

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.

@danjov danjov marked this pull request as ready for review August 8, 2024 06:53
@danjov danjov enabled auto-merge August 8, 2024 06:53
@danjov danjov merged commit 0d8ccf1 into main Aug 8, 2024
12 checks passed
@danjov danjov deleted the health-checks branch August 8, 2024 07:07
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