From e9d23932bf4dfa0024c090c717b43b1d00091071 Mon Sep 17 00:00:00 2001 From: Ben Knutson Date: Wed, 18 Feb 2026 02:44:33 +0000 Subject: [PATCH] Refactor Github Action per b/485167538 --- .github/workflows/RunTests.yml | 3 ++- .github/workflows/run_tests_against_package.yml | 9 ++++++--- 2 files changed, 8 insertions(+), 4 deletions(-) diff --git a/.github/workflows/RunTests.yml b/.github/workflows/RunTests.yml index a2851b13eb..f814f170ed 100644 --- a/.github/workflows/RunTests.yml +++ b/.github/workflows/RunTests.yml @@ -209,8 +209,9 @@ jobs: env: GH_TOKEN: ${{ secrets.GITHUB_TOKEN }} GH_REPO: ${{ github.repository }} + STEPS_FIND_ISSUE_OUTPUTS_ISSUE_NUMBER: ${{ steps.find_issue.outputs.issue_number }} run: | - ISSUE_NUMBER=${{ steps.find_issue.outputs.issue_number }} + ISSUE_NUMBER=${STEPS_FIND_ISSUE_OUTPUTS_ISSUE_NUMBER} LABELS=$(gh issue view $ISSUE_NUMBER --json labels -q '.labels[].name') if echo "$LABELS" | grep -q "success-run-2"; then diff --git a/.github/workflows/run_tests_against_package.yml b/.github/workflows/run_tests_against_package.yml index 9ad1f5e642..980e3b0e20 100644 --- a/.github/workflows/run_tests_against_package.yml +++ b/.github/workflows/run_tests_against_package.yml @@ -83,10 +83,10 @@ jobs: - name: Run Tests shell: bash run: | - if [ "${{ inputs.is_scheduled_run }}" == "true" ]; then - FINAL_PYTEST_MARKER="${{ inputs.pytest_marker }}" + if [ "${INPUTS_IS_SCHEDULED_RUN}" == "true" ]; then + FINAL_PYTEST_MARKER="${INPUTS_PYTEST_MARKER}" else - FINAL_PYTEST_MARKER="${{ inputs.pytest_marker }} and not scheduled_only" + FINAL_PYTEST_MARKER="${INPUTS_PYTEST_MARKER} and not scheduled_only" fi # TODO: Use package data for testing and remove the env vars export MAXTEXT_REPO_ROOT=$(pwd) @@ -95,3 +95,6 @@ jobs: export MAXTEXT_PKG_DIR=$(pwd)/src/MaxText # TODO: Fix the skipped tests and remove the deselect flags .venv/bin/python3 -m pytest -v -m "${FINAL_PYTEST_MARKER}" --durations=0 --deselect "tests/aot_hlo_identical_test.py::AotHloIdenticalTest::test_default_hlo_match" --deselect "tests/tokenizer_test.py::TokenizerTest::test_detokenize" + env: + INPUTS_IS_SCHEDULED_RUN: ${{ inputs.is_scheduled_run }} + INPUTS_PYTEST_MARKER: ${{ inputs.pytest_marker }}