From 62f8d1b345ec2c894289fd0a985d1223222f7d86 Mon Sep 17 00:00:00 2001 From: =?UTF-8?q?Bojan=20Ro=C5=A1ko?= <156314064+broskoTT@users.noreply.github.com> Date: Wed, 27 Nov 2024 18:39:53 +0100 Subject: [PATCH] Build all when building tt_metal (#341) ### 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. --- .github/workflows/build-clients.yml | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/.github/workflows/build-clients.yml b/.github/workflows/build-clients.yml index a2502367..921341c1 100644 --- a/.github/workflows/build-clients.yml +++ b/.github/workflows/build-clients.yml @@ -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: @@ -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: @@ -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