Skip to content

Commit

Permalink
#12775: Incorproate not needing to delete the folders and also mount…
Browse files Browse the repository at this point in the history
… the required files for permissions
  • Loading branch information
dimitri-tenstorrent committed Sep 22, 2024
1 parent 23a1cd7 commit ef67e26
Showing 1 changed file with 6 additions and 5 deletions.
11 changes: 6 additions & 5 deletions .github/actions/docker-run/action.yml
Original file line number Diff line number Diff line change
Expand Up @@ -60,10 +60,15 @@ runs:
password: ${{ inputs.docker_password }}
registry: ghcr.io
image: ghcr.io/${{ github.repository }}/${{ inputs.docker_image }}:${{ env.IMAGE_TAG }}
# The most important option below is `--rm`. The machines will fill up with containers.
# The most important option below is `--rm`. Otherwise, the machines will fill up with undeleted containers.
# The mounting of passwd, shadow, and bashrc is required in order for the correct file permissions
# for newly created files.
options: |
-u ${{ env.RUNNER_UID }}:${{ env.RUNNER_GID }}
--rm
-v /etc/passwd:/etc/passwd:ro
-v /etc/shadow:/etc/shadow:ro
-v /etc/bashrc:/etc/bashrc:ro
-v ${{ github.workspace }}:${{ github.workspace }}
--net=host
${{ inputs.docker_opts }}
Expand All @@ -73,10 +78,6 @@ runs:
-w ${{ github.workspace }}
run: |
if [ ${{ inputs.install_wheel }} ]; then
cp -r * /usr/app/
cd /usr/app/
export PYTHONPATH=/usr/app/
rm -rf tt_metal tt_eager
WHEEL_FILENAME=$(ls -1 *.whl)
pip3 install --user $WHEEL_FILENAME
fi
Expand Down

0 comments on commit ef67e26

Please sign in to comment.