ci: run Python checks on ARC runners#234
Merged
Merged
Conversation
c3ab7c2 to
f78a428
Compare
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
Move the generated Python Test/Lint CI workflows onto the
loongsuite-python-arcARC runner scale set and reduce job fan-out by batching tox environments with
tox run-parallel -p 4.This PR intentionally keeps ACK/ACS runner deployment details out of the public
repository. The runner image, ImageCache, Helm values, registry location, and
cluster rebuild runbook are maintained on the cluster/internal documentation
side. The public repository only needs the GitHub Actions runner label used by
the generated workflows.
Generated
MiscandLoongSuite Miscjobs stay on GitHub-hostedubuntu-latestbecause they include Docker, precommit, docs, and generationchecks that should not depend on the ACS Python runner pods.
Prior fork validation of the same ARC batching approach reduced the broad CI
wall time from about 48 minutes to about 19 minutes.
Type of change
How Has This Been Tested?
tox -e generate-workflowspython -m py_compile .github/workflows/generate_workflows.py .github/workflows/generate_workflows_loongsuite.py .github/workflows/generate_workflows_lib/src/generate_workflows_lib/__init__.py.github/workflows/*.ymlwith PyYAMLpython "$PIPELINE_SKILL_DIR/scripts/check_loongsuite_pr_readiness.py" --repo .PRE_COMMIT_HOME="$(mktemp -d)" tox -e precommitgit diff --check && git diff origin/main...HEAD --checkDoes This PR Require a Core Repo Change?
Checklist:
Changelog, unit-test, and documentation additions are N/A because this is a
generated CI workflow change. It does not change package runtime behavior.
Validation Evidence
PRE_COMMIT_HOME.Runtime and Telemetry Matrix
Runtime scenarios, GenAI telemetry checks, and Weaver live-check are N/A because
this PR only changes generated CI workflows. It does not change runtime package
behavior, instrumentation behavior, span attributes, or telemetry export paths.