Skip to content

Adding CI Test to DSL Executor#782

Merged
caiomcbr merged 4 commits intomainfrom
caiorocha/ci_executor_test
Apr 13, 2026
Merged

Adding CI Test to DSL Executor#782
caiomcbr merged 4 commits intomainfrom
caiorocha/ci_executor_test

Conversation

@caiomcbr
Copy link
Copy Markdown
Contributor

No description provided.

@caiomcbr caiomcbr requested a review from Copilot April 10, 2026 23:05
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Adds CI coverage for the DSL executor by introducing a dedicated Azure Pipelines job that runs executor validation against a set of newly added “golden” execution plans generated from accompanying algorithm definitions.

Changes:

  • Added multiple executor test plans (JSON) covering pack/unpack, reduce, TBG, and NVLS scenarios.
  • Added Python algo generators corresponding to those plans (including TBG and NVLS pipeline variants).
  • Added a new Azure Pipelines job + reusable template to deploy H100 runners and execute the executor test suite remotely.

Reviewed changes

Copilot reviewed 18 out of 18 changed files in this pull request and generated 4 comments.

Show a summary per file
File Description
test/executor-tests/exeuction-plans/transfer_pack_tbg.json Adds a TBG variant execution plan for transfer pack test coverage.
test/executor-tests/exeuction-plans/transfer_pack.json Adds baseline transfer pack execution plan used by executor tests.
test/executor-tests/exeuction-plans/reduce_tbg.json Adds TBG allreduce(Simple) plan to validate grouped TB execution.
test/executor-tests/exeuction-plans/reduce_pack_tbg.json Adds TBG allreduce(LL packet) plan for packetized reduce paths.
test/executor-tests/exeuction-plans/reduce_pack.json Adds baseline allreduce(LL packet) plan for executor validation.
test/executor-tests/exeuction-plans/reduce_nvls_pipeline.json Adds NVLS pipeline plan to test loop/pipeline execution in executor.
test/executor-tests/exeuction-plans/reduce_nvls.json Adds NVLS non-pipelined plan to test switch-channel reduce/broadcast.
test/executor-tests/exeuction-plans/reduce.json Adds baseline allreduce(Simple) plan with fused instruction sequence.
test/executor-tests/algos/transfer_pack_tbg.py Adds generator for transfer pack plan using thread block groups.
test/executor-tests/algos/transfer_pack.py Adds generator for transfer pack plan (non-TBG).
test/executor-tests/algos/reduce_tbg.py Adds generator for reduce(Simple) plan using thread block groups.
test/executor-tests/algos/reduce_pack_tbg.py Adds generator for reduce(LL packet) plan using thread block groups.
test/executor-tests/algos/reduce_pack.py Adds generator for reduce(LL packet) plan (non-TBG).
test/executor-tests/algos/reduce_nvls_pipeline.py Adds generator for NVLS pipeline plan using SwitchChannel + loop context.
test/executor-tests/algos/reduce_nvls.py Adds generator for NVLS plan using SwitchChannel reduce/broadcast.
test/executor-tests/algos/reduce.py Adds generator for reduce(Simple) plan with instruction fusion.
.azure-pipelines/ut.yml Adds a new CI job to run DSL executor tests on H100 pool.
.azure-pipelines/templates/ut-executor.yml Introduces a reusable template to deploy, run executor tests, and stop VMSS.

Comment thread .azure-pipelines/templates/exec.yml Outdated
Comment thread test/executor-tests/algos/reduce_pack.py Outdated
Comment thread test/executor-tests/algos/reduce_nvls_pipeline.py Outdated
Comment thread test/executor-tests/execution-plans/transfer_pack.json
@caiomcbr caiomcbr force-pushed the caiorocha/ci_executor_test branch from c0aca60 to 94f1f1a Compare April 10, 2026 23:23
@caiomcbr caiomcbr requested a review from Binyang2014 April 11, 2026 03:19
@caiomcbr caiomcbr merged commit b6d0ca1 into main Apr 13, 2026
7 checks passed
@caiomcbr caiomcbr deleted the caiorocha/ci_executor_test branch April 13, 2026 20:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants