Skip to content

Conversation

@polarG
Copy link
Contributor

@polarG polarG commented Oct 31, 2025

Python3.10 verification pipeline failed for conflict 'setuptools' version as below.
image

Root Cause:
The problem is that modern pip (25.3) uses an isolated build environment with the latest setuptools by default. The pipeline installs setuptools 65.7 in the user environment, but pip builds the package in an isolated environment with newer setuptools, which conflicts with the version check in [setup.py].

Solution:
Remove pip upgrade.

@polarG polarG requested a review from a team as a code owner October 31, 2025 18:39
@codecov
Copy link

codecov bot commented Oct 31, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 85.71%. Comparing base (f6e65a9) to head (e0d198d).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #754   +/-   ##
=======================================
  Coverage   85.71%   85.71%           
=======================================
  Files         102      102           
  Lines        7678     7678           
=======================================
  Hits         6581     6581           
  Misses       1097     1097           
Flag Coverage Δ
cpu-python3.10-unit-test 70.94% <ø> (ø)
cpu-python3.12-unit-test 70.94% <ø> (ø)
cpu-python3.7-unit-test 70.39% <ø> (ø)
cuda-unit-test 83.61% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@abuccts abuccts requested a review from Copilot November 4, 2025 23:58
Copy link
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

This PR adds the --no-build-isolation flag to pip install commands in Azure Pipeline configurations to prevent pip from creating isolated build environments during package installation.

Key changes:

  • Added --no-build-isolation flag to pip install commands in both CPU and CUDA test pipelines
  • Ensures setuptools is explicitly installed for Python 3.11+ in the CPU test pipeline

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.azure-pipelines/cuda-unit-test.yml Added --no-build-isolation flag to pip install command for CUDA worker dependencies
.azure-pipelines/cpu-unit-test.yml Added --no-build-isolation flag and explicit setuptools installation for CPU worker dependencies

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@polarG polarG enabled auto-merge (squash) November 5, 2025 03:15
@abuccts abuccts requested a review from Copilot November 5, 2025 04:12
@abuccts
Copy link
Member

abuccts commented Nov 5, 2025

@copilot revise the PR title accordingly

Copy link

Copilot AI commented Nov 5, 2025

@abuccts I've opened a new pull request, #755, to work on those changes. Once the pull request is ready, I'll request review from you.

Copy link
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

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@abuccts
Copy link
Member

abuccts commented Nov 5, 2025

@abuccts I've opened a new pull request, #755, to work on those changes. Once the pull request is ready, I'll request review from you.

@copilot I mean in this PR, rather than opening a new one..

Copy link

Copilot AI commented Nov 5, 2025

@abuccts I've opened a new pull request, #756, to work on those changes. Once the pull request is ready, I'll request review from you.

@polarG polarG merged commit 1b4377f into main Nov 5, 2025
32 of 33 checks passed
@polarG polarG deleted the hongtaozhang/fix-test-pipeline branch November 5, 2025 05:15
@guoshzhao guoshzhao mentioned this pull request Nov 10, 2025
30 tasks
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.

4 participants