Skip to content

Commit

Permalink
[toolset] 🔥 Revert "drop permissions of container to match host system (
Browse files Browse the repository at this point in the history
TechEmpower#8007)" (TechEmpower#8013)

* Revert "drop permissions of container to match host system (TechEmpower#8007)"

This reverts commit 1cf4924.

* [ci lang-only php] To test CI

* [ci run-all] To test CI
  • Loading branch information
joanhey authored and franz1981 committed Jun 23, 2023
1 parent 0ee961f commit 280af15
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 8 deletions.
7 changes: 0 additions & 7 deletions Dockerfile
Original file line number Diff line number Diff line change
@@ -1,7 +1,5 @@
FROM ubuntu:18.04

ARG USER_ID
ARG GROUP_ID
ARG DEBIAN_FRONTEND=noninteractive
# WARNING: DON'T PUT A SPACE AFTER ANY BACKSLASH OR APT WILL BREAK
# One -q produces output suitable for logging (mostly hides
Expand Down Expand Up @@ -31,9 +29,4 @@ RUN apt-get -yqq update && apt-get -yqq install \

ENV FWROOT=/FrameworkBenchmarks PYTHONPATH=/FrameworkBenchmarks

# Drop permissions of user to match those of the host system
RUN addgroup --gid $GROUP_ID user
RUN adduser --disabled-password --gecos '' --uid $USER_ID --gid $GROUP_ID user
USER user

ENTRYPOINT ["python", "/FrameworkBenchmarks/toolset/run-tests.py"]
2 changes: 1 addition & 1 deletion tfb
Original file line number Diff line number Diff line change
Expand Up @@ -102,5 +102,5 @@ if ! docker network inspect tfb >/dev/null 2>&1; then
fi

test -t 1 && USE_TTY="-t"
docker build -t techempower/tfb --build-arg USER_ID=$(id -u) --build-arg GROUP_ID=$(id -g) - < ${SCRIPT_ROOT}/Dockerfile
docker build -t techempower/tfb - < ${SCRIPT_ROOT}/Dockerfile
exec docker run -i ${USE_TTY} ${EXTRA_DOCKER_ARGS} --rm --network tfb -v /var/run/docker.sock:/var/run/docker.sock -v ${SCRIPT_ROOT}:/FrameworkBenchmarks techempower/tfb "${@}"

0 comments on commit 280af15

Please sign in to comment.