#11762: split docker image building out of build-artifacts

tenstorrent · Aug 27, 2024 · f126d1b · f126d1b
1 parent 1baea2b
commit f126d1b
Show file tree

Hide file tree

Showing 3 changed files with 84 additions and 12 deletions.
diff --git a/.github/workflows/all-post-commit-workflows.yaml b/.github/workflows/all-post-commit-workflows.yaml
@@ -17,19 +17,26 @@ jobs:
   static-checks:
     uses: ./.github/workflows/all-static-checks.yaml
     secrets: inherit
-  build-artifact:
-    uses: ./.github/workflows/build-artifact.yaml
-    secrets: inherit
-  build-docker-image-2204:
+  build-docker-image-2004:
     uses: ./.github/workflows/build-docker-artifact.yaml
     secrets: inherit
     with:
-      os: ubuntu-22.04-amd64
+      os: ubuntu-20.04-amd64
+  build-artifact:
+    needs: build-docker-image-2004
+    uses: ./.github/workflows/build-artifact.yaml
+    secrets: inherit
   build-artifact-profiler:
+    needs: build-docker-image-2004
     uses: ./.github/workflows/build-artifact.yaml
     with:
       tracy: true
     secrets: inherit
+  build-docker-image-2204:
+    uses: ./.github/workflows/build-docker-artifact.yaml
+    secrets: inherit
+    with:
+      os: ubuntu-22.04-amd64
   # UMD Unit Tests
   umd-unit-tests:
     secrets: inherit

diff --git a/.github/workflows/build-artifact-wrapper.yaml b/.github/workflows/build-artifact-wrapper.yaml
@@ -0,0 +1,71 @@
+name: "Build tt-metal artifacts"
+
+on:
+  workflow_call:
+    inputs:
+      arch:
+        required: false
+        type: string
+        default: '["grayskull", "wormhole_b0"]'
+      build-type:
+        required: false
+        type: string
+        default: "Release"
+      tracy:
+        required: false
+        type: boolean
+        default: false
+        description: "Build with tracy enabled"
+      os:
+        required: false
+        type: string
+        default: "ubuntu-20.04-amd64"
+  workflow_dispatch:
+    inputs:
+      arch:
+        required: false
+        type: string
+        default: '["grayskull", "wormhole_b0"]'
+      build-type:
+        required: false
+        type: choice
+        options:
+          - Release
+          - Debug
+          - RelWithDebInfo
+          - CI
+      tracy:
+        required: false
+        type: boolean
+        default: false
+        description: "Build with tracy enabled"
+      os:
+        required: false
+        type: choice
+        default: "ubuntu-20.04-amd64"
+        options:
+          - "ubuntu-20.04-amd64"
+          - "ubuntu-22.04-amd64"
+
+permissions:
+  actions: read
+  contents: read
+  pages: write
+  id-token: write
+  packages: write
+
+jobs:
+  build-docker:
+    uses: ./.github/workflows/build-docker-artifact.yaml
+    secrets: inherit
+    with:
+      os: ${{ inputs.os }}
+  build-artifact:
+    needs: build-docker
+    uses: ./.github/workflows/build-artifact.yaml
+    secrets: inherit
+    with:
+      arch: ${{ inputs.arch }}
+      os: ${{ inputs.os }}
+      build-type: ${{ inputs.build-type }}
+      tracy: ${{ inputs.tracy }}
diff --git a/.github/workflows/build-artifact.yaml b/.github/workflows/build-artifact.yaml
@@ -1,4 +1,4 @@
-name: "Build tt-metal artifacts"
+name: "[internal] Build tt-metal artifacts impl"
 
 on:
   workflow_call:
@@ -47,13 +47,7 @@ on:
           - "ubuntu-20.04-amd64"
           - "ubuntu-22.04-amd64"
 jobs:
-  build-docker-image:
-    uses: ./.github/workflows/build-docker-artifact.yaml
-    with:
-      os: ${{ inputs.os }}
-
   build-artifact:
-    needs: build-docker-image
     strategy:
       matrix:
         arch: ${{ fromJson(inputs.arch || '["grayskull", "wormhole_b0", "blackhole"]') }}