Skip to content

Add azure-storage-extensions to PyPy incompatibility map#46739

Open
Copilot wants to merge 1 commit intomainfrom
copilot/add-azure-storage-extensions
Open

Add azure-storage-extensions to PyPy incompatibility map#46739
Copilot wants to merge 1 commit intomainfrom
copilot/add-azure-storage-extensions

Conversation

Copy link
Copy Markdown
Contributor

Copilot AI commented May 5, 2026

azure-storage-extensions was missing from TEST_PYTHON_DISTRO_INCOMPATIBILITY_MAP, meaning CI could attempt to run its tests on PyPy despite the package (which includes a C extension for CRC64) being incompatible with that distribution.

Changes

  • Added "azure-storage-extensions": "pypy" to TEST_PYTHON_DISTRO_INCOMPATIBILITY_MAP in eng/tools/azure-sdk-tools/ci_tools/functions.py, consistent with the other azure-storage-* packages already excluded from PyPy test runs.

@l0lawrence l0lawrence marked this pull request as ready for review May 5, 2026 22:13
@l0lawrence l0lawrence requested review from mccoyp and scbedd as code owners May 5, 2026 22:13
Copilot AI review requested due to automatic review settings May 5, 2026 22:13
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 Azure SDK Python CI tooling to avoid attempting to run azure-storage-extensions tests on PyPy, where the package is incompatible due to its native extension usage.

Changes:

  • Added azure-storage-extensions to TEST_PYTHON_DISTRO_INCOMPATIBILITY_MAP with a pypy incompatibility entry.

Comment on lines 71 to 82
TEST_PYTHON_DISTRO_INCOMPATIBILITY_MAP = {
"azure-storage-blob": "pypy",
"azure-storage-queue": "pypy",
"azure-storage-file-datalake": "pypy",
"azure-storage-file-share": "pypy",
"azure-eventhub": "pypy",
"azure-servicebus": "pypy",
"azure-ai-projects": "pypy",
"azure-ai-agents": "pypy",
"azure-identity-broker": "pypy",
"azure-storage-extensions": "pypy",
}
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