Skip to content

Allow arbitrary --user values#54

Merged
yosifkit merged 2 commits intodocker-library:masterfrom
infosiftr:any-user
Dec 13, 2016
Merged

Allow arbitrary --user values#54
yosifkit merged 2 commits intodocker-library:masterfrom
infosiftr:any-user

Conversation

@tianon
Copy link
Member

@tianon tianon commented Dec 12, 2016

Closes #53 (carried)

This is implemented in the same way we've supported arbitrary values for docker run --user in many other images.

See also docker-library/rabbitmq#60, docker-library/cassandra#48, docker-library/mongo#81, redis/docker-library-redis#48, docker-library/mysql#161, and MariaDB/mariadb-docker#59, docker-library/percona#21.

@tianon
Copy link
Member Author

tianon commented Dec 12, 2016 

$ docker run -it --rm --user "$RANDOM:$RANDOM" --name ghost c0704a28c3d5
npm info it worked if it ends with ok
npm info using [email protected]
npm info using [email protected]
npm info prestart [email protected]
npm info start [email protected]

> [email protected] start /usr/src/ghost
> node index

WARNING: Ghost is attempting to use a direct method to send email. 
It is recommended that you explicitly configure an email service.
Help and documentation can be found at http://support.ghost.org/mail.

Migrations: Creating tables...
Migrations: Creating table: posts
Migrations: Creating table: users
Migrations: Creating table: roles
Migrations: Creating table: roles_users
Migrations: Creating table: permissions
Migrations: Creating table: permissions_users
Migrations: Creating table: permissions_roles
Migrations: Creating table: permissions_apps
Migrations: Creating table: settings
Migrations: Creating table: tags
Migrations: Creating table: posts_tags
Migrations: Creating table: apps
Migrations: Creating table: app_settings
Migrations: Creating table: app_fields
Migrations: Creating table: clients
Migrations: Creating table: client_trusted_domains
Migrations: Creating table: accesstokens
Migrations: Creating table: refreshtokens
Migrations: Creating table: subscribers
Migrations: Running fixture populations
Migrations: Creating owner
Ghost is running in development... 
Listening on 0.0.0.0:2368 
Url configured as: http://localhost:2368 
Ctrl+C to shut down
$ docker top ghost
UID                 PID                 PPID                C                   STIME               TTY                 TIME                CMD
27628               25665               25650               3                   15:10               pts/8               00:00:00            npm
27628               25902               25665               0                   15:10               pts/8               00:00:00            sh -c node index
27628               25903               25902               23                  15:10               pts/8               00:00:04            node index

@tianon
Copy link
Member Author

tianon commented Dec 12, 2016

Ok, this is weird:

$ docker run -it --rm --name ghost c0704a28c3d5
ln: failed to create symbolic link '/usr/src/ghost/config.js': Permission denied

(running it as root, thus getting the gosu behavior rather than simply running as a different user)

Not sure yet why that'd cause any difference at all.

@ghost
Copy link

ghost commented Dec 12, 2016 •
edited by ghost
Loading

May I suggest an update of the documentation to reflect this possibility?
And how do we set the '--user' when using docker-compose?

@yosifkit
Copy link
Member

yosifkit commented Dec 13, 2016

docker-compose would just be user: 1000 or user: 999:50, so it uses the same single string as --user.

@tianon
Copy link
Member Author

tianon commented Dec 13, 2016

Ok, this is updated now. 👍

(Creating the symlink at build-time instead of run-time.)

