Skip to content

MINOR: Add Docker version requirement and permission warning to README#19994

Merged
chia7712 merged 4 commits into
apache:trunkfrom
LoganZhuZzz:fix-docker-issue
Jun 19, 2025
Merged

MINOR: Add Docker version requirement and permission warning to README#19994
chia7712 merged 4 commits into
apache:trunkfrom
LoganZhuZzz:fix-docker-issue

Conversation

@LoganZhuZzz

@LoganZhuZzz LoganZhuZzz commented Jun 19, 2025

Copy link
Copy Markdown
Contributor

Adds a new section "Installation Preparation" to the Kafka Docker
examples README. It documents a known issue with Docker versions prior
to 20.10.4, where directory permissions (such as /opt/kafka/config)
are not properly set, causing startup failures for non-root users (e.g.,
appuser).

This issue was fixed in Docker 20.10.4
release
,
specifically in Use 0755 permissions when creating missing directories
moby/moby#42017.

Reviewers: Jhen-Yung Hsu [email protected], Lan Ding
[email protected], Chia-Ping Tsai [email protected]

@github-actions github-actions Bot added triage PRs from the community docs docker Official Docker image small Small PRs labels Jun 19, 2025

@chia7712 chia7712 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

@LoganZhuZzz Could you please share the error message with us if Docker is running on older versions?

Comment thread docker/examples/README.md Outdated

Note that the `Docker` version **must be >= 20.10.4**.

The prior Docker versions (e.g., 17.03.x–20.10.3) may cause permission errors when running the Kafka container, as they do not correctly set directory permissions when creating container paths like `/opt/kafka/config`.

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

It should be fine to assume the older versions have such issues, so it is not required to list older versions explicitly

@LoganZhuZzz

Copy link
Copy Markdown
Contributor Author

@LoganZhuZzz Could you please share the error message with us if Docker is running on older versions?

encounter the following error during container startup when using older Docker versions:
image

@Yunyung Yunyung left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for the PR. I left some comments.

Comment thread docker/examples/README.md Outdated
Comment thread docker/examples/README.md Outdated
Comment on lines +23 to +35
If you are using the prior version, you may encounter the following error during container startup:
```text
===> User
uid=1000(appuser) gid=1000(appuser) groups=1000(appuser)
===> Setting default values of environment variables if not already set.
===> Configuring …
Running in KRaft mode…
/opt/kafka/config/ file not writable
```

To avoid this, **please upgrade Docker to 20.10.4 or later**.

This issue was fixed in Docker [20.10.4 release](https://docs.docker.com/engine/release-notes/20.10/#20104), specifically in Use 0755 permissions when creating missing directories [moby/moby#42017](https://github.com/moby/moby/pull/42017).

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I prefer leaving the detailed errors in the PR discussion rather than explicitly writing them in the README.

@DL1231 DL1231 left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Thanks for the PR, LGTM.

@chia7712
chia7712 merged commit 772b7c2 into apache:trunk Jun 19, 2025
27 checks passed
@github-actions github-actions Bot removed the triage PRs from the community label Jun 20, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci-approved docker Official Docker image docs small Small PRs

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants