Skip to content

Adding support for Azure Blob Storage#56

Merged
axelsrz merged 7 commits intomainfrom
users/robrandao/blob
Jul 8, 2025
Merged

Adding support for Azure Blob Storage#56
axelsrz merged 7 commits intomainfrom
users/robrandao/blob

Conversation

@rodrigobr-msft
Copy link
Contributor

This pull request introduces a new library, microsoft-agents-blob, to provide blob storage functionality for Microsoft Agents. It includes updates to CI/CD workflows, the addition of the library's implementation, and its configuration files. Below is a breakdown of the most important changes:

Blob Storage Library Implementation:

  • Added BlobStorage class and BlobStorageSettings class to handle blob storage operations, including reading, writing, and deleting blobs in Azure Blob Storage. (libraries/Storage/microsoft-agents-blob/microsoft/agents/blob/blob_storage.py)
  • Updated __init__.py to expose BlobStorage and BlobStorageSettings via the __all__ attribute. (libraries/Storage/microsoft-agents-blob/microsoft/agents/blob/__init__.py)

CI/CD Workflow Updates:

  • Updated .azdo/ci-pr.yaml and .github/workflows/python-package.yml to include installation of the new microsoft-agents-blob wheel during CI/CD pipelines. (.azdo/ci-pr.yaml) [1] (.github/workflows/python-package.yml) [2]

Library Configuration:

  • Added pyproject.toml file to define the build system, dependencies, and metadata for the microsoft-agents-blob library. (libraries/Storage/microsoft-agents-blob/pyproject.toml)

@rodrigobr-msft rodrigobr-msft requested a review from rido-min June 26, 2025 23:00
@rodrigobr-msft rodrigobr-msft marked this pull request as draft June 26, 2025 23:00
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 adds support for Azure Blob Storage for Microsoft Agents by introducing a new library that implements blob storage operations, comprehensive test coverage, and necessary CI/CD workflow updates.

  • Implemented BlobStorage and BlobStorageSettings classes to handle blob operations (read, write, delete) using the Azure Blob SDK.
  • Updated CI/CD pipelines to install and test the new microsoft-agents-blob wheel.
  • Added a pyproject.toml file to define project metadata and dependencies.

Reviewed Changes

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

Show a summary per file
File Description
libraries/Storage/microsoft-agents-blob/tests/test_blob_storage.py Added tests for blob storage behavior using a mock store item implementation
libraries/Storage/microsoft-agents-blob/pyproject.toml Introduced project metadata and dependency definitions
libraries/Storage/microsoft-agents-blob/microsoft/agents/blob/blob_storage.py Implemented the BlobStorage functionality using the Azure Blob SDK, including container initialization and blob operations
libraries/Storage/microsoft-agents-blob/microsoft/agents/blob/init.py Exposed BlobStorage and BlobStorageSettings via all
.github/workflows/python-package.yml & .azdo/ci-pr.yaml Updated CI/CD workflows to install the new microsoft-agents-blob wheel
Comments suppressed due to low confidence (1)

libraries/Storage/microsoft-agents-blob/microsoft/agents/blob/blob_storage.py:164

  • Consider adding a check for an empty keys list (in addition to checking for None) in the delete() method to prevent unnecessary operations.
        if keys is None:

@rodrigobr-msft
Copy link
Contributor Author

@microsoft-github-policy-service agree company="Microsoft"

@rodrigobr-msft rodrigobr-msft marked this pull request as ready for review July 3, 2025 18:51
@axelsrz axelsrz merged commit 8258a2b into main Jul 8, 2025
4 checks passed
@rodrigobr-msft rodrigobr-msft linked an issue Jul 29, 2025 that may be closed by this pull request
@rodrigobr-msft rodrigobr-msft deleted the users/robrandao/blob branch September 18, 2025 22:30
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.

Blob Storage integration package

2 participants

Comments