Skip to content

Conversation

@scbedd
Copy link
Member

@scbedd scbedd commented Nov 13, 2025

Investigating blocking issue in #44006

It's just spinning attempting to find a package it can build. Since they're already skipping pypy311 in their release builds I'm just mirroring that here.

@scbedd scbedd self-assigned this Nov 13, 2025
Copilot AI review requested due to automatic review settings November 13, 2025 19:44
Copilot finished reviewing on behalf of scbedd November 13, 2025 19:47
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 fixes a CI blocking issue where agentserver tests were hanging and timing out during pypy builds. The solution replaces an imported matrix configuration with an explicit one that excludes all pypy versions (matching the release build configuration).

Key Changes:

  • Replaced $IMPORT statement with explicit platform matrix defining Python 3.9-3.14 (excluding pypy versions)
  • Updated MatrixConfig name from "communication_ci_matrix" to "agentserver" for consistency
  • Removed redundant MatrixFilters since pypy versions are now excluded at the matrix definition level

Reviewed Changes

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

File Description
sdk/agentserver/platform-matrix.json Replaced imported matrix with explicit configuration excluding pypy versions to prevent CI hangs
sdk/agentserver/ci.yml Updated matrix config name, removed redundant pypy filter, and updated comment reference from pypy39 to pypy311

@scbedd scbedd moved this from 🤔 Triage to 🔬 Dev in PR in Azure SDK EngSys 🍔🌭 Nov 13, 2025
@scbedd
Copy link
Member Author

scbedd commented Nov 13, 2025

Existing analyze failure. Service devs can resolve it on next PR.

@scbedd scbedd merged commit 58fcb0a into main Nov 13, 2025
26 of 29 checks passed
@scbedd scbedd deleted the unblock-agentserver-packages branch November 13, 2025 20:25
@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

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants