Skip to content

Ensure that azure-storage-extensions is added to build set on changes to other data-plane storage packages#46582

Merged
scbedd merged 1 commit intoAzure:mainfrom
scbedd:patch-storage-triggers
Apr 28, 2026
Merged

Ensure that azure-storage-extensions is added to build set on changes to other data-plane storage packages#46582
scbedd merged 1 commit intoAzure:mainfrom
scbedd:patch-storage-triggers

Conversation

@scbedd
Copy link
Copy Markdown
Member

@scbedd scbedd commented Apr 28, 2026

There is a PR incoming to main that will add azure-storage-extensions to the dev_requirements.txt for the 4 data-plane storage packages.

When that happens, we will immediately see timeouts during the test phase, as if azure-storage-extensions is NOT triggered by a change, then each test platform will need to assemble the wheel on the fly.

This is an expensive assembly that we enable with direct usage of cibuildwheel, so by adding these data plane packages to the triggering paths for azure-storage-extensions, we'll always have azure-storage-extensions added to the build set, and as such will be available for install.

…y of azure-storage-blob, azure-storage-file-datalake, azure-storage-file-share, or azure-storage-queue, changes to those packages properly trigger an assembly of the platform-specific wheels for azure-storage-extensions
Copilot AI review requested due to automatic review settings April 28, 2026 18:20
@scbedd scbedd requested review from lmazuel and msyyc as code owners April 28, 2026 18:20
@scbedd scbedd self-assigned this Apr 28, 2026
@scbedd scbedd moved this from 🤔 Triage to 🔬 Dev in PR in Azure SDK EngSys 🚀🌒🧑‍🚀 Apr 28, 2026
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 storage service CI configuration so azure-storage-extensions is included in the build set when related data-plane storage packages change, avoiding expensive on-the-fly wheel builds during test runs.

Changes:

  • Adds triggeringPaths to the azure-storage-extensions artifact to trigger builds when azure-storage-blob, azure-storage-file-datalake, azure-storage-file-share, or azure-storage-queue change.
Show a summary per file
File Description
sdk/storage/ci.yml Adds artifact-level trigger paths intended to pull azure-storage-extensions into the build set when sibling storage packages change.

Copilot's findings

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

Comment thread sdk/storage/ci.yml
@scbedd scbedd merged commit a1e9a01 into Azure:main Apr 28, 2026
33 checks passed
@kurtzeborn kurtzeborn moved this from 🔬 Dev in PR to 🎊 Closed in Azure SDK EngSys 🚀🌒🧑‍🚀 Apr 28, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Development

Successfully merging this pull request may close these issues.

3 participants