@yosifkit yosifkit merged commit 46432ab into docker-library:master Dec 13, 2016
@yosifkit yosifkit deleted the any-user branch December 13, 2016 18:59
tianon added a commit to infosiftr/stackbrew that referenced this pull request Dec 13, 2016
@tianon
Update docker-library images
643871a 
- `celery`: 4.0.1
- `docker`: 1.12.4
- `elasticsearch`: add Alpine variants (docker-library/elasticsearch#149)
- `ghost`: allow arbitrary `--user` values (docker-library/ghost#54)
- `haproxy`: 1.7.1
- `java`: debian 9~b148-1
- `mariadb`: add explicit `--socket` (MariaDB/mariadb-docker#92)
- `mysql`: 5.7.17-1debian8, 5.6.35-1debian8, 5.5.54
- `openjdk`: debian 9~b148-1
- `percona`: add explicit `--socket` (docker-library/percona#37)
- `php`: move `CFLAGS` (and friends) to `PHP_CFLAGS`, add `-pie` (docker-library/php#352)
- `postgres`: 9.6.1-2.pgdg80+1, use `pgsql -f ...` instead of `pgsql < ...` for relative includes (docker-library/postgres#234)
- `rocket.chat`: 0.48.1
- `tomcat`: 8.5.9, 9.0.0.M15
- `wordpress`: add `fpm-alpine` variant (docker-library/wordpress#190)
@tianon tianon mentioned this pull request Dec 13, 2016
Merged
This was referenced Jan 19, 2017
Merged
Merged
@tianon tianon mentioned this pull request Aug 29, 2017
Closed
@tianon tianon mentioned this pull request Sep 26, 2017
Closed
@tianon tianon mentioned this pull request Dec 21, 2017
Merged
@tianon tianon mentioned this pull request Jan 4, 2018
Closed
9 tasks
@tianon tianon mentioned this pull request Mar 19, 2018
Merged
This was referenced Nov 1, 2018
Merged
Open
lag-linaro pushed a commit to lag-linaro/fluentd-docker-image that referenced this pull request Nov 16, 2018
Do not manually fix permissions issues by manipulating /etc/passwd
3f6bae6 
If permissions problems exist, they should be overcome properly.

For examples see:

  LINK: docker-library/rabbitmq#60
  LINK: docker-library/cassandra#48
  LINK: docker-library/mongo#81
  LINK: redis/docker-library-redis#48
  LINK: docker-library/mysql#161
  LINK: MariaDB/mariadb-docker#59
  LINK: docker-library/percona#21
  LINK: docker-library/ghost#54
  LINK: docker-library/postgres#253

As suggested by @tianon

  LINK: docker-library/official-images#3724 (comment)

This is part of an effort to make FluentD part of Docker's Official Images.

Signed-off-by: Lee Jones <[email protected]>
lag-linaro pushed a commit to lag-linaro/fluentd-docker-image that referenced this pull request Nov 19, 2018
Do not manually fix permissions issues by manipulating /etc/passwd
450a194 
If permissions problems exist, they should be overcome properly.

For examples see:

  LINK: docker-library/rabbitmq#60
  LINK: docker-library/cassandra#48
  LINK: docker-library/mongo#81
  LINK: redis/docker-library-redis#48
  LINK: docker-library/mysql#161
  LINK: MariaDB/mariadb-docker#59
  LINK: docker-library/percona#21
  LINK: docker-library/ghost#54
  LINK: docker-library/postgres#253

As suggested by @tianon

  LINK: docker-library/official-images#3724 (comment)

This is part of an effort to make FluentD part of Docker's Official Images.

Signed-off-by: Lee Jones <[email protected]>
lag-linaro pushed a commit to lag-linaro/fluentd-docker-image that referenced this pull request Nov 23, 2018
Do not manually fix permissions issues by manipulating /etc/passwd
8859ab1 
If permissions problems exist, they should be overcome properly.

For examples see:

  LINK: docker-library/rabbitmq#60
  LINK: docker-library/cassandra#48
  LINK: docker-library/mongo#81
  LINK: redis/docker-library-redis#48
  LINK: docker-library/mysql#161
  LINK: MariaDB/mariadb-docker#59
  LINK: docker-library/percona#21
  LINK: docker-library/ghost#54
  LINK: docker-library/postgres#253

As suggested by @tianon

  LINK: docker-library/official-images#3724 (comment)

This is part of an effort to make FluentD part of Docker's Official Images.

Signed-off-by: Lee Jones <[email protected]>
lag-linaro pushed a commit to lag-linaro/fluentd-docker-image that referenced this pull request Nov 30, 2018
Do not manually fix permissions issues by manipulating /etc/passwd
e9c1c17 
If permissions problems exist, they should be overcome properly.

For examples see:

  LINK: docker-library/rabbitmq#60
  LINK: docker-library/cassandra#48
  LINK: docker-library/mongo#81
  LINK: redis/docker-library-redis#48
  LINK: docker-library/mysql#161
  LINK: MariaDB/mariadb-docker#59
  LINK: docker-library/percona#21
  LINK: docker-library/ghost#54
  LINK: docker-library/postgres#253

As suggested by @tianon

  LINK: docker-library/official-images#3724 (comment)

This is part of an effort to make FluentD part of Docker's Official Images.

Signed-off-by: Lee Jones <[email protected]>
lag-linaro pushed a commit to lag-linaro/fluentd-docker-image that referenced this pull request Dec 19, 2018
Do not manually fix permissions issues by manipulating /etc/passwd
34ed7d5 
If permissions problems exist, they should be overcome properly.

For examples see:

  LINK: docker-library/rabbitmq#60
  LINK: docker-library/cassandra#48
  LINK: docker-library/mongo#81
  LINK: redis/docker-library-redis#48
  LINK: docker-library/mysql#161
  LINK: MariaDB/mariadb-docker#59
  LINK: docker-library/percona#21
  LINK: docker-library/ghost#54
  LINK: docker-library/postgres#253

As suggested by @tianon

  LINK: docker-library/official-images#3724 (comment)

This is part of an effort to make FluentD part of Docker's Official Images.

Signed-off-by: Lee Jones <[email protected]>
@yosifkit yosifkit mentioned this pull request Feb 12, 2019
Merged
@tianon tianon mentioned this pull request May 17, 2019
Closed
@wglambert wglambert mentioned this pull request Dec 23, 2019
Closed
@wglambert wglambert mentioned this pull request Oct 26, 2020
Closed
@yosifkit yosifkit mentioned this pull request Nov 28, 2022
Closed
@yosifkit yosifkit mentioned this pull request Apr 24, 2025
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@yosifkit yosifkit yosifkit approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tianon @yosifkit @toanhoang