Skip to content

Add app healthcheck to docker compose example#8

Merged
JavierMonton merged 6 commits intoJavierMonton:mainfrom
a-aznar:main
Apr 15, 2026
Merged

Add app healthcheck to docker compose example#8
JavierMonton merged 6 commits intoJavierMonton:mainfrom
a-aznar:main

Conversation

@a-aznar
Copy link
Copy Markdown
Contributor

@a-aznar a-aznar commented Apr 13, 2026

Add app healthcheck to docker compose example

  • Add wget to Dockerfile to perform the health check

JavierMonton
JavierMonton reviewed Apr 14, 2026
View reviewed changes
Copy link
Copy Markdown
Owner

@JavierMonton JavierMonton left a comment •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good, thanks!

After seeing your PR, I created a new /health endpoint, it should work better and faster than the root as it doesn't need to render the whole site. Do you think you could point the healthcheck to that endpoint?

Comment thread docker-compose.example.yml Outdated
@a-aznar @JavierMonton
Update docker-compose.example.yml
339e04c 
Co-authored-by: Javier Montón <32640454+JavierMonton@users.noreply.github.com>
@JavierMonton JavierMonton merged commit 1789ed6 into JavierMonton:main Apr 15, 2026
1 check passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@JavierMonton JavierMonton JavierMonton left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@a-aznar @JavierMonton