From 83274d3f35c0e5a56ca94843be87758f8e01b55a Mon Sep 17 00:00:00 2001 From: Michael Chiou Date: Thu, 16 May 2024 15:20:45 -0700 Subject: [PATCH] #8082: profiler logic check --- .github/workflows/all-post-commit-workflows.yaml | 2 +- .github/workflows/run-profiler-regression.yaml | 14 ++++++++++---- 2 files changed, 11 insertions(+), 5 deletions(-) diff --git a/.github/workflows/all-post-commit-workflows.yaml b/.github/workflows/all-post-commit-workflows.yaml index 0dd3de8a392..7d73eead71f 100644 --- a/.github/workflows/all-post-commit-workflows.yaml +++ b/.github/workflows/all-post-commit-workflows.yaml @@ -35,7 +35,7 @@ jobs: needs: static-checks uses: ./.github/workflows/run-profiler-regression.yaml with: - manual_call: true + build_docker: true secrets: inherit # eager-package-main: # uses: ./.github/workflows/eager-package-main.yaml diff --git a/.github/workflows/run-profiler-regression.yaml b/.github/workflows/run-profiler-regression.yaml index dc646003103..549ecc65062 100644 --- a/.github/workflows/run-profiler-regression.yaml +++ b/.github/workflows/run-profiler-regression.yaml @@ -2,9 +2,15 @@ name: "[post-commit] metal - Run profiler regression" on: workflow_dispatch: + inputs: + build_docker: + description: 'To distinguish workflow_call from workflow_dispatch' + type: boolean + required: false + default: true workflow_call: inputs: - manual_call: + build_docker: description: 'To distinguish workflow_call from workflow_dispatch' type: boolean required: false @@ -12,12 +18,12 @@ on: jobs: build-docker-artifact: - if: ${{ github.event_name == 'workflow_dispatch' }} + if: ${{ inputs.build_docker }} uses: ./.github/workflows/build-docker-artifact.yaml secrets: inherit profiler-regression-dispatch: - if: ${{ github.event_name == 'workflow_dispatch' }} + if: ${{ inputs.build_docker }} needs: build-docker-artifact strategy: # Do not fail-fast because we need to ensure all tests go to completion @@ -55,7 +61,7 @@ jobs: ./scripts/docker/run_docker_cmd.sh --device /dev/tenstorrent -- ./tests/scripts/run_profiler_regressions.sh profiler-regression-call: - if: ${{ inputs.manual_call }} + if: ${{ !inputs.build_docker }} strategy: # Do not fail-fast because we need to ensure all tests go to completion # so we try not to get hanging machines