Skip to content

Conversation

@scbedd
Copy link
Member

@scbedd scbedd commented Nov 10, 2025

The immediate reason for #43872 will be negated after this pr. But that's only until we want to build/analyze on 3.11 instead of 3.10 or whatever other situation. So that issue will still be executed on.

@l0lawrence I'm touching agentserver because I prefer to not have cruft remain behind on ci.yml defs.

@scbedd scbedd self-assigned this Nov 10, 2025
Copilot AI review requested due to automatic review settings November 10, 2025 22:08
@scbedd scbedd moved this from 🤔 Triage to 🔬 Dev in PR in Azure SDK EngSys 🍔🌭 Nov 10, 2025
Copilot finished reviewing on behalf of scbedd November 10, 2025 22:10
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 migrates the default Python version from 3.9 to 3.10 for build and analyze operations across the Azure SDK for Python repository. This change addresses a pipeline configuration issue by updating the global default and removing a redundant override.

  • Updates the global default PythonVersion from 3.9 to 3.10 in the pipelines globals configuration
  • Removes the PythonVersionForAnalyze: '3.10' override from the agentserver CI configuration, as it will now inherit the global default

Reviewed Changes

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

File Description
eng/pipelines/templates/variables/globals.yml Updates the global PythonVersion variable from 3.9 to 3.10
sdk/agentserver/ci.yml Removes the redundant PythonVersionForAnalyze parameter override now that the global default is 3.10

@l0lawrence
Copy link
Member

Can we test this on the pkgs that relate to core?

@scbedd
Copy link
Member Author

scbedd commented Nov 11, 2025

Can we test this on the pkgs that relate to core?

Definitely caught an issue with that. Thanks for the reminder 👍

new run after adjustments

@scbedd scbedd requested a review from mccoyp as a code owner November 11, 2025 02:32
@scbedd scbedd merged commit 6bfd630 into Azure:main Nov 13, 2025
32 of 35 checks passed
@kurtzeborn kurtzeborn moved this from 🔬 Dev in PR to 🎊 Closed in Azure SDK EngSys 🍔🌭 Nov 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: 🎊 Closed

Development

Successfully merging this pull request may close these issues.

2 participants