Merge branch 'main' into brosko/grays-mock

.github/workflows/on-pr-opt.yml → .github/workflows/build-and-run-all-tests.yml

@@ -1,10 +1,12 @@
-# Optional PR checks
-name: On PR - Optional
+# Build and then run all tests, on all supported archs.
+name: Build and run all tests
 
 on:
   workflow_dispatch:
   pull_request:
     branches: ["main"]
+  push:
+    branches: ["main"]
 
 jobs:
   build-tests:

.github/workflows/build-clients.yml

@@ -0,0 +1,56 @@
+name: Build clients on newest UMD
+
+on:
+  workflow_dispatch:
+    inputs:
+      timeout:
+        required: true
+        description: 'The timeout for the job in minutes'
+        type: number
+        default: 30
+  pull_request:
+    branches: ["main"]
+  push:
+    branches: ["main"]
+
+jobs:
+  build-tt-metal:
+    # Due to parsing bug, fromJSON is used to convert string to number.
+    # In pull_request or push events, the input context is not available, stating the default again here.
+    timeout-minutes: ${{ fromJSON(inputs.timeout || '30') }}
+    strategy:
+      fail-fast: false
+      matrix:
+        arch_name: [grayskull, wormhole_b0, blackhole]
+
+    name: Build tt-metal for ${{ matrix.arch_name }} with newest UMD
+    runs-on: ubuntu-20.04
+    container:
+      image: ghcr.io/tenstorrent/tt-metal/tt-metalium/ubuntu-20.04-amd64:latest
+      options: --user root
+
+    steps:
+      - name: Checkout client repo
+        uses: actions/checkout@v4
+        with:
+          # Clone under tt-metal directory
+          path: tt-metal
+          repository: tenstorrent/tt-metal
+          submodules: recursive
+          lfs: 'true'
+
+      - name: Checkout UMD
+        uses: actions/checkout@v4
+        with:
+          # Clone directly into tt-metal directory for umd
+          path: tt-metal/tt_metal/third_party/umd
+          submodules: recursive
+          lfs: 'true'
+
+      - name: Build tt-metal
+        run: |
+          cd tt-metal
+          export ARCH_NAME=${{ matrix.arch_name }}
+          export TT_METAL_HOME=$(pwd)
+          export PYTHONPATH=$(pwd)
+          ./build_metal.sh

.github/workflows/build-device.yml

@@ -1,19 +1,19 @@
 # Builds device.
 # Build is performed on all supported OS versions.
-name: Build Target
+name: Build Device
 
 on:
-  workflow_call:
-    inputs:
-      timeout:
-        required: true
-        type: number
   workflow_dispatch:
     inputs:
       timeout:
         required: true
-        description: 'The timeout for the build job in minutes'
+        description: 'The timeout for the job in minutes'
         type: number
+        default: 15
+  pull_request:
+    branches: ["main"]
+  push:
+    branches: ["main"]
 
 env:
   BUILD_TARGET: device
@@ -25,7 +25,9 @@ env:
 
 jobs:
   build:
-    timeout-minutes: ${{ inputs.timeout }}
+    # Due to parsing bug, fromJSON is used to convert string to number.
+    # In pull_request or push events, the input context is not available, stating the default again here.
+    timeout-minutes: ${{ fromJSON(inputs.timeout || '15') }}
     strategy:
       fail-fast: false
       matrix:

.github/workflows/build-image.yml

@@ -3,11 +3,17 @@ name: Build and Publish Docker Image
 
 on:
   workflow_dispatch:
-  workflow_call:
+    inputs:
+      timeout:
+        required: true
+        description: 'The timeout for the job in minutes'
+        type: number
+        default: 15
 
 jobs:
   build:
-    timeout-minutes: 15
+    # Due to parsing bug, fromJSON is used to convert string to number
+    timeout-minutes: ${{ fromJSON(inputs.timeout) }}
     strategy:
       fail-fast: false
       matrix:

.github/workflows/build-tests.yml

@@ -37,7 +37,8 @@ env:
 
 jobs:
   build:
-    timeout-minutes: ${{ inputs.timeout }}
+    # Due to parsing bug, fromJSON is used to convert string to number
+    timeout-minutes: ${{ fromJSON(inputs.timeout) }}
     strategy:
       fail-fast: false
       matrix:

.github/workflows/pre-commit.yml

@@ -2,8 +2,11 @@
 name: Run Pre-commit Hooks
 
 on:
-  workflow_call:
   workflow_dispatch:
+  pull_request:
+    branches: ["main"]
+  push:
+    branches: ["main"]
 
 jobs:
   pre-commit:

.github/workflows/run-tests.yml

@@ -45,7 +45,8 @@ env:
 
 jobs:
   test:
-    timeout-minutes: ${{ inputs.timeout }}
+    # Due to parsing bug, fromJSON is used to convert string to number
+    timeout-minutes: ${{ fromJSON(inputs.timeout) }}
     strategy:
       fail-fast: false
       matrix:

.github/workflows/test-runner.yaml

@@ -2,10 +2,17 @@ name: Check runner
 
 on:
   workflow_dispatch:
+    inputs:
+      timeout:
+        required: true
+        description: 'The timeout for the job in minutes'
+        type: number
+        default: 10
 
 jobs:
   check-runners-host:
-    timeout-minutes: 10
+    # Due to parsing bug, fromJSON is used to convert string to number
+    timeout-minutes: ${{ fromJSON(inputs.timeout) }}
     strategy:
       fail-fast: false
       matrix:
@@ -52,7 +59,8 @@ jobs:
           du -h --max-depth=1 | sort -rh
 
   check-runners-docker:
-    timeout-minutes: 10
+    # Due to parsing bug, fromJSON is used to convert string to number
+    timeout-minutes: ${{ fromJSON(inputs.timeout) }}
     strategy:
       fail-fast: false
       matrix: