Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[CI]Add distributed UT in CI #1312

Merged
merged 2 commits into from
Jan 24, 2025
Merged

[CI]Add distributed UT in CI #1312

merged 2 commits into from
Jan 24, 2025

Conversation

zxd1997066
Copy link
Contributor

No description provided.

@zxd1997066 zxd1997066 force-pushed the add_xccl_ut branch 3 times, most recently from 9353bb4 to 7d71fb5 Compare January 22, 2025 04:53
.github/workflows/_linux_ut.yml Outdated Show resolved Hide resolved
.github/workflows/_linux_ut.yml Outdated Show resolved Hide resolved
.github/workflows/pull.yml Outdated Show resolved Hide resolved
@zxd1997066 zxd1997066 force-pushed the add_xccl_ut branch 2 times, most recently from a1eaa8e to 0599d8d Compare January 22, 2025 05:06
@chuanqi129
Copy link
Contributor

@zxd1997066 please rebase your PR to the latest main, and the build issue should be fixed by PR #1319

@zxd1997066 zxd1997066 force-pushed the add_xccl_ut branch 9 times, most recently from 082963e to 1b12a20 Compare January 23, 2025 09:41
@chuanqi129
Copy link
Contributor

@zxd1997066 extend the UT workflow log artifact name with ut test scope to resolve the conflict as https://github.com/intel/torch-xpu-ops/actions/runs/12926393635/job/36050487321?pr=1312#step:15:16

@zxd1997066 zxd1997066 force-pushed the add_xccl_ut branch 2 times, most recently from 1ba396c to 0c46a30 Compare January 24, 2025 02:13
@@ -259,5 +274,5 @@ jobs:
if: always()
uses: actions/upload-artifact@v4
with:
name: Inductor-XPU-UT-Data-${{ github.event.pull_request.number || github.sha }}-${{ inputs.abi }}
name: Inductor-XPU-UT-Data-${{ github.event.pull_request.number || github.sha }}-${{ inputs.abi }}-${{ inputs.ut }}
Copy link
Contributor

Choose a reason for hiding this comment

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

We'd better to replace the , as - in the inputs.ut and use it in artifact name

Copy link
Contributor Author

Choose a reason for hiding this comment

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

update in pull.yml

@zxd1997066 zxd1997066 force-pushed the add_xccl_ut branch 2 times, most recently from 5747e2d to f9bda8f Compare January 24, 2025 03:58
@chuanqi129 chuanqi129 merged commit f535fdb into intel:main Jan 24, 2025
8 checks passed
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.

2 participants