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

[improve][misc] Specify valid home dir for the default user in the Ubuntu based docker image #22446

Merged
merged 1 commit into from
Apr 5, 2024

Conversation

lhotari
Copy link
Member

@lhotari lhotari commented Apr 5, 2024

Motivation

A default user was added to the docker image #21695. The problem is that the home directory for the user is invalid.
This causes issues such as #22440 .

The home directory for the pulsar user is /home/pulsar, which is invalid.

❯ docker run --rm -it apachepulsar/pulsar:3.0.4 bash -c "cat /etc/passwd|grep pulsar"
pulsar:x:10000:0::/home/pulsar:/bin/sh

Modifications

Make the home directory /pulsar/data for the pulsar user.

Documentation

  • doc
  • doc-required
  • doc-not-needed
  • doc-complete

@lhotari lhotari self-assigned this Apr 5, 2024
@github-actions github-actions bot added the doc-not-needed Your PR changes do not impact docs label Apr 5, 2024
@lhotari lhotari changed the title [improve][misc] Specify valid home dir for the default user in docker image [improve][misc] Specify valid home dir for the default user in the Ubuntu based docker image Apr 5, 2024
Copy link
Member

@dao-jun dao-jun left a comment

Choose a reason for hiding this comment

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

lgtm

@lhotari lhotari merged commit e008580 into apache:branch-3.0 Apr 5, 2024
42 of 49 checks passed
lhotari added a commit that referenced this pull request Apr 5, 2024
…untu based docker image (#22446)

(cherry picked from commit e008580)
lhotari added a commit that referenced this pull request Apr 5, 2024
…untu based docker image (#22446)

(cherry picked from commit e008580)
nikhil-ctds pushed a commit to datastax/pulsar that referenced this pull request Apr 5, 2024
…untu based docker image (apache#22446)

(cherry picked from commit e008580)
(cherry picked from commit c9b656c)
srinath-ctds pushed a commit to datastax/pulsar that referenced this pull request Apr 8, 2024
…untu based docker image (apache#22446)

(cherry picked from commit e008580)
(cherry picked from commit c9b656c)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants