Skip to content

fix: bump Python 3.8 → 3.10 and drop redundant az cli install#2545

Open
BrendanWalsh wants to merge 1 commit intomasterfrom
brwals/bump-python-build-dep
Open

fix: bump Python 3.8 → 3.10 and drop redundant az cli install#2545
BrendanWalsh wants to merge 1 commit intomasterfrom
brwals/bump-python-build-dep

Conversation

@BrendanWalsh
Copy link
Copy Markdown
Collaborator

Problem

templates/update_cli.yml pins Python 3.8, which is no longer pre-installed on the ubuntu-22.04 agent image (toolset-2204.json only includes 3.10-3.14). Every job downloads 3.8 from GitHub at runtime via UsePythonVersion, which is fragile — build #213983677 hit a 502 and the recommendation job failed.

The template also pip install azure-cli==2.60.0, which actually downgrades the agent's pre-installed az CLI (currently 2.84.0) and wastes ~30s per job.

Changes

  • Python 3.8 → 3.10: Uses the pre-installed version, no download needed
  • Remove pip install azure-cli==2.60.0: The AzureCLI@2 task already uses the agent's pre-installed az CLI

Impact

This template is used by every major job (UnitTests, PythonTests, RTests, Publish, Databricks E2E, Fabric E2E, WebsiteSamples, Release). The change eliminates a runtime download and ~30s of az cli installation across all of them.

Audit

Checked all other build dependencies against the agent image toolset. No other wasteful downloads found — Java 8 uses PreInstalled, conda env creation is necessary (isolated env), and other downloads (graphviz, ffmpeg, Spark tarball) are not on the image.

Python 3.8 is not pre-installed on the ubuntu-22.04 agent image
(toolcache only has 3.10-3.14). Every job was downloading 3.8 from
GitHub at runtime, which is fragile — build 213983677 hit a 502.

Also removes the pip install azure-cli==2.60.0 step. The AzureCLI@2
task uses the agent's pre-installed az (currently 2.84.0), so this
was actually downgrading it and wasting ~30s per job.
Copilot AI review requested due to automatic review settings April 6, 2026 18:44
@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 6, 2026

Hey @BrendanWalsh 👋!
Thank you so much for contributing to our repository 🙌.
Someone from SynapseML Team will be reviewing this pull request soon.

We use semantic commit messages to streamline the release process.
Before your pull request can be merged, you should make sure your first commit and PR title start with a semantic prefix.
This helps us to create release messages and credit you for your hard work!

Examples of commit messages with semantic prefixes:

  • fix: Fix LightGBM crashes with empty partitions
  • feat: Make HTTP on Spark back-offs configurable
  • docs: Update Spark Serving usage
  • build: Add codecov support
  • perf: improve LightGBM memory usage
  • refactor: make python code generation rely on classes
  • style: Remove nulls from CNTKModel
  • test: Add test coverage for CNTKModel

To test your commit locally, please follow our guild on building from source.
Check out the developer guide for additional guidance on testing your change.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Apr 6, 2026

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Snapshot Warnings

⚠️: No snapshots were found for the head SHA 60a6695.
Ensure that dependencies are being submitted on PR branches and consider enabling retry-on-snapshot-warnings. See the documentation for more information and troubleshooting advice.

Scanned Files

None

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

Updates the shared Azure DevOps pipeline template to better align with the Ubuntu 22.04 hosted agent toolset by avoiding fragile runtime downloads and redundant installs.

Changes:

  • Bump UsePythonVersion from Python 3.8 to 3.10 (uses preinstalled agent Python instead of downloading 3.8).
  • Remove the pip install azure-cli==2.60.0 step so AzureCLI@2 uses the agent’s preinstalled az CLI.
Show a summary per file
File Description
templates/update_cli.yml Switches Python selection to 3.10 and removes the pinned azure-cli pip install to reduce job overhead and failures.

Copilot's findings

  • Files reviewed: 1/1 changed files
  • Comments generated: 0

@BrendanWalsh
Copy link
Copy Markdown
Collaborator Author

/azp run

@azure-pipelines
Copy link
Copy Markdown

Azure Pipelines successfully started running 1 pipeline(s).

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Apr 6, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 84.66%. Comparing base (a833941) to head (60a6695).

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #2545      +/-   ##
==========================================
- Coverage   84.70%   84.66%   -0.04%     
==========================================
  Files         335      335              
  Lines       17753    17753              
  Branches     1615     1615              
==========================================
- Hits        15037    15031       -6     
- Misses       2716     2722       +6     

☔ 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.

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