starred/karakeep

Fork 0

mirror of https://github.com/karakeep-app/karakeep.git synced 2026-04-25 16:06:04 +03:00

[GH-ISSUE #326] [FR] Adding Healthchecks do the dockerfiles #211

New issue

Closed

opened 2026-03-02 11:47:39 +03:00 by kerem · 12 comments

kerem commented

2026-03-02 11:47:39 +03:00

Owner

Originally created by @wuast94 on GitHub (Jul 26, 2024).
Original GitHub issue: https://github.com/karakeep-app/karakeep/issues/326

would be nice if healthchecks would be added to the containers 😊 im running a swam cluster and containers without a healtheck can make problems

Originally created by @wuast94 on GitHub (Jul 26, 2024). Original GitHub issue: https://github.com/karakeep-app/karakeep/issues/326 would be nice if healthchecks would be added to the containers 😊 im running a swam cluster and containers without a healtheck can make problems

kerem

2026-03-02 11:47:39 +03:00

closed this issue
added the
feature request
label

kerem commented

2026-03-02 11:47:40 +03:00

Author

Owner

@MohamedBassem commented on GitHub (Jul 27, 2024):

@wuast94 Makes sense. You can define the healthchecks manually from the compose file as well btw.

@MohamedBassem commented on GitHub (Jul 27, 2024): @wuast94 Makes sense. You can define the healthchecks manually from the compose file as well btw.

kerem commented

2026-03-02 11:47:40 +03:00

Author

Owner

@wuast94 commented on GitHub (Jul 27, 2024):

yeah i know, the thing is on the worker part i dont even know how to define the healthcheck.

on the web part it should be an extra endpoint that just return a json or something like that if the web part and all sub parts are working as intended (redis db and so on), its more efficient than just checking a rendered html page.

@wuast94 commented on GitHub (Jul 27, 2024): yeah i know, the thing is on the worker part i dont even know how to define the healthcheck. on the web part it should be an extra endpoint that just return a json or something like that if the web part and all sub parts are working as intended (redis db and so on), its more efficient than just checking a rendered html page.

kerem commented

2026-03-02 11:47:40 +03:00

Author

Owner

@MohamedBassem commented on GitHub (Jul 27, 2024):

@wuast94 makes sense. The good news is that the worker container is going away in the next release. For the web container, I can add a dummy ping endpoint.

@MohamedBassem commented on GitHub (Jul 27, 2024): @wuast94 makes sense. The good news is that the worker container is going away in the next release. For the web container, I can add a dummy ping endpoint.

kerem commented

2026-03-02 11:47:40 +03:00

Author

Owner

@wuast94 commented on GitHub (Jul 27, 2024):

well than it can be a thing i can do maybe. im in the beginning of development (atm im building a web app with svelte), but an healtheck endpoint seams like a nice thing to do for learning 😊

@wuast94 commented on GitHub (Jul 27, 2024): well than it can be a thing i can do maybe. im in the beginning of development (atm im building a web app with svelte), but an healtheck endpoint seams like a nice thing to do for learning 😊

kerem commented

2026-03-02 11:47:40 +03:00

Author

Owner

@MohamedBassem commented on GitHub (Jul 27, 2024):

@wuast94 Sure! Hoarder uses Next.js. You can drop a ping/route.ts here (https://github.com/hoarder-app/hoarder/tree/main/apps/web/app/api) and implement a dummy endpoint (using this docs: https://nextjs.org/docs/app/api-reference/file-conventions/route) to return a plain text containing "PONG" for example. Then you should be able to call it with http://localhost:3000/api/ping

@MohamedBassem commented on GitHub (Jul 27, 2024): @wuast94 Sure! Hoarder uses Next.js. You can drop a `ping/route.ts` here (https://github.com/hoarder-app/hoarder/tree/main/apps/web/app/api) and implement a dummy endpoint (using this docs: https://nextjs.org/docs/app/api-reference/file-conventions/route) to return a plain text containing "PONG" for example. Then you should be able to call it with `http://localhost:3000/api/ping`

kerem commented

2026-03-02 11:47:40 +03:00

Author

Owner

@wuast94 commented on GitHub (Jul 27, 2024):

yeah sounds like a nice starting point. if i can i will also return if the redis and db connection are ok 😊

@wuast94 commented on GitHub (Jul 27, 2024): yeah sounds like a nice starting point. if i can i will also return if the redis and db connection are ok 😊

kerem commented

2026-03-02 11:47:41 +03:00

Author

Owner

@MohamedBassem commented on GitHub (Jul 27, 2024):

Redis is going away, and the db is sqlite db, so it's usually always ok :)

@MohamedBassem commented on GitHub (Jul 27, 2024): Redis is going away, and the db is sqlite db, so it's usually always ok :)

kerem commented

2026-03-02 11:47:41 +03:00

Author

Owner

@wuast94 commented on GitHub (Jul 27, 2024):

my bad i meaned meilisearch

@wuast94 commented on GitHub (Jul 27, 2024): my bad i meaned meilisearch

kerem commented

2026-03-02 11:47:41 +03:00

Author

Owner

@MohamedBassem commented on GitHub (Sep 22, 2024):

This is now done with your PR, thanks!

@MohamedBassem commented on GitHub (Sep 22, 2024): This is now done with your PR, thanks!

kerem commented

2026-03-02 11:47:41 +03:00

Author

Owner

@huyz commented on GitHub (Oct 7, 2024):

Should the Dockerfile and/or docker-compose.yml files be updated to include the healthcheck?

@huyz commented on GitHub (Oct 7, 2024): Should the Dockerfile and/or docker-compose.yml files be updated to include the healthcheck?

kerem commented

2026-03-02 11:47:41 +03:00

Author

Owner

@wuast94 commented on GitHub (Oct 7, 2024):

i will do a PR

@wuast94 commented on GitHub (Oct 7, 2024): i will do a PR

kerem commented

2026-03-02 11:47:41 +03:00

Author

Owner

@wuast94 commented on GitHub (Oct 7, 2024):

@huyz #508 will add this

@wuast94 commented on GitHub (Oct 7, 2024): @huyz #508 will add this

kerem referenced this issue

2026-03-02 11:58:10 +03:00

[PR #211] [MERGED] Cli improvement #1555

No milestone

No project

No assignees

1 participant

Notifications

Due date

The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference

starred/karakeep#211

No description provided.

Rows
Columns