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

docs: add hint for healthcheck.start_interval #12643

Merged
merged 2 commits into from
Sep 16, 2024

Conversation

kastl-ars
Copy link
Contributor

Add a hint to comment out the healthcheck.start_interval setting when running with (rootless) Podman and Docker compose.

(#12638)

@github-actions github-actions bot added the documentation Improvements or additions to documentation label Sep 13, 2024
@mmomjian
Copy link
Contributor

So I think this is actually due to the old version of docker compose, not podman, can we make the tip a bit more generic to the error being an old version?

@kastl-ars
Copy link
Contributor Author

So I think this is actually due to the old version of docker compose, not podman, can we make the tip a bit more generic to the error being an old version?

Sure. I only encountered it with Podman, I did not check with docker.

In my case, I am using docker-compose-2.29.2-2.1.x86_64, which is the latest version.

@kastl-ars kastl-ars force-pushed the 20240913_docs_podman_compose_error branch from b2d1e3c to 4d8175b Compare September 13, 2024 11:46
@kastl-ars
Copy link
Contributor Author

So I think this is actually due to the old version of docker compose, not podman, can we make the tip a bit more generic to the error being an old version?

Done.

@mertalev mertalev changed the title [docs] add hint for Podman and healthcheck.start_interval docs: add hint for healthcheck.start_interval Sep 14, 2024
Copy link
Member

@danieldietzler danieldietzler left a comment

Choose a reason for hiding this comment

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

With Mert's suggestion this looks good to me

@jrasm91 jrasm91 enabled auto-merge (squash) September 16, 2024 20:50
@jrasm91 jrasm91 merged commit de0fd06 into immich-app:main Sep 16, 2024
32 checks passed
Copy link
Contributor

📖 Documentation deployed to pr-12643.preview.immich.app

@kastl-ars kastl-ars deleted the 20240913_docs_podman_compose_error branch September 18, 2024 07:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
changelog:documentation documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants