Skip to content

Commit

Permalink
Build all when building tt_metal (#341)
Browse files Browse the repository at this point in the history
### Issue
No issue

### Description
Build more stuff during "build client" workflow, to match what is done
in tt_metal ci.

### List of the changes
- Added build options --build-all --enable-ccache --enable-profiler

### Testing
CI test

### API Changes
There are no API changes in this PR.
  • Loading branch information
broskoTT authored Nov 27, 2024
1 parent 766cc54 commit 62f8d1b
Showing 1 changed file with 3 additions and 3 deletions.
6 changes: 3 additions & 3 deletions .github/workflows/build-clients.yml
Original file line number Diff line number Diff line change
Expand Up @@ -7,7 +7,7 @@ on:
required: true
description: 'The timeout for the job in minutes'
type: number
default: 30
default: 90
pull_request:
branches: ["main"]
push:
Expand All @@ -17,7 +17,7 @@ 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') }}
timeout-minutes: ${{ fromJSON(inputs.timeout || '90') }}
strategy:
fail-fast: false
matrix:
Expand Down Expand Up @@ -53,4 +53,4 @@ jobs:
export ARCH_NAME=${{ matrix.arch_name }}
export TT_METAL_HOME=$(pwd)
export PYTHONPATH=$(pwd)
./build_metal.sh
./build_metal.sh --build-all --enable-ccache --enable-profiler

0 comments on commit 62f8d1b

Please sign in to comment.