Skip to content

Commit 45a9c0e

Browse files
committed
Removes redundant Docker ownership adjustment and sets user
Eliminates unnecessary ownership adjustment in the workflow file to streamline setup. Adds explicit user specification to Docker run command for improved consistency and security. Signed-off-by: Christopher Turner <christopher.g.turner@intel.com>
1 parent ea5dcc0 commit 45a9c0e

2 files changed

Lines changed: 1 addition & 3 deletions

File tree

.github/workflows/testbench.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -45,9 +45,6 @@ jobs:
4545
sudo apt-get -y install valgrind alsa-utils libasound2-dev ninja-build
4646
octave octave-signal
4747

48-
- name: Adjust ownership for Docker
49-
run: sudo chown -R 1000:1000 /github/workspace
50-
5148
# testbench needs some topologies.
5249
- name: build test topologies
5350
continue-on-error: true

scripts/docker-run.sh

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -35,6 +35,7 @@ test "$(id -u)" = 1000 ||
3535
set -x
3636
docker run -i -v "${SOF_TOP}":/home/sof/work/sof.git \
3737
-v "${SOF_TOP}":/home/sof/work/sof-bind-mount-DO-NOT-DELETE \
38+
--user=sof \
3839
--env CMAKE_BUILD_TYPE \
3940
--env PRIVATE_KEY_OPTION \
4041
--env USE_XARGS \

0 commit comments

Comments
 (0)