Skip to content

Fix custom Memory Stores LRO poller operation#45662

Merged
dargilco merged 2 commits intomainfrom
dargilco/memory-store-fix
Mar 12, 2026
Merged

Fix custom Memory Stores LRO poller operation#45662
dargilco merged 2 commits intomainfrom
dargilco/memory-store-fix

Conversation

@dargilco
Copy link
Copy Markdown
Member

@dargilco dargilco commented Mar 12, 2026

Validated by running all sync and async Memory Stores samples (/samples/memories), to make sure there are no errors, and view logs to see that the appropriate HTTP request header is now present.

We will add automated tests in our feature branch for the following release.

@dargilco dargilco self-assigned this Mar 12, 2026
Copilot AI review requested due to automatic review settings March 12, 2026 16:58
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

This PR fixes the Memory Stores long-running update operation by ensuring the required Foundry-Features: MemoryStores=V1Preview opt-in header is included on LRO polling requests (sync + async), and bumps the package version/changelog accordingly.

Changes:

  • Add the Foundry-Features header to the custom Memory Stores LRO polling method instantiation (sync and async).
  • Bump azure-ai-projects version to 2.0.1.
  • Add a 2.0.1 changelog entry documenting the bug fix.

Reviewed changes

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

File Description
sdk/ai/azure-ai-projects/azure/ai/projects/operations/_patch_memories.py Adds required Foundry-Features header to sync LRO polling method for Memory Stores updates.
sdk/ai/azure-ai-projects/azure/ai/projects/aio/operations/_patch_memories_async.py Adds required Foundry-Features header to async LRO polling method for Memory Stores updates.
sdk/ai/azure-ai-projects/azure/ai/projects/_version.py Bumps library version to 2.0.1.
sdk/ai/azure-ai-projects/CHANGELOG.md Documents the bug fix in the 2.0.1 release notes.

You can also share your feedback on Copilot code review. Take the survey.

Comment thread sdk/ai/azure-ai-projects/CHANGELOG.md Outdated
Comment thread sdk/ai/azure-ai-projects/azure/ai/projects/operations/_patch_memories.py Outdated
@dargilco dargilco enabled auto-merge (squash) March 12, 2026 17:54
@dargilco dargilco merged commit ae769c4 into main Mar 12, 2026
20 checks passed
@dargilco dargilco deleted the dargilco/memory-store-fix branch March 12, 2026 18:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants