Skip to content

Conversation

@stanislaw
Copy link
Collaborator

This done to implement the following constraints:

  1. The Docker container runs a non-root user because otherwise
    Chrome/ChromeDriver are not happy as per this issue:
    [🐛 Bug]: Selenium SessionNotCreatedException: "User data directory is already in use" SeleniumHQ/selenium#15327

  2. StrictDoc runs within a container and outputs artifacts that have the
    UID/GID of the host user.

The approaches that I could not make work:

  1. Running with a default root user because of the permissions issue and
    Chrome raising an error about the environment.

  2. Using USER inside the container and then running an image with --user $(id -u):$(id -g). With this approach, the Chrome would not be happy
    because the container will be running from a user that does not have
    a proper home directory.

Maybe there are better solutions but the one with the entrypoint.sh seems to be good enough.

…er user

This done to implement the following constraints:

1) The Docker container runs a non-root user because otherwise
   Chrome/ChromeDriver are not happy as per this issue:
   SeleniumHQ/selenium#15327

2) StrictDoc runs within a container and outputs artifacts that have the
   UID/GID of the host user.

The approaches that I could not make work:

1) Running with a default root user because of the permissions issue and
   Chrome raising an error about the environment.

2) Using `USER` inside the container and then running an image with `--user $(id
   -u):$(id -g)`. With this approach, the Chrome would not be happy
   because the container will be running from a user that does not have
   a proper home directory.

Maybe there are better solutions but the one with the entrypoint.sh
seems to be good enough.
@stanislaw stanislaw added this to the 2025-Q1 milestone Mar 3, 2025
@stanislaw stanislaw merged commit 5bf5069 into main Mar 3, 2025
14 checks passed
@stanislaw stanislaw deleted the stanislaw/meta branch March 3, 2025 14:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants