Skip to content

Commit

Permalink
Merge pull request #9870 from Alexey-Rivkin/topic/root-files-residues
Browse files Browse the repository at this point in the history
CI/MAD: Run Docker as a user to avoid creating files as root
  • Loading branch information
yosefe committed May 19, 2024
2 parents 7d60651 + 1eb8316 commit d593201
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions buildlib/tools/test_mad.sh
Original file line number Diff line number Diff line change
Expand Up @@ -23,23 +23,24 @@ build_ucx() {

build_ucx_in_docker() {
docker run --rm \
--user "$(id -u)":"$(id -g)" \
--name ucx_build_"$BUILD_BUILDID" \
-e BUILD_SOURCESDIRECTORY="$BUILD_SOURCESDIRECTORY" \
-v "$PWD":"$PWD" -w "$PWD" \
-v /hpc/local:/hpc/local \
$IMAGE \
bash -c "source ./buildlib/tools/test_mad.sh && build_ucx"

sudo chown -R swx-azure-svc:ecryptfs "$PWD"
}

docker_run_srv() {
local test_name="$1"
HCA=$(detect_hca)
sudo chmod 777 /dev/infiniband/umad*
docker run \
--rm \
--detach \
--net=host \
--user "$(id -u)":"$(id -g)" \
--name ucx_perftest_"$BUILD_BUILDID" \
-e BUILD_SOURCESDIRECTORY="$BUILD_SOURCESDIRECTORY" \
-v "$PWD":"$PWD" -w "$PWD" \
Expand Down

0 comments on commit d593201

Please sign in to comment.