Try without venv inside Docker

tenstorrent · Dec 9, 2024 · 71a1bc7 · 71a1bc7
1 parent eae2559
commit 71a1bc7
Show file tree

Hide file tree

Showing 3 changed files with 0 additions and 14 deletions.
diff --git a/.github/actions/docker-run/action.yml b/.github/actions/docker-run/action.yml
@@ -26,11 +26,6 @@ inputs:
     default: |
       -v /dev/hugepages-1G:/dev/hugepages-1G
       --device /dev/tenstorrent
-  prepare_metal_run:
-    description: "Prepare the environment for running Metalium"
-    type: boolean
-    required: false
-    default: false
   install_wheel:
     description: "Install the wheel that contains all of the Python environment. The artifact needs to be present."
     type: boolean
@@ -94,13 +89,6 @@ runs:
         run: |
           set -eu
 
-          prepare_metal_run=${{ inputs.prepare_metal_run }}
-          if [[ "${prepare_metal_run,,}" == "true" ]]; then
-            rm -rf ${PYTHON_ENV_DIR}
-            ./create_venv.sh
-            source python_env/bin/activate
-          fi
-
           install_wheel=${{ inputs.install_wheel }}
           if [[ "${install_wheel,,}" == "true" ]]; then
             WHEEL_FILENAME=$(ls -1 *.whl)

diff --git a/.github/workflows/build-and-unit-tests.yaml b/.github/workflows/build-and-unit-tests.yaml
@@ -64,7 +64,6 @@ jobs:
         with:
           docker_os_arch: ${{ inputs.os }}-amd64
           docker_password: ${{ secrets.GITHUB_TOKEN }}
-          prepare_metal_run: true
           docker_opts: |
             -e ARCH_NAME=${{ inputs.arch}}
             -e TT_METAL_HOME=${{ github.workspace }}

diff --git a/.github/workflows/cpp-post-commit.yaml b/.github/workflows/cpp-post-commit.yaml
@@ -79,7 +79,6 @@ jobs:
         with:
           docker_os_arch: ${{ inputs.os }}-amd64
           docker_password: ${{ secrets.GITHUB_TOKEN }}
-          prepare_metal_run: true
           docker_opts: |
             -e TT_METAL_HOME=${{ github.workspace }}
             -e ARCH_NAME=${{ inputs.arch }}