Skip to content

Commit

Permalink
#8082: profiler logic check
Browse files Browse the repository at this point in the history
  • Loading branch information
ttmchiou committed May 16, 2024
1 parent 807544d commit 83274d3
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 5 deletions.
2 changes: 1 addition & 1 deletion .github/workflows/all-post-commit-workflows.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -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
Expand Down
14 changes: 10 additions & 4 deletions .github/workflows/run-profiler-regression.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -2,22 +2,28 @@ 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
default: false

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
Expand Down Expand Up @@ -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
Expand Down

0 comments on commit 83274d3

Please sign in to comment.