From fab5dac734344105ae107e85c08151758a4a9b4d Mon Sep 17 00:00:00 2001 From: Huy Do Date: Tue, 19 Aug 2025 17:56:07 +0000 Subject: [PATCH] Tweak dependabot to run inductor jobs (#160935) After https://github.com/pytorch/pytorch/pull/160635, I can see dependabot creating the PR to bump `transformers` version at https://github.com/pytorch/pytorch/pull/160807. This a good start, but there are several tweaks we need: 1. Run inductor tests on the PR including one round of perf benchmark, which is always needed. So, we need `ciflow/inductor` label and a `pull_request` trigger for the benchmark 2. Per @anijain2305 feedback, we don't need to update patch version. So, I add a rule to ignore it. Again, we would need to test this out after this lands. Pull Request resolved: https://github.com/pytorch/pytorch/pull/160935 Approved by: https://github.com/anijain2305 --- .github/dependabot.yml | 4 ++++ .github/workflows/inductor-perf-test-nightly-h100.yml | 10 +++++++--- 2 files changed, 11 insertions(+), 3 deletions(-) diff --git a/.github/dependabot.yml b/.github/dependabot.yml index f86e5f08a2bb..944d3fec3565 100644 --- a/.github/dependabot.yml +++ b/.github/dependabot.yml @@ -8,6 +8,9 @@ updates: target-branch: "main" allow: - dependency-name: "transformers" + ignore: + - dependency-name: "*" + update-types: ["version-update:semver-patch"] commit-message: prefix: "[Dependabot] Update" include: "scope" @@ -18,3 +21,4 @@ updates: - "topic: not user facing" - "module: ci" - "module: inductor" + - "ciflow/inductor" diff --git a/.github/workflows/inductor-perf-test-nightly-h100.yml b/.github/workflows/inductor-perf-test-nightly-h100.yml index 2b59777aae8c..dfaec8240d6c 100644 --- a/.github/workflows/inductor-perf-test-nightly-h100.yml +++ b/.github/workflows/inductor-perf-test-nightly-h100.yml @@ -58,9 +58,14 @@ on: required: false type: string default: inductor_huggingface_perf_cuda_h100,inductor_timm_perf_cuda_h100,inductor_torchbench_perf_cuda_h100 + pull_request: + # Changing these files guarantees that this workflow needs to be run + paths: + - .github/workflows/inductor-perf-test-nightly-h100.yml + - .ci/docker/ci_commit_pins/huggingface-requirements.txt concurrency: - group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.ref_name }}-${{ github.ref_type == 'branch' && github.sha }}-${{ github.event_name == 'workflow_dispatch' }}-${{ github.event_name == 'schedule' }} + group: ${{ github.workflow }}-${{ github.event.pull_request.number || github.sha }}-${{ github.event_name == 'workflow_dispatch' }}-${{ github.event_name == 'schedule' }} cancel-in-progress: true permissions: @@ -160,10 +165,9 @@ jobs: name: cuda12.8-py3.10-gcc9-sm90 uses: ./.github/workflows/_linux-test.yml needs: build - if: github.event_name == 'workflow_dispatch' with: build-environment: linux-jammy-cuda12.8-py3.10-gcc9-sm90 - dashboard-tag: training-${{ inputs.training }}-inference-${{ inputs.inference }}-default-${{ inputs.default }}-dynamic-${{ inputs.dynamic }}-cudagraphs-${{ inputs.cudagraphs }}-cppwrapper-${{ inputs.cppwrapper }}-aotinductor-${{ inputs.aotinductor }}-maxautotune-${{ inputs.maxautotune }}-freezing_cudagraphs-${{ inputs.freezing_cudagraphs }}-cudagraphs_low_precision-${{ inputs.cudagraphs }} + dashboard-tag: training-${{ inputs.training || 'true' }}-inference-${{ inputs.inference || 'true' }}-default-${{ inputs.default || 'true' }}-dynamic-${{ inputs.dynamic || 'true' }}-cudagraphs-${{ inputs.cudagraphs || 'true' }}-cppwrapper-${{ inputs.cppwrapper || 'false' }}-aotinductor-${{ inputs.aotinductor || 'false' }}-maxautotune-${{ inputs.maxautotune || 'false' }}-freezing_cudagraphs-${{ inputs.freezing_cudagraphs || 'false' }}-cudagraphs_low_precision-${{ inputs.cudagraphs || 'false' }} docker-image: ${{ needs.build.outputs.docker-image }} test-matrix: ${{ needs.build.outputs.test-matrix }} timeout-minutes: 720