Skip to content

Implement Azure Store #1703

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Closed
wants to merge 7 commits into from
Closed

Conversation

Katianna1800
Copy link

@Katianna1800 Katianna1800 commented Apr 20, 2025

Description

This change introduces support for Azure Blob Storage as a backend store implementation. It mirrors the functionality and structure of the existing S3Store, including support for upload, download, retries, and health checks. It also adds a test suite and environment variable-based configuration for local and CI testing.

Fixes claim #1542 (issue)

Type of change

Please delete options that aren't relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • Breaking change (fix or feature that would cause existing functionality to
    not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

Implemented an integration test that uploads and downloads a test blob using a real Azure Blob Storage account.

Test relies on credentials and container configuration provided through a .env file.

Tested locally using cargo test --lib azure_store on a development machine.

Validated that the data written to and read from Azure Blob Storage matches byte-for-byte.

Checklist

  • Updated documentation if needed
  • Tests added/amended
  • bazel test //... passes locally
  • PR is contained in a single commit, using git amend see some docs

This change is Reviewable

@CLAassistant
Copy link

CLAassistant commented Apr 20, 2025

CLA assistant check
All committers have signed the CLA.

@MarcusSorealheis MarcusSorealheis changed the title Claim #1542 pull request Implement Azure Store Apr 20, 2025
@MarcusSorealheis
Copy link
Collaborator

This looks like AI spam again.

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.

4 participants