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

Install curl in Docker containers #655

Merged
merged 1 commit into from
Sep 18, 2024
Merged

Conversation

sisou
Copy link
Member

@sisou sisou commented Sep 17, 2024

To do automatic healthchecks with Docker, usually the curl command is used to check a certain HTTP endpoint. A curl command is also what Coolify auto-generates (no way to change that part). The healthchecks on my containers where failing because curl was not available in the container. The NodeJS slim container images do not include the curl command (it's removed in the last step of that image).

Pull request checklist

  • All tests pass. Demo project builds and runs.
  • I have resolved any merge conflicts.

To do automatic healthchecks with Docker, usually the `curl` command is used to check a certain HTTP endpoint. A curl command is also what Coolify auto-generates (no way to change that part). The healthchecks on my containers where failing because curl was not available in the container.
@sisou sisou added the devops label Sep 17, 2024
@sisou sisou self-assigned this Sep 17, 2024
@jeffesquivels jeffesquivels merged commit 4b699e9 into master Sep 18, 2024
20 checks passed
@jeffesquivels jeffesquivels deleted the soeren/docker-curl branch September 18, 2024 05:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants